Error executing action create on resource 'remote_file[/tmp/prometheus-1.7.1.linux-amd64.tar.gz]'

Net::HTTPServerException
403 "Forbidden"
Resource Declaration:
In /var/lib/aws/opsworks/cache.stage2/cookbooks/prometheus_master/recipes/master.rb
16: remote_file '/tmp/prometheus-1.7.1.linux-amd64.tar.gz' do
17:   source 'https://github.com/prometheus/prometheus/releases/download/v1.7.1/prometheus-1.7.1.linux-amd64.tar.gz'
18:   owner 'prometheus'
19:   group 'prometheus'
20:   mode '0755'
21:   action :create
22: end
23:
Compiled Resource:
Declared in /var/lib/aws/opsworks/cache.stage2/cookbooks/prometheus_master/recipes/master.rb:16:in `from_file'
remote_file("/tmp/prometheus-1.7.1.linux-amd64.tar.gz") do
provider Chef::Provider::RemoteFile
action [:create]
retries 0
retry_delay 2
path "/tmp/prometheus-1.7.1.linux-amd64.tar.gz"
backup 5
atomic_update true
source ["https://github.com/prometheus/prometheus/releases/download/v1.7.1/prometheus-1.7.1.linux-amd64.tar.gz"]
use_etag true
use_last_modified true
cookbook_name "prometheus_master"
recipe_name "master"
owner "prometheus"
group "prometheus"
mode "0755"
end
[2017-10-03T14:52:32+00:00] INFO: Running queued delayed notifications before re-raising exception
[2017-10-03T14:52:32+00:00] ERROR: Running exception handlers
[2017-10-03T14:52:32+00:00] ERROR: Exception handlers complete
[2017-10-03T14:52:32+00:00] FATAL: Stacktrace dumped to /var/lib/aws/opsworks/cache.stage2/chef-stacktrace.out
[2017-10-03T14:52:33+00:00] ERROR: remote_file[/tmp/prometheus-1.7.1.linux-amd64.tar.gz] (prometheus_master::master line 16) had an error: Net::HTTPServerException: 403 "Forbidden"
[2017-10-03T14:52:33+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

This appears to be an issue with your chef recipe, and nothing to do with the alertmanager. I'd suggest taking this up with the recipe author.


Improve Error Handling For Cassandra Persistence Implementation

What changed?

Check if err == context.DeadlineExceeded when checking for timeout error
Replace duplicated error handling logic with a common error conversion function, which can return more detailed error types instead of just InternalServiceError.

Why?
Improve error handling

How did you test it?
Tested locally

Coverage increased (+0.2%) to 59.7% when pulling e5f2a5752d48825f77006b7b717bd379ad39b45e on yycptt:convert-persistence-timeout-error into 6a1c6a0a86b6528cba7467bdc8d12386708ef5da on uber:master.


Format on save hangs and never saves file.

Describe the bug
Unable to format document as it just hangs. Tried both format on Save and Format docuemnt from command pallete
Steps to reproduce

Create a folder with an .fsx file inside.
Type some code
Save with "Format on Save" Active. Simpley Hangs

Link to sample reproduction
No Sample
Expected behaviour
I expect the document to format and save.
Screenshots

Machine info

OS: Mac OS Big Sur 11.6
.NET SDK version: 6.0.100
Ionide version: 5.9.0
Fantomas Version : 4.6.0-alpha-007 (installed as gloabl tool as writing scripts, not projects)

Additional context
I have found the issue listed at https://github.com/ionide/ionide-vscode-fsharp/issues/1606
I have tried the steps in there and fantomas standalone works (see screenshot terminal) and i get a fantomas process running.

Duplicate of #1606


EnvFile pluggin not supported in Idea 2021.2

Plugin no longer appears for test templates nor run configurations

Duplicate


better debug messages and sensitivity

Closes #1143

@whoburg, ready to review

test models please

test models please

passes all tests, merging


README clarification: 9.9x faster than what?

Please forgive my ignorance, what does 9.9x faster mean? faster than what?
https://github.com/Portkey-AI/gateway/blob/fe0899bbf1a0f65f8f751aca8fca1b0a4409acb4/README.md?plain=1#L22

Hi @elgalu here we are benchmarking against a typical ExpressJS server (which is already considered fast) - Portkey, which is an optimized proxy for LLM workflows performs ~9.9x times faster than if you were using an Express JS server.
In real time - the local gateway adds very minimal latency (to the tune of 5-10 millisecond) while the hosted gateway (i.e. available on https://api.portkey.ai/ ) typically adds 20ms latency to your requests.
Hope that answers your question


Pass RUSTDOCFLAGS to rustdoc spawned through cargo test (fixes #4738)

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

This is currently duplication the logic (buggily) of the normal compliation process and doesn't currently include any tests, could both of those be fixed before merging?

If you explain which duplication you mean, and in which way it is buggily done, yes.

Out of curiosity did you attempt to understand how RUSTDOCFLAGS works today? Or RUSTFLAGS? I would have thought that even a cursorty investigation would have pointed you at the definition which has all manner of features such as loading directly from Config as well.

Yep I looked at it, but I couldn't really make heads of how that ends up get called, compared to the codepath for cargo invoking rustdoc, so I just took the bits specific to RUSTFLAGS. Is there an easy way to just reuse build_env from that code?

In the code you are modifying the rustdoc invocation is configured in large part from the compilation local variable, such as when --cfg flags are passed. The Compilation type, tracing backwards, is mostly filled in after the compilation finishes, such as filling all the cfgs.
What you'll probably want to do is add another field to the Compilation which represents custom flags, in this case rustdocflags. You'll then be able to call one of the functions that calls env_args, rustdocflags_args to fill in the Compilation.
Does that help?

Yes! Thank you, I'll try to fix the patch with these new details.

I'm still confused by something: the current call to Context::rustdocflags_args in cargo_rustc::rustdoc is passed the unit for which we are building the documentation, right? If I fill a new Compilation::rustdocflags field in cargo_rustc::compile_targets, how do I know which unit I should use?

I just realised I am supposed to store a map from packages to rustdoc-specific flags. All that remains is a test, I guess.

I added a test.

:pushpin: Commit 0d4970b has been approved by alexcrichton

@bors: r+
Thanks!

:hourglass: Testing commit 0d4970b180918a59f03d2f00e839b3433ce7960f with merge 7f0583bd400dcb6ad59ca4b172c2fd41a9c1e5d9...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 7f0583bd400dcb6ad59ca4b172c2fd41a9c1e5d9 to master...


Ripple node does not give all transactions to the address (Version: 1.6.0)

I am a newbie. I've tried to run rippled on my Ubuntu 18.04 server. I've installed the ripple node according to the instructions https://xrpl.org/install-rippled.html. For some reasons, the ripple node does not give all transactions to the address. For example, there are only 7 transactions on one of the addresses, in fact, there are more than a hundred of them. What could be the reason for it? Where can I look for the cause? In logs?

This is probably because your rippled server is not a full history server. rippled servers do not need to store the entire history of the XRP Ledger to function correctly. The server can function correctly with a very small subset of history. If you execute server_info, what is the value of complete_ledgers? This value is the range of history currently stored by your server. Full history is around 10TB, so it's not always desirable to store full history. In fact, the server continuously deletes older ledgers in order to save space. This process is called online delete. You can adjust the amount of history your server stores via the config file. More information about online delete can be found here: https://xrpl.org/configure-online-deletion.html .
Here are instructions about how to configure full history, if you have the space and that's what you want: https://xrpl.org/configure-full-history.html

Many thanks for the help. Your advices were very useful.


Escape path and entrypoint in typst init output

Fixes #3747.
Alternative to #3752 because I didn't find out that it existed until after I implemented it.
This PR uses a very small and well-known crate shell-escape that will also escape things like tabs or quotation marks or anything else that could cause problems. It also escapes the path used for cd.

I'm fine with closing #3752 in favor of this one @frozolotl .

Thanks!


Evaluation of [Sx:laugh] tags

Thanks for organizing this challenge!
I noticed that the reference annotations of Dev-Part1 contain intervals tagged as [Sx:laugh].
When computing diarization error rates, how are those intervals handled?

are they ignored?
are they assigned to the corresponding Sx speaker?
are Sx and [Sx:laugh] considered as two different speakers?

Such entries are ignored while calculating DER.
On Fri, Jan 27, 2023 at 5:24 PM Hervé BREDIN @.***>
wrote:

Thanks for organizing this challenge!
I noticed that the reference annotations of Dev-Part1 contain intervals
tagged as [Sx:laugh].
When computing diarization error rates, how are those intervals handled?

are they ignored?
are they assigned to the corresponding Sx speaker?
are Sx and [Sx:laugh] considered as two different speakers?

—
Reply to this email directly, view it on GitHub
https://github.com/displace2023/DISPLACE_Baselines/issues/2, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A4YJSMXWUUCMHTC5EWHFGETWUOZN3ANCNFSM6AAAAAAUITNWEM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Team DISPLACE
Indian Institute of Science, Bengaluru,
National Institute of Technology Karnataka, Surathkal
India

Thanks for the precision.


Add Musify (song app)

Hey. I'm GSSOC CONTRIBUTOR.
https://github.com/gokadzev/Musify

The project is already present in the list.


model selection!

now you can change the model that you are using

you can chat with multiple copilots :)

Thank you so much for the PR, this looks great. Tested locally @BishoyHanyRaafat


How to insert an entry by specifying not all attributes ?

How to insert a record with id (auto increment) PostgREST?
I have a function
        axios.post('http://localhost:3000/unitsmeasure', {
            id: 20,
            name: 'name'
        })
        .then(function (response) {
            console.log(response);
        })
        .catch(function (error) {
            console.log(error);
        });

It inserts an entry in the table. Its works.
But when I do not specify the id. It does not work. (id serial PRIMARY KEY).
        axios.post('http://localhost:3000/unitsmeasure', {
            name: 'name'
        })
        .then(function (response) {
            console.log(response);
        })
        .catch(function (error) {
            console.log(error);
        });

This does not work
POST http://localhost:3000/unitsmeasure 401 (Unauthorized)
SQL Code Table
CREATE TABLE "unitsmeasure" (
  "id" serial PRIMARY KEY,
  "name" varchar(100)
)

Create function
	CREATE OR REPLACE FUNCTION views.insertname(fname varchar(20)) RETURNS VOID AS
$$
BEGIN
    INSERT INTO views.unitsmeasure (name) VALUES (fname);
END
$$
  LANGUAGE 'plpgsql';

How to create a query?
http://localhost:3000/rpc/insertname ?

What's the error message on POST http://localhost:3000/unitsmeasure 401 (Unauthorized)?
Maybe the SEQUENCE privilege is missing, perhaps you need to run GRANT USAGE on SEQUENCE unitsmeasure_id_seq TO $your_anonymous_role.
There are docs on stored procedures and how to call them here https://postgrest.com/en/v4.4/api.html#stored-procedures.

@Vladimir-Golub  Did the suggestion worked for you?

Considering this resolved. Also for getting acquainted with PostgREST I suggest first using  curl or any http cli/gui, after that you can use your preferred lib with more confidence.


Multiple table constraints

Show that you can have multiple table constraints for disk-based tables.  This will align the disk-based definition with the memory-optimized definition.

@gbritton1 : Thanks for your contribution! The author, @CarlRabeler, has been notified to review your proposed change.

#sign-off


Glitch Effect Button

This is a(n):

[ ] Update to an existing Animated Button
[ ] Any Error
[x] Proposal to the Repository
[ ] Documentation / README.md changes

I wanna add a button with a cool glitch effect when clicked, made purely in Html and CSS, which is easy to replicate.

@Atharva-3000 Assigned to you
All the best.
Make sure to ⭐ the repo and follow me.
I would love to connect with you all. ❤
https://www.linkedin.com/in/om-gawande/

@Spyware007 I want to contribute.Please assign me

@PayalPadmalyaPrusty You can create another issue and I will assign it to you!


Misleading error message when an option is included in invalid order when running a ballerina service

Description:
ballerina run data_backed_service -c ../ballerina.conf
ballerina: arguments not allowed for services
Run 'ballerina help' for usage.

Here -c ../ballerina.conf  should have been placed before the package/balfile name to fulfill the intention. However, here with this error message user might misunderstand that "options" are not allowed for services where as it is the "arguments" that are not supported by the services.
Affected Versions:
0.982.1-SNAPSHOT

@Manuri Sorry, I edited my response. The edited response should clarify my point regarding why we can't suggest to the user to place -c ../ballerina.conf before the run command

Adding to what @pubudu91 mentioned, even if it is by error, in order to provide an error message as suggested, we would have to specifically check if the arguments match any of the options.
IMO, this is unnecessary additional processing.
Also, the order is specified in the help text:
$ ballerina run --help
...
ballerina run [-e] [--offline] [--observe]
         [--sourceroot] [-c|--config] [-printreturn] [-B]
         <balfile | packagename | balxfile>[:function] [args...]
...

Yes, I'm not suggesting to specifically check anything. In addition to the actual error message we just could provide a hint to let the user know where options should be placed, if that was his intention.
Anyway, if it can be found in the help text may be we don't have to.

With 1.2.0
ballerina run mod1 --b7a.config.file=ballerina.conf
works fine. We have done improvements to messages as well. Also -c option is nolonger valid. Hence closing the issue.


weird inconsistency between macos and ubuntu

I made a gist reproducible test case for ubuntu here: https://gist.github.com/maxogden/0ea1f28b977c79a7a2cf
on macos both a and b from the gist behave as expected. however, on ubuntu b never emits close

If you replace /bin/sh with /bin/bash on ubuntu, the child_process successfully emits the close event.. could this be related to https://github.com/nodejs/node/issues/4432?

If you replace /bin/sh with /bin/bash on ubuntu, the child_process successfully emits the close event.. could this be related to https://github.com/nodejs/node/issues/4432?

I'm a bit busy, so I'm going to wait for a solution to nodejs/node#4432. If you have good solution fell free to tell me or make a pull request.

@AndreasMadsen alright I made a PR here, #3

Added support for the shell option. So at least you can do:
execspawn(command, { shell: process.platform === 'win32' ? 'cmd.exe' : '/bin/bash' });

@AndreasMadsen yay thanks, we are green on travis now :) https://travis-ci.org/maxogden/dat/jobs/100433170
heres my hack: https://github.com/maxogden/dat/blob/master/tests/helpers/spawn.js

@maxogden Lovely. Hopefully we will see a solution from node core.


FIX: [ui] prevent blank page when status is unknown

What type of PR is this?
Bug
Which issue(s) this PR fixes (if exists):
Fixes #210

Hey @liorbd,
Do we support cancelled status in fronted?

@kaplanelad Currently no, This PR will prevent the blank page, and will display the status cell empty

I've also added support for cancelled status is UI

Thanks @liorbd.


Janepaper

ETA: This is my first pass at updating the GLM methods/results and the abstract/intro/data/discussion. I'll definitely be adding more stuff to GLM later, and will most likely update the other sections on Sunday when I'm organizing and proofreading everybody else's sections. Please merge before you all start doing crazy edits to avoid conflicts.
Added description comment to new blank files and re-formated time series stuff into Appendix. GLM updates and abstract/intro/data/discussion updates to come.

Thanks for creating the rest of the empty tex files. I am going to merge


10.0 fix updating base

Original commit history before squashing: https://github.com/yelizariev/addons-yelizariev/commits/10.0-fix-updating-base-backup

lint updates https://github.com/yelizariev/addons-yelizariev/commit/37b08eb95abd64f7e43ebf748f250b610342e028


safari pagehide event not fired

if iPadUseNativeControls = false then
safari pagehide event is not fired.
Any hint or fix appriciated

@kanandaraj Is this still an issue?

@kanandaraj Closing old issues. If you'd like to reopen this, let us know. Thanks!


Mentions in Lists etc seem not to be possible

When trying to @ mention an external identity in a shared channel, the external user is not listed and its not possible to mention him.
The external participant can also not use @ mentions to tag members.
As you see here, there are more users that are complaining about this issue
https://techcommunity.microsoft.com/t5/microsoft-teams/mentioning-in-teams-shared-channels-seems-to-be-not-working-or/td-p/3552474

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Hi @scanum,
i've opened a ticket (#33519103) for this and it was already solved with a link to this list specific user documentation
https://support.microsoft.com/en-us/office/add-and-reply-to-comments-in-list-items-9407aab9-6d43-4756-af72-d38177814a6e
There is one point listed that mentions are not possible for guests & anonymous users.
I think this shared channel feature limitation should also be stated out here in the docs article


⚠ Bofrost.de has degraded performance

In 238d641, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1256 ms

Resolved: Bofrost.de performance has improved in fe2a5d6.


Analyze code coverage of unit and validation tests

I'd like to analyze how well each package is covered by the execution of both unit and validation tests. If the code coverage looks low, we will open new issues to address that.

https://codecov.io/gh/sylabs/singularity-cri


[Bug]: Undocumented side effects of keep_folds_across_sessions

Bug Description
This is not necessarily a bug but it cost me some time to figure out what was happening, because the following behavior is undocumented.
The keep_folds_across_sessions  option uses the mkview function to store the folds. However, by default, mkview includes the curdir (current directory) of the file. (this is defined using the viewoptions variable (docs)). This caused my Neovim to seemingly randomly change the directory when entering files that I opened from another directory before. I did not expect this behavior from an option called keep_folds_across_sessions.
Because of the above, I propose to document this behavior in the README by explaining that mkview is used to implement said option and that mkview saves the lcd by default and how to change that with a link to the NeoVim docs.
Thanks for considering my proposal, and thanks for creating this plugin :)
Relevant Screenshot
No response
To Reproduce
No response
neovim version
v0.9.4

Make sure you have done the following

[X] I have updated to the latest version of the plugin.

good point. added to the docs


June release breaks auto-resizing grouped editors

Issue Type: Bug
Open two columns of editors in the workbench.
Resize one column to minimum width.
Click on the smaller column.
I expect the smaller column to auto-expand as usual, but since June release it doesn't. I have to manually resize each column every time. Please fix or point me to a setting for this.
VS Code version: Code 1.25.1 (1dfc5e557209371715f655691b1235b6b26a06be, 2018-07-11T15:33:29.235Z)
OS version: Darwin x64 17.6.0
Extensions (28)

Extension
Author (truncated)
Version

EditorConfig
Edi
0.12.4

JSDocTagComplete
Hoo
0.0.2

vscode-docker
Pet
0.0.27

snapshot-tools
asv
0.0.7

scratchpad
awe
0.1.0

path-intellisense
chr
1.4.2

gitignore
cod
0.5.0

jshint
dba
0.10.19

vscode-eslint
dba
1.4.12

githistory
don
0.4.1

xml
Dot
2.3.1

gitlens
eam
8.4.1

tslint
eg2
1.0.33

vsc-material-theme
Equ
2.3.0

flow-for-vscode
flo
0.8.2

code-runner
for
0.9.3

vscode-less
mrm
0.6.3

vscode-scss
mrm
0.6.2

python
ms-
2018.6.0

Go
ms-
0.6.84

sublime-keybindings
ms-
4.0.0

ide-purescript
nwo
0.19.0

language-purescript
nwo
0.1.2

material-icon-theme
PKi
3.5.1

sass-indented
rob
1.4.9

elm
sbr
0.18.0

python
tht
0.2.3

vscode-todo-highlight
way
0.5.12

(3 theme extensions excluded)

Duplicate of #51614
It's already done in the insiders builds, and will ship with next release.


Version 1.19.10+ GKE Platform - Proxy config_dump shows incorrect locality labels

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
I tried to do a canary install of istio 1.19 recently and I am noticing something off with locality labels. If I do a config_dump of my proxy, the node locality labels don't look right
     "locality": {
      "region": "us",
      "zone": "us-east4"
     },

I expect to see
     "locality": {
      "region": "us-east4",
      "zone": "us-east4-c"
     },

The proxies pointing to 1.17.3 look correct.
Appreciate any pointers
Vidhya
Version
/istioctl version
client version: 1.19.10
istiod version: 1.17.3
istiod version: 1.17.3
istiod version: 1.17.3
istiod version: 1.17.3
istiod version: 1.17.3
istiod version: 1.17.3
istiod version: 1.19.10
istiod version: 1.19.10
istiod version: 1.19.10
istiod version: 1.19.10
istiod version: 1.19.10
istiod version: 1.19.10
data plane version: 1.17.3 (188 proxies), 1.19.10 (23 proxies)

kubectl version
Client Version: v1.30.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.27.12-gke.1115000
WARNING: version difference between client (1.30) and server (1.27) exceeds the supported minor version skew of +/-1
$

Additional Information
No response

1.19 is EOL and what is the node label or any locality label set on the pod

Node has the correct labels
kl describe node gke-sift-online-expr-classifier-rpc-u-3a62c5be-azym
Name:               gke-sift-online-expr-classifier-rpc-u-3a62c5be-azym
Roles:              <none>
Labels:             beta.kubernetes.io/arch=amd64
                    beta.kubernetes.io/instance-type=n1-standard-8
                    beta.kubernetes.io/os=linux
                    cloud.google.com/gke-boot-disk=pd-standard
                    cloud.google.com/gke-container-runtime=containerd
                    cloud.google.com/gke-cpu-scaling-level=8
                    cloud.google.com/gke-local-ssd=true
                    cloud.google.com/gke-logging-variant=DEFAULT
                    cloud.google.com/gke-max-pods-per-node=32
                    cloud.google.com/gke-netd-ready=true
                    cloud.google.com/gke-nodepool=classifier-rpc-us-e4
                    cloud.google.com/gke-os-distribution=cos
                    cloud.google.com/gke-provisioning=standard
                    cloud.google.com/gke-stack-type=IPV4
                    cloud.google.com/machine-family=n1
                    env=expr
                    failure-domain.beta.kubernetes.io/region=us-east4
                    failure-domain.beta.kubernetes.io/zone=us-east4-a
                    group=classifier-rpc
                    iam.gke.io/gke-metadata-server-enabled=true
                    kubernetes.io/arch=amd64
                    kubernetes.io/hostname=gke-sift-online-expr-classifier-rpc-u-3a62c5be-azym
                    kubernetes.io/os=linux
                    node.kubernetes.io/instance-type=n1-standard-8
                    node.kubernetes.io/masq-agent-ds-ready=true
                    project=sift-online-expr
                    projectcalico.org/ds-ready=true
                    topology.gke.io/zone=us-east4-a
                    topology.kubernetes.io/region=us-east4
                    topology.kubernetes.io/zone=us-east4-a

I know 1.19 is EOL. We want to upgrade 1.17 to 1.19 , then bump to 1.21 (since upgrading from 1.17 to 1.21 is not recommended)
Also with 1.17 -- proxy comes up with the correct region and zone. Not sure why 1.19 does not have correct labels

can you try with 1.21+, I don't think we will backport to 1.19, because it's EOL.

Can I go from 1.17 all the way to 1.21? I thought it was not recommended

Not sure if this is caused by platform,
	var l *core.Locality
	if meta.Labels[model.LocalityLabel] == "" && options.Platform != nil {
		// The locality string was not set, try to get locality from platform
		l = options.Platform.Locality()
	} else {

I am running on GKE. The node labels are set correctly and  since 1.17 is running in the same platform -- I can see that they set the locality right.

Btw, I also tried installing 1.21 for testing and Isee the same thing
{
 "configs": [
  {
   "@type": "type.googleapis.com/envoy.admin.v3.BootstrapConfigDump",
   "bootstrap": {
    "node": {
     "id": "router~<IP_ADDRESS>~istio-gateway-7994564474-8c559.istio-system~istio-system.svc.cluster.local",
     "cluster": "istio-ingressgateway.istio-system",
     "metadata": {
      "HTTP10": "1",
      "NAMESPACE": "istio-system",
      "ISTIO_PROXY_SHA": "bcf6c19288e9d4a133f815657c951539018bc9bb",
      "UNPRIVILEGED_POD": "true",
      "INSTANCE_IPS": "<IP_ADDRESS>",
      "PLATFORM_METADATA": {
       "gcp_project": "sift-online-expr",
       "gcp_gke_cluster_name": "sift-online-expr-us-e4",
       "gcp_project_number": "670798215845",
       "gcp_location": "us-east4",
       "gcp_gce_instance_id": "7639483462336842716",
       "gcp_gce_instance": "gke-sift-online-expr-istio-control-pl-717cae4e-7rxt",
       "gcp_gke_cluster_url": "https://container.googleapis.com/v1/projects/sift-online-expr/locations/us-east4/clusters/sift-online-expr-us-e4"
      },
      "ISTIO_VERSION": "1.21.2",
      "ENVOY_STATUS_PORT": 15021,
      "WORKLOAD_NAME": "istio-gateway",
      "PROXY_CONFIG": {
       "extraStatTags": [
        "region",
        "zone"
       ],
       "serviceCluster": "istio-proxy",
       "drainDuration": "45s",
       "concurrency": 3,
       "terminationDrainDuration": "60s",
       "controlPlaneAuthPolicy": "MUTUAL_TLS",
       "statusPort": 15020,
       "proxyAdminPort": 15000,
       "tracing": {
        "customTags": {
         "CUSTOMER_ID": {
          "header": {
           "name": "CUSTOMER_ID",
           "defaultValue": "none"
          }
         }
        },
        "sampling": 10,
        "zipkin": {
         "address": "otel-collector.opentelemetry-collector.svc.cluster.local:9411"
        }
       },
       "statNameLength": 189,
       "configPath": "./etc/istio/proxy",
       "binaryPath": "/usr/local/bin/envoy",
       "discoveryAddress": "istiod-1-21-2.istio-system.svc:15012"
      },
      "CLUSTER_ID": "Kubernetes",
      "NODE_NAME": "gke-sift-online-expr-istio-control-pl-717cae4e-7rxt",
      "LABELS": {
       "chart": "gateways",
       "istio.io/rev": "1-21-2",
       "install.operator.istio.io/owning-resource": "unknown",
       "heritage": "Tiller",
       "sidecar.istio.io/inject": "false",
       "service.istio.io/canonical-name": "istio-gateway",
       "istio": "ingressgateway",
       "operator.istio.io/component": "IngressGateways",
       "release": "istio",
       "app": "istio-ingressgateway",
       "service.istio.io/canonical-revision": "1-21-2"
      },
      "OWNER": "kubernetes://apis/apps/v1/namespaces/istio-system/deployments/istio-gateway",
      "MESH_ID": "cluster.local",
      "ENVOY_PROMETHEUS_PORT": 15090,
      "PILOT_SAN": [
       "istiod-1-21-2.istio-system.svc"
      ],
      "SERVICE_ACCOUNT": "istio-gateway-service-account",
      "NAME": "istio-gateway-7994564474-8c559",
      "ANNOTATIONS": {
       "prometheus.io/port": "15020",
       "kubernetes.io/config.seen": "2024-07-29T21:20:03.575944899Z",
       "sidecar.istio.io/inject": "false",
       "kubernetes.io/config.source": "api",
       "cluster-autoscaler.kubernetes.io/safe-to-evict": "true",
       "prometheus.io/scrape": "true",
       "istio.io/rev": "1-21-2",
       "prometheus.io/path": "/stats/prometheus"
      }
     },
     "locality": {
      "region": "us",
      "zone": "us-east4"
     },

@hzxuzhonghu  You made a code reference in your earlier comment. Is this the file related to that? I am reading this code, it seems like this particular code sets zone based on Cluster Metadata. My cluster is regional - so cluster url will not have a zone. So I expect zone to be read from the node labels.  Does that happen elsewhere?

I am not familiar with GKE, sorry cannot help much

Appreciate any response on this -- this is blocking us.   If this is intentional, are there other ways to extract zone information in Telemetry?

Here is a related issue to this: https://github.com/istio/istio/issues/50379

I appreciate any help/pointers to this problem. I need to extract zone information into envoy stats. I used to do it in an envoy filter in the older versions by extracting node.locality.region With the 1.19+ I am trying to create a Telemetry resource like this which does not work. If there are other ways/work-arounds to do this, please let me know. We are seriously blocked with our upgrade since we have a lot of usecases (alerts, metrics, dashboards, etc) relying on this.
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: zone-metrics
  namespace: istio-system
spec:
  metrics:
  - providers:
    - name: prometheus
    overrides:
    - match:
        metric: ALL_METRICS
      tagOverrides:
        zone:
          value: "xds.node.locality.zone"
        region:
          value: "node.locality.region"

cc @kyessenov from google may know this well

https://github.com/istio/istio/issues/50379 is probably the cause for the different value - that was traced to a breaking change in some underlying library. If you want a workaround, you can probably inject metadata into xds.node.metadata  with ISTIO_META environment variables.

@kyessenov How do I do this? I tried using Telemetry with "xds.node.locality.zone" and it returned empty. Appreciate your response asap on this
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: zone-metrics
  namespace: istio-system
spec:
  metrics:
  - providers:
    - name: prometheus
    overrides:
    - match:
        metric: ALL_METRICS
      tagOverrides:
        zone:
          value: "xds.node.locality.zone"

To be more specific:

@kyessenov  Any word on this? We are stuck in 1.17 and would like to upgrade. Appreciate your response asap

@kyessenov  Ping again. Appreciate your help on this

Pinging again .. please advise.  Also lmk if this is something that will be fixed in the next release

lifecycle/needs-escalation

/not stale

This is fixed by #50379 I believe.


Adding check for verified flag before login into Users example

I am not sure if it is the best sql way, but it works ok for me.

Thanks for the doc contribution. I've been hesitating whether the verification detail makes the example harder to understand, or whether it will be useful because someone will probably want the logic in there if they are using this for a real API...

@michalskop hey sorry for the delayed response.  I'm thinking of merging this and just wondering if you've tried running the code to be sure it works?


After installing Ollama on macOS, I attempted to run the model using the ollama run llama3.1 command. However, when I tried running the model by inputting "hello," it returned what appeared to be random ASCII characters, which didn't make sense. I then used ollama pull llama2 and ollama pull llama3 to see if that would resolve the issue. While ollama run llama3.1 still resulted in nonsensical output, ollama run llama3 and ollama run llama2 worked perfectly. I thought I would share my experience in case it helps someone facing a similar issue.
OS
macOS
GPU
Apple
CPU
Apple
Ollama version
3.1

It seems that the Q4_0 quant of llama3.1 doesn't work for a subset of Mac users.  What type of Mac are you running?  Can you provide server logs?

If you are running a Mac M1 with 8GB of RAM, the PR https://github.com/ollama/ollama/pull/6260 may fix the issue in the next release.

Hi @erfan-khalaji , this should be fixed now. You'll need to re-pull llama3.1: ollama pull llama3.1 (sorry!). Let me know if you're still seeing it afterwards.

@rick-github @jmorganca
After re-pulling Llama 3.1, the problem was resolved. Thank you, folks!


I want a hybrid(horizontal + vertical) chart

I want the display effect of the following picture, can you support it?

Similar layout, won't be possible  with the current implementation


Portfolio : Hamburger menu

At this point, we:

Added JavaScript linters to project
Added mobile menu functionality

Hi team,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.

Invalid Code Review Request

Kindly use Microverse Readme template and modify the file to match your project and remove sections that don't apply. Please remove all the instructions targeting you. For example: "Describe how a new developer could make use of your project." This is crucial to have a professional and instructive repository. 👌

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


No ol/layer/WebGLPoints found in the packaged code

You can use the source code
import WebGLPointsLayer from'ol/layer/WebGLPoints';
To use WebGLPointsLayer;
But WebGLPoints cannot be used in the packaged code
var vectorSource = new ol.source.Vector({
attributions:'NASA',
});
var wgl = new ol.layer.WebGLPoints({
style: style,
source: vectorSource,
disableHitDetection: true,
});
ol.layer.WebGLPoints is undefined.
But the export has been seen in src/ol/layer.js.
Why is this

It should be fixed by #11715
There are 6.4.4-dev versions at
https://openlayers.org/en/main/build/ol.js
https://openlayers.org/en/main/build/ol.js.map
https://openlayers.org/en/main/css/ol.css
https://openlayers.org/en/main/css/ol.css.map

Please don't use the above URLs directly in your applications. If you really need them, download and host them locally.

I also had this issue when using the packaged code for the workshop, and received the error:
...Uncaught TypeError: ol_layer__WEBPACK_IMPORTED_MODULE_4__.WebGLPoints is not a constructor

The workshop uses the ol package, so this is probably an unrelated issue.

I also had this issue when using the packaged code for the workshop, and received the error:
...Uncaught TypeError: ol_layer__WEBPACK_IMPORTED_MODULE_4__.WebGLPoints is not a constructor

I cannot judge whether this is an unrelated issue, but I solved it by changing the import line.
from:
import {WebGLPoints as WebGLPointsLayer} from 'ol/layer';

to:
import WebGLPointsLayer from 'ol/layer/WebGLPoints';

It looks like the original import line makes an alias for a constructer that doesn't exist (anymore). However, the suggested alias is itself a valid constructer.

I also had this issue when using the packaged code for the workshop, and received the error:
...Uncaught TypeError: ol_layer__WEBPACK_IMPORTED_MODULE_4__.WebGLPoints is not a constructor

I cannot judge whether this is an unrelated issue, but I solved it by changing the import line.
from:
import {WebGLPoints as WebGLPointsLayer} from 'ol/layer';

to:
import WebGLPointsLayer from 'ol/layer/WebGLPoints';

It looks like the original import line makes an alias for a constructer that doesn't exist (anymore). However, the suggested alias is itself a valid constructer.


feat: use hosted zone from another AWS account

Description

Provide a way to create DNS validation record in hosted zone present in another AWS account.
Motivation and Context

In some cases, hosted zone is maintained in a different account (say production) and if want to create a ACM certificate in development AWS account, then this would come handy.
Breaking Changes

Yes, this would be a BREAKING CHANGE. As older module code needs to be provided a provider with default aws provider.
module "acm" {
 # ... omitted

  providers = {
    aws = aws
    # hosted zone is in same aws account
    aws.route53 = aws
  }
}

How Has This Been Tested?

terraform init
terraform validate
terraform plan

So to make an existing usage of this module continue to work after this change, do I need to add the block
  providers = {
    aws = aws
    # use different aws provider
    # if hosted zone is in another aws account
    aws.route53 = aws
  }

to my module "acm" {} block?
Is there a way that the above provider block could be established as the default, so that this will not be a breaking change?

Yes, you have to add provider block to your existing module block. As per the documentation, I do not see a way to pass default provider for aws.route53 without specifying.

I'm trying to do some testing on this over the next few days.

@RishikeshDarandale It looks like #25 is relevant to this. An alternative workflow was suggested there, along with an explanation of why multi-provider ACM would not be implemented. Can you comment on how what is implemented in this PR is different from the case discussed in that Issue?

@douglasnaphas ,

Currently, there is no way to provide different providers to a module (one to create ACM certificate and validate, and another - to create Route53 records) due to a lack of Terraform dynamic configurations in providers meta-argument

This is what I have implemented in this pull request.
I can see the alternate option provided in #25, but I will prefer to do in a single module if possible.

@douglasnaphas , any further comment?

@RishikeshDarandale I have some more thoughts, specifically on the trade-off between (1) accommodating your use case of accessing resources in multiple accounts from the same configuration, and (2) adding in some boilerplate (the providers block) that everyone will have to add even if that use case doesn't apply to them. I'll try to state this in more detail later today or tomorrow.
We will need to get feedback from @antonbabenko, since I'm not a maintainer.

@RishikeshDarandale I have some more thoughts, specifically on the trade-off between (1) accommodating your use case of accessing resources in multiple accounts from the same configuration, and (2) adding in some boilerplate (the providers block) that everyone will have to add even if that use case doesn't apply to them. I'll try to state this in more detail later today or tomorrow.
We will need to get feedback from @antonbabenko, since I'm not a maintainer.

Sure!

My thoughts on this PR
These are my thoughts on this PR as a humble user of this module.
Summary
Given that there are other ways of using a Route 53 zone from another account, and given that I don't agree that creating a cert for another account's domain is necessarily a best practice, I do not think that the functionality added by this PR is worth the boilerplate that it will require for users of the module who are not referencing two accounts when they instantiate the module. Of course I'll defer to any final decision by maintainer @antonbabenko (I think he's the only maintainer).
Reasons
There are other ways to accomplish multi-account usage
Users can accomplish multi-account usage with the workaround described in this comment on #25--creating the cert without validation, and creating the validation resources in the account that owns the Route 53 zone.
Creating a cert for another account's domain may not be a best practice
The motivating use case for this PR is: a prod account owns mydomain.com, and a dev account wants to do something with dev.mydomain.com. In this scenario, I don't think it's best for the dev account to be creating certs for mydomain.com and *.mydomain.com in this case, because it doesn't own those domains. The point of a cert is to say this entity owns this domain. By letting the dev account create certs for the production domain mydomain.com, you're one step closer to having the production domain maliciously or inadvertently point to the wrong thing.
I would prefer, and have used, a setup where the prod account creates an NS record set for dev.mydomain.com, with the value equal to the name servers assigned for the Route 53 zone dev.mydomain.com in the dev account. Then the dev account could create and validate certs with this module for dev.mydomain.com, and only dev.mydomain.com. It could also point dev.mydomain.com and its subdomains to other AWS assets without requiring other multi-account state files.
We should avoid adding boilerplate to accommodate specific use cases
With this PR released, every usage of this module that does not have multiple accounts in the same configuration will have to include the boilerplate
  providers = {
    aws = aws
    aws.route53 = aws
  }

Users who want to upgrade to version 3 of this module (or whatever major version includes this change) will need to go and check the upgrade guide (which someone will have to write) and add the boilerplate. Or their configuration will just break when they try to upgrade, and they'll stay on the old version.
Breaking changes should be introduced judiciously. My impression is that most Terraform users separate their workspaces and state files by account, meaning I do not think it is common or even really advisable to have, for example, the production and dev accounts living together in the same state file. There is too much potential for error or confusion with this kind of setup for it to be making an appearance via the providers argument in every instance of this module.
I also don't think it's a good practice to introduce boilerplate arguments for specific use cases. If we make a habit of this, then for the default usage of this module users will have to copy in a bunch of standard arguments that rarely differ among usages, and the module will become annoying to use. I thought the meaning of due to a lack of Terraform dynamic configurations in providers meta-argument in this comment was that, because we can't use a dynamic block to set the aws.route53 provider by default to the overall provider for the configuration in which the module is used, we aren't going to support multiple accounts fed into this module.

Great details by @douglasnaphas! I have nothing else to add really.


Upgrade Crystal version

Tested it work on 0.27.0

Well, it is incorrect to only update the compiler version without revalidation of the timings. I will, eventually, rerun all the benchmarks with the newer versions of compilers.


Bug: Bicep/ARM: Cannot modify this site because another operation is in progress.

When I try to add function app using the bicep template provided into an existing managed environment (Azure Container App), I got the following error:
Cannot modify this site because another operation is in progress. Details: Id: 21b225f8-715c-44db-8f6c-7e39656e4b88, OperationName: Delete, CreatedTime: 9/15/2023 2:06:55 PM, RequestId: 00000000-0000-0000-0000-000000000000, EntityType: 2 (Target: /subscriptions/XXXX/resourceGroups/RG_XXX_PRD/providers/Microsoft.Resources/deployments/xxx-container-apps)

The bicep is based on the exmaple provided in this repo:
resource kontextContainerAppEnvironment 'Microsoft.App/managedEnvironments@2023-05-02-preview' = {
  name: containerAppEnvName
  location: location
  properties: {
    daprAIInstrumentationKey: kontextContainerAppsInsight.properties.InstrumentationKey
    appLogsConfiguration: {
      destination: 'log-analytics'
      logAnalyticsConfiguration: {
        customerId: logAnalyticsCutomerId
        sharedKey: logAnalyticsSharedKey
      }
    }
  }
}

resource kontextFunctionApp 'Microsoft.Web/sites@2022-09-01' = {
  name: kontextFunctionsAppName
  location: location
  kind: 'functionapp'
  properties: {
    managedEnvironmentId: kontextContainerAppEnvironment.id
    siteConfig: {
      linuxFxVersion: 'Docker|ghcr.io/***/kontext-functions:latest'
      appSettings: [
        {
          name: 'FUNCTIONS_WORKER_RUNTIME'
          value: 'dotnet-isolated'
        }
        {
          name: 'FUNCTIONS_EXTENSION_VERSION'
          value: '~4'
        }
        {
            name: 'DOCKER_REGISTRY_SERVER_URL'
            value: 'ghcr.io'
        }

The same error occurs if I use Azure Portal to add the app.
{
    "status": "Failed",
    "error": {
        "code": "Conflict",
        "message": "Cannot modify this site because another operation is in progress. Details: Id: 21b225f8-715c-44db-8f6c-7e39656e4b88, OperationName: Delete, CreatedTime: 9/15/2023 2:06:55 PM, RequestId: 00000000-0000-0000-0000-000000000000, EntityType: 2",
        "details": [
            {
                "message": "Cannot modify this site because another operation is in progress. Details: Id: 21b225f8-715c-44db-8f6c-7e39656e4b88, OperationName: Delete, CreatedTime: 9/15/2023 2:06:55 PM, RequestId: 00000000-0000-0000-0000-000000000000, EntityType: 2"
            },
            {
                "code": "Conflict"
            },
            {}
        ]
    }
}

Any suggestions about how to handle this error? The managed environment is hosting another web container app without any errors.

I have temporarily hosted the function in a new container app environment and it is working:

However I still want to be able to do this to use one single env for both my web app containers and Azure function app container.

Having the same issue trying to deploy a powershell func app. I'm using an existing Container App Environment and have specified existing managedEnvironments in the bicep. Any idea what is causing the issue? Is it a setting in the Container App Environment?

Having the same issue trying to deploy a powershell func app. I'm using an existing Container App Environment and have specified existing managedEnvironments in the bicep. Any idea what is causing the issue? Is it a setting in the Container App Environment?

I have added this issue to this bug one: https://github.com/Azure/azure-functions-dotnet-worker/issues/1920

Taking a look into this. @FahaoTang @rphoon

Taking a look into this. @FahaoTang @rphoon

Thanks @v-shenoy .
It will be appreciated if you can help to look into another related 'bug': The above worked bicep template will create another resource group with name:
container-app-env-name_FunctionApps_****
I would expect the function container app itself to be created within the same resource group of the Function app itself.

This is by design. For a given container apps environment, we create a resource group to hold the backing container apps. This is to separate the Azure resources managed by the user, and the ones managed by the Microsoft.Web resource provider.

This is by design. For a given container apps environment, we create a resource group to hold the backing container apps. If you use the same environment to create more function apps, the container apps will be created in the same resource group.
This is to separate the Azure resources managed by the user, and the ones managed by the Microsoft.Web resource provider.

This is a strange design to me but I can live with it.
Any update about the original issue I raised?

Hi @v-shenoy  just to follow up this again - is the issue fixed now? i.e. we cannot add azure function app to existing managed environment.

Hi, @FahaoTang. Sorry for the delay here. Can you clarify few things for me?

When was the managed environment created (if possible)?
Are you able to create FAs on other MEs?

I will get back on this from my end with more info as well.

Hi, @FahaoTang. Sorry for the delay here. Can you clarify few things for me?

When was the managed environment created (if possible)?

Are you able to create FAs on other MEs?

I will get back on this from my end with more info as well.

It was created a few months ago for my website container app. I can create FAs with new MEs just not the ones that already have other web apps.

@FahaoTang This was a known bug in the back end which has been fixed, but the issue is persisting in some cases where function app creation was done before the fix. I will manually run a fix-up operation and see if it resolves it.

Hi @v-shenoy  any updates?
I am really keen to deploy my containerized function app to the same Azure Contains Apps environment as my website so that I can share the same file share, etc.

@FahaoTang We have verified from our end that the operation has been cleaned up. Could you check again? If not, then something else is causing this and we need to take a look, in which case please share the function app name and error messages.

I feel this feature is half-cooked and doesn't align with my expected experience of deploying containerized function app into Azure Container Apps. Hence, I've reverted back to use service plan server farms.

Hello @v-shenoy, I'm running into the same issue when creating an Azure Function on an AppService Plan (that has other web apps) using ARM (I'm using Pulumi).
I'm getting the below error:
autorest/azure: Service returned an error. Status= . Cannot modify this site because another operation is in progress. Details: Id: be69ecf9-667f-41c9-8fac-9af2380f2c0d, OperationName: Create, CreatedTime: 3/11/2024 8:44:01 AM, RequestId: 2553d0db-c058-4d24-9632-6cf4ce719199, EntityType: 3
However, I can provision the Azure Function manually through the Portal.
My FA uses VNet integration. I wonder if this is causing Create action by ARM to run into this kind of deadlock.
Please let me know if you need more info.

This was closed but what is the solution?
I'm still facing it when using Azure Function App on Container Deploy via Azure DevOps release pipeline.

I am getting this error deploying Azure Functions App to a Container Apps Environment:
Cannot modify this site because another operation is in progress. Details: Id: ae8c14cd-77da-49bb-b644-77a8851f0a44, OperationName: ContainerAppUpdateSiteConfig, CreatedTime: 9/13/2024 9:01:46 PM, RequestId: b975d295-9e8d-4339-96c6-284b06cb2691, EntityType: 3
It appears that the AzureFunctionAppContainer@1 task is triggering the operation for ContainerAppUpdateSiteConfig twice


Rule processor should support writing complex conditions

Some examples are :

Parenthesized expressions like ((x OR y) AND z)
Expressions with constants x > y*10
Expressions with functions - UPPER(name) = 'FOO'

One option could be to have a code text editor with pop up for suggestions.

@shahsank3t , the current API should support passing any valid SQL expressions in the condition.
I was able to send the below request to update the rule with a complex condition like UPPER(driverName) = 'ABC' OR (truckId = 10 AND driverId=10)
curl 'http://localhost:8080/api/v1/catalog/topologies/1/rules/1' -X PUT -H 'Content-Type: application/json' -H 'Accept: application/json' -H 'Cache-Control: no-cache' --data-binary $'{
  "name": "test",
  "description": "test",
  "streams": [
    "kafka_stream_1"
  ],
  "condition": "UPPER(driverName) = \'ABC\' OR (truckId = 10 AND driverId=10)",
  "actions": [

  ],
  "outputStreams": [
    "rule_transform_stream_1",
    "rule_notifier_stream_1"
  ]
}'


SMIRNOFF: Support charge increments

This is one of the last charge handlers in the toolkits that needs to receive some basic support. Once it does, I think that the to_smirnoff function could be a way to sync up the BCCs from openff-recharge.

Having read into some more of the details, this is a little tricky. A custom set of BBCs (using a ChargeIncrementModelHandler) conflicts somewhat with using the default AM1BCC, but this is undocumented in the SMIRNOFF spec.
Electrostatics in general are tricky, and this case is complicated by the fact that it's a mix of "take it or leave it" data from (AM1 or similar methods) and force field parameters (BCCs). The AM1 charges need to be stored at some point, but the BCCs need to be exposed as differentiable objects.
This is probably the time to make electrostatics in general parsed better; the current approach muddies everything up and prevent BCCs or library charges from being exposed as tunable.
There also needs to be some structure for handling metadata (cutoffs, methods, 1-4 scaling factors, etc.) and dispatching the partial charge assignment methods, ideally in a way that is not too specific to SMIRNOFF.

I think re-applying BCCs is a use case we value, and we probably want to avoid re-calculating AM1 charges. I'll try caching the AM1 charges so that BCCs can be quickly re-applied.

I supposed it's possible that the AM1/AM1-Mulliken call here could be executed by a single function that can also cache AM1BCC values for the <ToolkitAM1BCC> tag ... both could involve similar toolkit calls and caching of results.


Allow #[cfg(test)] in decl_module

Perhaps not super critical, but there is one use case in system in which we are exposing a pretty darn bad transaction only for testing.
https://github.com/paritytech/substrate/blob/740115371b096af4ca2c3b2c860d95ba801cffcb/frame/system/src/lib.rs#L566-L572
Ideally, we should allow such #cfg[test]s in the decl_mdoule, or the test that is using this code should be reformulated to achieve the same goal in another way.

Use a different way. We can add a custom module with this function.

No need for this anymore. We still don't allow it in the new one either, but not important anyhow.


Access token authorization bug

Description of issue:
The token generated with /oauth2/token is unable to use parts of the panel api that is outside of /daemon. This leads to Response 401 when trying to send a GET request to /api/nodes.
How to reproduce it:

Create an OAuth2 Client using an administrator account.
Generate a key with the OAuth2 client's id and secret.
Try sending a GET request to /api/nodes

I have uploaded the script I'm using so you can test it with that too, but this online tool can be used as well:
Tool: https://reqbin.com/
Test script: https://pastebin.com/5YdMsKXS
Expected behavior:
Expected behavior would be to get a Response 200 and a body, containing all the informations of the panel's nodes. Instead what happens is a 401 Unauthorized response with no body content.
Thank you for looking into this.
If you need any more testing or info, feel free to reach out to me either here or on discord.
blackruby#2562

Same issue for me
It seems that the error here is in token. I tried in many ways to send token with puffer_auth cookie and with Authorization Header but this isn't working


Attributes with quotation marks in the value get busted on attributesToString

For example if you have a background image in the style attribute:
elem.attr( 'style', 'background-image: url("http://somepath.to/image.jpg")' );

or even without the quotes since they get added when you get the attribute:
elem.attr( 'style', 'background-image: url(http://somepath.to/image.jpg)' );

Upon saving, the quotes breaks the output of the function attributesToString into
style="background-image: url("");" http:="" somepath.to="" image.jpg=""

Here's my fix:
(function() {
	var proxied = window.ContentEdit.attributesToString;
	window.ContentEdit.attributesToString = function( attributes ) {
		for ( var i in attributes ) {
			if ( typeof attributes[ i ] === 'string' ) {
				attributes[ i ] = attributes[ i ].replace( /\"/g, '\'' );
			}
		}
		return proxied.call( this, attributes );
	};
})();

@bfintal hey bud I'm not getting this exact error I'm getting a different one. I don't believe there should be an issue with replacing " with " within HTML attributes but at the moment the editor isn't escaping attributes - so setting a background image as per your first example causes the editor to output the HTML with attributes that are unescaped e.g style="background-image: url("http://somepath.to/image.jpg")". At least that's what I'm seeing output from the editor.
I believe CT should just be escaping attributes when they're set against the element, will have to take a look at this today and get something in place.

I can see that too, but that's while I'm editing. I see the broken output when I stop editing and when everything gets unmounted.
You can see this if you feed this object into the attributesToString function:
    'style': 'background-image: url("http://somepath.to/image.jpg")'
}```

@bfintal I think that should fix the issue now. Basically similar to your approach I've updated the attributesToString function in the ContentEdit library only instead of replacing " with ' I escape the output (including quotes with ") which I think is the correct behaviour.

:+1:


Adrenaline tuning settings

First of all, so far, it works! So thank you!
But as with every development, some funny things happen:
Fan #0 reports the same fan speed as the newly created fan #8. But Fan #0 is not able to control the speed, only the new one does.

0 RPM setting seems to not be working correctly

And last but not least, fancontrol seems to change the advanced control settings off for GPU and VRAM tuning options

These are just details, though! thank you!!

Because than Fan0 must be coming from LibreHardwareMonitor, while the new one is from the plugin.
As for the tuning options, I'm not sure why they get reset.

I can confirm that FanControl with this plugin changes all the settings, including tuning settings, in Adrenalin. Not sure it applies them tho. RX 6900 XT here.

No issue with the 0RPM for me but i can also confirm that the advanced control switches are automatically disabled by the plugin for GPU and VRAM settings. This is particularly annoying when setting the GPU voltage because every mV counts (at least for 7x00 series) and AMD hasn't bothered to add decimal percentages (so 1% corresponds to 11mV~12mV).

6700 XT, also noticing the plugin disables advanced control switches. Otherwise, it's been a great plugin for me so far!

The zero RPM works for me but setting the graph anywhere below 15% on the fans doesn't work. I also do not believe that if the GPU is over 50c that zero rpm works

Good news:
https://github.com/Rem0o/FanControl.ADLX/discussions/36
0 RPM mode should work properly now.


SB Track2 Sender  scheduleMessage

public void scheduleMessage(Iterable message, ZonedDateTime scheduledEnqueueTime) {}
conniey
The names are conflicting here, because scheduleMessage implies a single message. But you're taking an Iterable. And same with the one below.

Closed in favour of #4506


Project generated by "Holographic DirectX 11 App (Universal Windows) (C++/WinRT)" template uses C++/CLI "sealed" keyword

The keyword "sealed" is used a couple of times in the AppView.h. Would it be possible to use "final" instead to keep it standard compliant?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 372cc44e-2d9f-2daf-b05c-66dad900cc30
Version Independent ID: 7a80fd45-80ec-75c8-5f92-5baec7aa4b8c
Content: Creating a holographic DirectX project - Mixed Reality
Content Source: mixed-reality-docs/creating-a-holographic-directx-project.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @MikeRiches
Microsoft Alias: mriches

We removed the keyword from the next version of the template, it is not necessary with how we are set up in CPP/WinRT 2.0.

Thanks @MikeRiches . Is that version available already? We should make sure the link to HL 2 emulator here points to it.


Navigating back from CupertinoPageRoute/MaterialPageRoute prevents tap gestures from getting recognized

Steps to reproduce

Run following sample on Android or iOS
Tap on "Navigate to next page"
Swipe back to previous page and quickly tap on the button again
The first tap is not recognized
Only tap after few hundred ms gets recognized

Expected results
Tap should be recognized immediately
Actual results
https://github.com/user-attachments/assets/f5826d9c-613e-402b-abce-f0b179a893cb
Code sample
Code sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            ElevatedButton(
              child: const Text('Navigate to next page'),
              onPressed: () {
                Navigator.push(
                  context,
                  CupertinoPageRoute(
                    builder: (context) {
                      return const MyHomePage(title: 'Next page');
                    },
                  ),
                );
              },
            ),
          ],
        ),
      ),
    );
  }
}

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.22.3, on macOS 14.5 23F79 darwin-arm64, locale pl-PL)
    • Flutter version 3.22.3 on channel stable at /Users/dominik/fvm/versions/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (7 weeks ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/dominik/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Android Studio (version 2024.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)

[✓] VS Code (version 1.92.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.96.0

[✓] Connected device (4 available)
    • iPhone SE (3rd generation) (mobile) • C81ADE82-008E-438E-A905-500CE2F8EECA • ios           • com.apple.CoreSimulator.SimRuntime.iOS-17-5 (simulator)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Edit: realized this issue occurs with MaterialPageRoute as well

I use a custom fork with this change to override per-route
https://github.com/moffatman/flutter/commit/8d00c34b9095d8c84b959ecea562b10965e7fc4c
probably an opt-in solution like that will be best to pursue

The issue is reproducible on the latest Flutter stable 3.24.2, master 3.25.0-1.0.pre.303, and also on the previous stable release 3.22.3.

AFAIK, pointers are canceled right after navigation
https://github.com/flutter/flutter/blob/b79ee9282c3403197dece9dfeb82a90abe85a5f3/packages/flutter/lib/src/widgets/navigator.dart#L5620-L5634


MmapDll MmapDll

Driver.MmapDll(); map
dll process with protobuf library crashes

windows10 19044  driver MmapDll    tsl   nonsupport      use protobuf lib


LCD window not working

The ILI9486 LCD controller has an interesting feature, that allows to define a rectangle region on the LCD, followed by data to fill this area. Unfortunately it doesn't work as explained in the data sheet, this often draws outside the defined window and not inside. Looks like the LCD controller won't increase the row/column counters after each data appropriate.
I also suspect the Waveshare developers got aware about the problem, because an alternative method was implemented, that fills the rectangle by drawing lines inside. Unfortunately this is much slower than using a display window :-(
The drawing method is defined in lcd/lcd/LCD_GUI.h:
#define LOW_Speed_Show 1
#define HIGH_Speed_Show 0
Inverse the values of the defines, then you'll see the problem.

They were sending 16bit parameters after each command, the data sheet says the parameters are 8 bit each. This must have confused the LCD controller, with propper 8bit parameters this works now.


[MM-49821] Ability to accept pasting multiple email addresses into the invite modal

Summary
When pasting multiple valid emails separated with  , ,, ; it will be handled as if a user typed all those and pressed enter afterward.
Ticket Link
https://mattermost.atlassian.net/browse/MM-49821
Related Pull Requests
N/A
Screenshots
https://user-images.githubusercontent.com/684680/214080247-d9d46c1c-98a5-4d7d-afcd-0f5feb1c076f.mp4
Release Note
Handle multiple emails at once when inviting users.

/update-branch

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@laneycs Do you think we're okay with this state so we can move on to dev review and QA? Or is there more functionality we want to include at this stage?

Is this meant to be draft still or ready for us to review?

Is this meant to be draft still or ready for us to review?

I had in mind to convert it to regular PR, but I forgot – sorry.

@BenCookie95 Sounds good. I'll add this part to this PR.

@BenCookie95 Sounds good. I'll add this part to this PR.

Thanks Kostas!

New cases handled:

Email pasted that is an existing user is being converted to the actual username
Email that are existing users and the user is already in the list is omitted
Usernames that are valid are being added now

@BenCookie95 Does this work for you? If this prop is triggered more than once, we can assume it's being done by paste. If not, then by type.

@BenCookie95 Does this work for you? If this prop is triggered more than once, we can assume it's being done by paste. If not, then by type.

That works, thanks Kostas

Not sure if this is an issue, as the % it's just being discarded. It's actually 3 tokens separated by a space, so<EMAIL_ADDRESS>%<EMAIL_ADDRESS>-> valid, discarded, valid
But maybe we should handle this some other way, e.g., have a red chip with the "discarded" items cc @annapohorielova111

Your right, I'll look into it. It's related to how I did the search for the username. (It's funny how the MM-50229 frustrates you as well 😛)

@furqanmlk Not sure if this is better, but beyond that is potentially a different UX direction.
https://user-images.githubusercontent.com/684680/217549761-4d23345c-09a9-4442-8ce1-9c7e906fc618.mp4

/update-branch

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

@kostaspt
Can you please fix the conflict ?


Need to be able to select between multiple gamepad / xinput sources.

If the player has more than one gamepad plugged in, we shouldn't just pick the first one.

hi, so i connected my Retrolink nintendo64 usb controller (with no adapter) on my computer and ALL the inputs are wrong except the anolog stick. please help :)

What input source did you select? Generic gamepad? If so, then you have to configure the skin to use the proper buttons.

I implemented this, expect it tomorrow on my branch and as pull request.


Tooltip close handling for auto scenario

Close on click outside
Fixed text wrapping in grid

@dlasky @shuwen - probably could have gone straight to develop with this, but wanted you both to be aware of the changes...

👍

lgtm


Halftone plugin

Hi there. First congratulation for the nice project.
Here is a classic halftone effect that I feel is missing from the actual plugins.

Right now is very simple, but could be possible to add some options to make it more fun

Lovely.
I might try and make it easier to have variations on circles like stipple does. It would be nice to be able to add spiral filled circles easily, possibly I'll stick something in helpers.js.


Perm enum

I want to merge this change because...
resolves #2986

Screenshots

Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.

Could you please fix codeclimate issues?

Yes, I am fixing that.

The ReactJS part needs adjustment, are you able to handle that or shall we?

Alright. I'll handle it.

It just occurred to me that this is going to need some reworking as well:
def format_permissions_for_display(permissions):
    """Transform permissions queryset into PermissionDisplay list.
    Keyword arguments:
    permissions - queryset with permissions
    """
    return [
        PermissionDisplay(
            code='.'.join(
                [permission.content_type.app_label, permission.codename]), name=permission.name)
                for permission in permissions]

to return an enum so the frontend can perform its permission checks. Right now it is comparing string to PermissionEnum, which is crashing it.
Here is what I came up with:
formatted_permissions = []
    for permission in permissions:
        codename = '.'.join(
            [permission.content_type.app_label, permission.codename])
        formatted_permissions.append(
            PermissionDisplay(
                code=graphene.Enum(
                    'PermissionEnum', str_to_enum(codename), codename),
                name=permission.name))
    return formatted_permissions

def format_permissions_for_display(permissions):
    """Transform permissions queryset into PermissionDisplay list.
    Keyword arguments:
    permissions - queryset with permissions
    """
    return [
        PermissionDisplay(
            code='.'.join(
                [permission.content_type.app_label, permission.codename]),
name=permission.name) for permission in permissions]

This is going to need some modification. Because the frontend is going to be making its checks based on the PermissionEnum, and not on strings.
I was doing this:
formatted_permissions = []
    for permission in permissions:
        codename = '.'.join(
            [permission.content_type.app_label, permission.codename])
        formatted_permissions.append(
            PermissionDisplay(
                code=graphene.Enum(
                    'PermissionEnum', str_to_enum(codename), codename),
                name=permission.name))
    return formatted_permissions

What do you think?

@the-bionic If the result is as expected, I'm fine with that

Or actually, you can just use getattr and existing PermissionEnum
Like code=getattr(PermissionEnum, codename)

I used Permission.get(codename) which does about the same thing.

Thanks! I am on it.
I have reverted the snapshot I updated to what it was before. I will investigate further and let you know you know if I need help.

npm run storybook runs without errors, but 2 tests are still failing though.

@the-bionic In such case, please take a look if stories in storybook are rendered properly, especially configuration page stories.

@Pacu2 it renders okay (I am not sure how it would look when it doesn't.) After inspecting the differences in the snapshot, it turns out
<input
                              class="MuiSwitchBase-input-id"
    -                         name="account.impersonate_users"
    +                         name="IMPERSONATE_USERS"
                              type="checkbox"
                            />

That is happening in an input tag. That is what is breaking the test.

@the-bionic I think that's a correct behavior, as we've changed strings to Enums, we should adjust those tests and are good to go.

It can be inspected here, too:
https://travis-ci.org/mirumee/saleor/jobs/440580916

Great. I'll update the snapshot.
You can also see here for the full details: https://travis-ci.org/mirumee/saleor/jobs/440580916

@dominik-zeglen we good?


File manager - why show the folder when it is the only one?

In the picture you can see that the Home folder only have one folder and when i open it, it will show that chevron that the parent folder seems to have more than this folder but when i press on it i get the same folder name as i'm already in.

How hard is it to detect this and never show that chevron sign (or show something else) when the parent folder doesn't have any other folders?
I can then as user easy see if it contains any more folders or not. Maybe you have any bright idea?

Without creating extra load, you can't do it, as far as I remember. by the way, this shows not the folder's content but the parent's folder content.
There is noting to fix here.
Thanks for reporting.


.github: Add a new workflow to cross compile and build

Summary of the PR
In the past, we have seen that there were failure while cross compiling the crate and due to which we broke pipeline of the dependent crates. Thus, add a cross compile build test for the crate to avoid such breakage in the future.
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[ ] All commits in this PR have Signed-Off-By trailers (with
git commit -s), and the commit message has max 60 characters for the
summary and max 75 characters for each description line.
[ ] All added/changed functionality has a corresponding unit/integration
test.
[ ] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[ ] Any newly added unsafe code is properly documented.

@russell-islam There are some tokens missing for CloudHypervisor build. Could you please look into it?


mix gradient: ensure that deps are imported correctly to Gradualizer

I found out that we don't need to import files to the gradualizer_db manually because it is done in the gradualizer_db:init function.
https://github.com/josefs/Gradualizer/blob/ab75f28f9e6f6195c01ddd6a132cd7a07b7e52a2/src/gradualizer_db.erl#L165-L169
So I think I can safely remove the code that imports dependencies in the Gradient mix task. What do you think @erszcz?
https://github.com/esl/gradient/blob/370c895927263efb2ecd08ae6ca280ac58351b40/lib/mix/tasks/gradient.ex#L102-L109


Lost overrides after swapping

• duplicated symbol page into library file (hosted in Abstract)
• swapped all symbols with ones from library
• lost majority of overrides (in nested symbols)
• all symbols names have remained the same

+1 this issue. but it seems to be a Sketch problem. I think it sometimes happens even if you have 2 versions of the same symbol (e.g. just different color) and they have exactly the same nested symbols. Sometimes swapping one for another just doesn't "remember" which of the nested symbols were selected, and instead displays either this "missing symbol" or just selects the nested symbol that is used by default in the parent symbol, and ignores the overrides.

@abertossi Are you still having issues? I don't use Abstract, I don't know if that could be causing problems or not. Please try the latest versions of the Symbol Swapper (version 0.14 is now available), and see if that solves anything for you. Nested symbols and overrides shouldn't have data loss as long as the layer structures and names are similar.

I just tested it with v0.14 and the issue still exist.
When I have a symbol library in Abstract and import the symbols into a project the overrides are gone.

I’ve been getting this problem with my text symbols through shared library’s (using abstract) on sketch 51.3 & 51.2. My text symbols are within nested overrides. Symbol swapper also refuses to swap local  symbols that are currently in use on the design under the option “swap by name” & “swap by ID” :(

I have the same problem! Please fix it.
My problem:
I have a library A and a library B.
In one separate document i build my entire screens with the library A components.
In the document i override some text, symbols, etc.
When i'm swapping the library, i lost the whole overrides.


Comparison classes only generate in database class

Is your feature request related to a problem? Please describe.
I split my code in three parts. The first part is the frontend, the second the backend and a shared part.
In the shared part I have all my row classes and table definition. The database classes are in the separate front and backend package. The problem: I cannot access to the comparison class like UsersComparison to add this to the custom row class. These classes are in the package where the database is defined.
Describe the solution you'd like
The generator should generate the comparison classes in the same file (user.g.dart) like the table class.

I think this is more or less a duplicate of https://github.com/simolus3/moor/issues/490

Yes, I'll close this as a duplicate.


Need to be able to write tenant key to a file or specify your own tenant key

In V4, a new tenantKey requirement was added for multi tenant support.
We are required to use the peristence API with the --allocate-tenant option to add tenant and get key.
The key is not output other than in a very verbose amount of other output which then requires any script that is attempting to automate the onboarding of a tenant to the FHIR server to grep  the output for a specific information message line of text and parse out the key.
If that message were to ever change the script would be broken. The current design does not lend itself to automation of user subscription and provisioning
My suggestion is two-fold, either or both of the items would make life easier for automation - like when provisioning a tenant for PDM which includes installing FHIR and provisioning the tenant there as well:

Add an option to output the tenant key to a file. E.g. --allocate-tenant  --tenantKeyOutputFile <path/file>

AND/OR

Allow the caller to "bring their own key". I.e. let the caller pass in the  tenant key to use - document any required format/valid characters in the persistence doc of course
E.g.
--allocate-tenant  --tenant-key

Added bring your own feature and write-out if the key file doesn't exist.
The code checks if a tenantkey already exists when allocating and adding a tenantkey

Merged in for 4.2.0


cannot install service account token due to install order

Given a chart containing the following template
apiVersion: v1
automountServiceAccountToken: false
kind: ServiceAccount
metadata:
  name: tmp
secrets:
- name: tmp
---
apiVersion: v1
kind: Secret
metadata:
  name: tmp
  annotations:
    kubernetes.io/service-account.name: tmp-token
type: kubernetes.io/service-account-token

Helm will install the service account token (Secret) first and Kubernetes will instantly delete this secret, since no service account is available for this secret (see Token Controller).
Then Helm installs the service account, but the token is gone.
When doing an upgrade of the Helm release after the initial install the token is generated as expected.
The reason for this behavior is the current KindSortOrder where secrets come before service accounts.
Currently I cannot see why one would ever need a secret before a service account, but there are good reasons for the other way around.
I propose to change the order and install service accounts before secrets.
Output of helm version:
version.BuildInfo{Version:"v3.0.0", GitCommit:"e29ce2a54e96cd02ccfce88bee4f58bb6e2a28b6", GitTreeState:"clean", GoVersion:"go1.13.4"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.4", GitCommit:"a87e9a978f65a8303aa9467537aa59c18122cbf9", GitTreeState:"clean", BuildDate:"2019-07-08T08:51:16Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.3", GitCommit:"b3cbbae08ec52a7fc73d334838e18d17e8512749", GitTreeState:"clean", BuildDate:"2019-11-16T01:01:59Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
kind

After triaging this with @bridgetkromhout, it looks like the behaviour seems in line with Kubernetes' expected behaviour. It's unclear for the reasoning behind the ServiceAccount being created after the Secret. I think it should be safe to re-arrange the install/uninstall order to create the ServiceAccount first. I think it may be worthwhile to do a bit of digging first to see if there was a reason that the resources were to be created in that particular order. Would you be willing to look further into this? Thanks!

@bacongobbler I did a quick research. The order Secret vs. ServiceAccount in the current helm source originates in https://github.com/helm/helm/pull/1238 discussed in https://github.com/helm/helm/issues/1228.
https://github.com/helm/helm-classic/blob/master/action/install.go#L18 was mentioned as source for the proposed order. The order was refactored in https://github.com/helm/helm-classic/pull/271 but service accounts were originally added to the sort order in https://github.com/helm/helm-classic/pull/244, discussed in https://github.com/helm/helm-classic/issues/237. I did not find any evidence on why this specific order was chosen. It seems there was no discussion about the order at the time.

After triaging this with @bridgetkromhout, it looks like the behaviour seems in line with Kubernetes' expected behaviour. It's unclear for the reasoning behind the ServiceAccount being created after the Secret. I think it should be safe to re-arrange the install/uninstall order to create the ServiceAccount first. I think it may be worthwhile to do a bit of digging first to see if there was a reason that the resources were to be created in that particular order. Would you be willing to look further into this? Thanks!

hi， seems "fix(helm): move ServiceAccount before Secret in InstallOrder. " don't solve the issue of "https://github.com/helm/helm/issues/6036 helm rollback unsuccessful with service account", I'm using helm v3.6.0, Any help would be appreciated.

Had the same problem in helm version 3.5.3


Allow to set specific terraform version

Consider removing terraform binary from executor image and download/install on demand specific version.

parking this one for now


Updating pgcat through Debian package resets config

Describe the bug
We have a system setup with pgcat and chose the APT repository as an installation method, just as described in the official docs. For the original installation, this works fine so far, but yields problems with upgrading the package: Each time we've upgraded the package, the service got stopped and the configuration file reset to the package maintainer's version. Effectively, this disabled pgcat and caused issues for all applications using pgcat (rendering a fatal exception).
To Reproduce
Since the APT repository only provides the most recent version of pgcat, the following steps "trick" APT into upgrading to the same version again, simulating an upgrade.
Steps to reproduce the behavior:

echo "deb [trusted=yes] https://apt.postgresml.org jammy main" | sudo tee -a /etc/apt/sources.list
sudo apt-get update
sudo apt-get install pgcat
Modify /etc/pgcat.toml to make some changes to the default configuration
sudo systemctl start pgcat
Trick APT into a fake, new version. Edit /var/lib/apt/lists/apt.postgresml.org_dists_jammy_main_binary-amd64_Packages and increase the version for pgcat temporarily:
Before:Package: pgcat
Version: 1.1.2-dev4
# ...

After:Package: pgcat
Version: 1.1.2-dev5
# ...

sudo apt-get upgrade pgcat
⚡ Check /etc/pgcat.toml: The previous modifications were lost, the file got replaced during the upgrade
⚡ Check the service status with sudo systemctl status pgcat: pgcat is no longer running.

Expected behavior
Upgrading pgcat through the official APT repository should 1) keep the current user configuration and 2) ensure that pgcat is in the same state as before (i.e., running).
Environment:
Distributor ID:	Ubuntu
Description:	Ubuntu 22.04.3 LTS
Release:	22.04
Codename:	jammy

pgcat 1.1.2-dev4

https://github.com/postgresml/pgcat/actions/runs/7082031269

Hi @levkk,
thanks for working on the changes. Unfortunately, I am not able to verify the fix you've made for two reasons:

pgcat is still not running after an upgrade, but still listed as Active: inactive (dead)
Now, the config file is not overwritten, but removed entirely. Instead, the example /etc/pgcat.example.toml is provided.

Hence, I would kindly ask to have another look at this issue. Many thanks for your time and effort in fixing this bug.
Please let me know if anything is unclear!

Happy New Year and all the best for 2024!
I just wanted to check in with this issue again, @levkk. As I mentioned, I don't think that this issue has been resolved or should be closed.
Therefore, I am writing again to consider reopening this issue. Otherwise, I'll open another one referencing this one.


More robust install.sh

The ./install.sh is pretty lacking and requires a lot of user intervention. I've tried to make it more robust and support several Linux distros other than Ubuntu/Debian.
I'm currently using EndeavourOS (Arch based) and we're on Python 3.8. Tensorflow 1.15 only works up to Python 3.6.
Because of this, I added a check to see what version of Python you're on. If you're not on 3.6 it'll ask you to manually install it (no way I was going to try to manage that.) or it allows you to enter a manual path for pip3.6.
Also during my testing, it was annoying that the script just assumed I didn't have any problems if it found ./generator/gpt2/models so now it asks if you want to reinstall.
I've tested this script in several different ways and it has passed all of my tests, but I don't have any other distros installed to try it out on.
I'm not going to pretend I'm a Bash wizard so any feed back is appreciated.

Thanks for doing that! Can someone test this and confirm that it works?

Made all commented changes, but also fixed #96 by changing the torrent URL.

Lol I'm not a shell wizard either, but this does look pretty good.


Added Zeal support.

Zeal handles the Dash URI differently, at least on Windows. Instead of
"open -g " it has to call "zeal ". Which also doesn't work
sometimes for some reason or other.
I've only tested this on a Windows 10 machine, will do more testing this week on a VM for Linux and Windows if I can find a copy. Can anyone confirm if the lack of support was just a Windows thing?

Coverage decreased (-90.0%) to 10.0% when pulling 9977150b9c9d4a8336508658ac6a39fb802e176a on szhongren:master into b598f48c8e672bc0c47791a4c26809f02d113390 on deerawan:master.

Coverage remained the same at 100.0% when pulling ada6c74f05432a498df87ab389df40c002b72492 on szhongren:master into b598f48c8e672bc0c47791a4c26809f02d113390 on deerawan:master.

It should work on both Linux and Windows now with Zeal right out of the box. Will do more testing and I'll rewrite the tests in another commit if you are okay with the changes.

looks good, you can start fixing the test 😃

Coverage remained the same at 100.0% when pulling 99066afd74e579179f5b6aa95d986d70f535fe6a on szhongren:master into b598f48c8e672bc0c47791a4c26809f02d113390 on deerawan:master.

Coverage remained the same at 100.0% when pulling a2f8dce98ebfbac8ab94caba682ebd43b54c0319 on szhongren:master into b598f48c8e672bc0c47791a4c26809f02d113390 on deerawan:master.

Thank you @szhongren for your contribution 👍

No prob, thanks for making this extension in the first place!


bump tailwind to v3.3.3

This looks obsolete now with #103 @josevalim

Yes, looks like this is superseded by #103,


Getting error while launching  roslaunch vrx_gazebo vrx.launch

Original report (archived issue) by Anonymous.

Unable to find uri[model://ocean_waves].
I am using ros melodic gazebo 9

I believe this issue can be closed.

I'm closing this issue. Please reopen if you still have the error with the latest released version.