[GHA] bump up timeout to 40mins

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/diem/diem/tree/main/developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

Can you also open an issue for this job? The issue is to look into reducing docker build times (DLC is not leveraged in this job) and build trigger (does it need to build all docker images all the time? if not, can we pull existing images published to GH docker registry?)
ticket is lodged: https://github.com/diem/diem/issues/9674

/land


Storyblok-nuxt is making a GET request even on static website

Hi. I'm switching between dynamic routes with nuxt-link. Those routes are being populated by Storyblok. I noticed that switching between pages was very slow, taking like 1-2 seconds. When I generated a static site with 'npm run generate', it was still very slow, even though everything was now hard-coded. I then disconnected from Internet and then It was giving me a network error in console:
VM1200:1 GET https://api.storyblok.com/v1/cdn/stories/category/causes?version=published&cv=1591282427418&token=W7dzRkbNVkoPKhCzUNP2fwtt net::ERR_INTERNET_DISCONNECTED
However, when I refresh the page, the hard-coded content loads properly.
So kindly guide as to why it is too slow with nuxt-link and why it is making a GET request even though the content is hard-coded and how can I fix it.
Thanks.

I see, thank you for your response.

Nuxt generate currently does not do a full static generation for page transitions. As soon as you're transitioning pages you switch into a client side rendering mode. Only the first request will be static. The team behind nuxt is currently working on a full static deployment.

Hi,
Any idea if there is a PR for this and if/when it may be sorted? Are there any work arounds?


Add automated continuous benchmarking

Closes #1634

In order to get consistent results (there's a bunch of noise in VM performance: https://bheisler.github.io/post/benchmarking-in-the-cloud/), might be worth trying https://github.com/bheisler/iai.

Thanks for implementing this! Sorry it took me a while to look at this, I have a few questions:

Do you think it's possible to do in a single job instead of another matrix? It seems unneccessary to split this up into so many small CI jobs.
Is it possible to also add the benchmarks from pyo3-benchmarks example?
What's the URL to see the graphs? (Let's add this to the guide? 😃)

Do you think it's possible to do in a single job instead of another matrix? It seems unneccessary to split this up into so many small CI jobs.

Done.

Is it possible to also add the benchmarks from pyo3-benchmarks example?

Will try add that later.

What's the URL to see the graphs? (Let's add this to the guide? 😃)

I think it will be https://pyo3.rs/dev/bench/ , it adds a dev/bench directory in gh-pages branch.

In order to get consistent results (there's a bunch of noise in VM performance: https://bheisler.github.io/post/benchmarking-in-the-cloud/), might be worth trying https://github.com/bheisler/iai.

iai is a relatively new project. I don't think github-action-benchmark supports its output format yet.

Great! Let's merge this and start getting a feel for what this tells us.


Garage Door Opens but does not close

As per the title, I can use the webpage to open/stop but not close the garage door, any help would be appreciated.

What firmware are you running, and do you have obstruction sensors attached? Does the Toggle command work to close the door rather than the discrete Close command?

Thanks for the reply, seems like it is a known issue with certain GDO's that don't have obstruction sensors attached as they are optional.
Yes toggle command works.
Looks to be a couple of solutions, buy obstructions sensors or potentially wait to see if it can be bypassed. Going to close the issue as I forgot to check the main ratgdo page first and its already reported underhttps://github.com/ratgdo/esphome-ratgdo/issues/146

Yup, that's about the sum of it.
It seemed that at least one person was able to make it work, but for most, it's either add obstruction sensors, or you could use a workaround like the MQTT code where you toggle the door conditionally on whether it is open when you need to specifically close it. Of course, that requires that the RATGDO or RAT-RATGDO always have accurate statuses, so it is a little sketchy sometimes.


Bad Escape

 """Hello world"""  print

---Data stack:
""
"Hello world"
""
On  Mac OS X  EI Captian, latest Build.

@erg removed support for triple-quote strings in the nightly builds (you can still have new lines in normal strings and will have to escape "\"" if you need quotation marks).  We are planning on supporting Lua-style long strings ([[hello world]]) instead. @erg felt the triple-quote strings were a little inconsistent, but it will be a backwards-compatibility issue unless we bring them back in addition to Lua strings.

@erg should we bring triple quote strings back for 0.98 since new parser is 0.99?


Would it be possible for CQL Proxy to authenticate using the clients credentials?

So when a client connects, could it authenticate a new connection for that client using the client's Astra username and password?
Then use that connection to make all the requests to Astra for that client? This way limitations imposed by the client's Astra role would apply, such as limiting them to only a specific keyspace.
The motivation behind this request is that we'd like to use a single CQL Proxy cluster to access our Astra database, with multiple apps connecting through it that we want to only be able to access their own keyspace.

Thanks for the question @ims-swilkinson!
At the moment cql-proxy only supports a collection of connections for a single username/password; all client requests are then sent over that channel.  cql-proxy could be modified to support a use case like the one you describe but the set of changes required would certainly be non-trivial:

The arguments to specify username and password would have to be made optional
Connections to Astra are currently indexed by keyspace.  These connections would need to be modified by some combination of keyspace, username and credentials

You need to include credentials so that proxy users must provide them; you don't want users to assume identities just by knowing the name of the identity

Connection logic would need to be changed to defer creation of connections to Astra until a client actually connects

This could probably be mitigated by providing the ability to specify username/password combinations that should be pre-allocated

There's almost certainly more than what's outlined above; I'm just musing out loud about what would be necessary.  I'm also assuming all the username/password combinations are on the same Astra database.  Extending this support to cover multiple Astra databases in addition to multiple username/password combinations would add yet another layer of complexity.

Hi Bret, yes we would only ever want to attach a specific CQL Proxy instance or cluster to a specific target Astra database.
I did suspect it would be complex to implement, just thought I'd ask in case it's something you'd already started considering.
The use case it supports isn't the recommended one, it's just one that we wanted to try as it would make our lives easier, but we can deploy CQL Proxy as a sidecar.
Thanks for the answer.
Steve Wilkinson
Principal Software Engineer
e<EMAIL_ADDRESS>w www.ims.techhttp://www.ims.tech/
t  +44 (0) 1270 501 212
[A red and white sign  Description automatically generated with low confidence]https://ims.tech/ @.*** https://ims.tech/about-us/

@.https://twitter.com/IMSTechHQ @. https://www.linkedin.com/company/imstechhq/
Trak (Global Solutions) Limited, trading as IMS, is a company registered in England and Wales with company registration number 06944694 and registered address at Global House, Westmere Drive, Crewe Business Park, Crewe, Cheshire, CW1 6ZD.
This email and any attachments to it may be confidential, may be legally privileged and are intended solely for the use of the individual to whom it is addressed.  Any views or opinions expressed are solely those of the author and do not necessarily represent those of the Trak Global Group.  If you are not the intended recipient of this email, you must not take any action based upon its contents, nor copy or show it to anyone.  Please contact the sender if you believe you have received this email in error.
From: Bret McGuire @.>
Sent: Friday, October 13, 2023 7:42 AM
To: datastax/cql-proxy @.>
Cc: Steve Wilkinson @.>; Mention @.>
Subject: Re: [datastax/cql-proxy] Would it be possible for CQL Proxy to authenticate using the clients credentials? (Issue #120)
Thanks for the question @ims-swilkinsonhttps://github.com/ims-swilkinson!
At the moment cql-proxy only supports a collection of connections for a single username/password; all client requests are then sent over that channel. cql-proxy could be modified to support a use case like the one you describe but the set of changes required would certainly be non-trivial:

The arguments to specify username and password would have to be made optional
Connections to Astra are currently indexed by keyspace. These connections would need to be modified by some combination of keyspace, username and credentials
*   You need to include credentials so that proxy users must provide them; you don't want users to assume identities just by knowing the name of the identity
Connection logic would need to be changed to defer creation of connections to Astra until a client actually connects
*   This could probably be mitigated by providing the ability to specify username/password combinations that should be pre-allocated

There's almost certainly more than what's outlined above; I'm just musing out loud about what would be necessary. I'm also assuming all the username/password combinations are on the same Astra database. Extending this support to cover multiple Astra databases in addition to multiple username/password combinations would add yet another layer of complexity.

Reply to this email directly, view it on GitHubhttps://github.com/datastax/cql-proxy/issues/120#issuecomment-1760985908, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3VDYJIUJUCVNNMERMWN47TX7DPBTANCNFSM6AAAAAA52STY2M.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Sounds good and happy to help @ims-swilkinson !  I'll keep this issue open so that we can have this issue in our backlog.  I don't have any idea when or if any such fix would be implemented (as mentioned there aren't any plans to do anything like this now) but it's an intriguing idea.


Quickly switch between clusters in the side navigation

Several users have previously mentioned that quickly changing between clusters would be really useful in some scenarios. Right now, they're required to deploy the sidenav to do it. We want to keep a minimised version of it always visible.

This is an attempt to fit 3 characters into the cluster blocks and the active blue as a background colour. There might be some issues to discuss about.

After the UX roundtable, we decided to go for the rectangular blocks.
Adobe Xd prototype

@kwwii should we close this issue and create another one for Dev? the design work is finished.

@edenhernandez-suse Yes, we should create a new issue for the development of these features (probably breaking it down into separate issues during grooming).
I added the QA/None label so that when the card is marked as "Done" it will not go through QA cycles.

Regarding the "See all" link (and the "14 more" button), I would suggest linking them temporarily to Cluster Management > Clusters:

In the future, when we take care of Cluster Management (and probably get rid of it), we would need to replicate that clusters table in a new view; but for the time being, I would just use that Clusters view.

https://confluence.suse.com/pages/viewpage.action?pageId=1190330706

We need to split this ticket into various chunks and work in a separate branch since this all has to land at the same time:

Bar
Products
Clusters

@richard-cox & @rak-phillip can help based on their past experiences, particularly with Rancher Desktop.

Primary focus for this ticket: update the high-level menu.
Down the road: cluster management and other sub-components. We won't fix the convoluted logic that ties into generating some of the other menus since that's been a sticky point for a while.

To allow for more time to work on this, we'll push this to Q3.

To be considered existing tests and related issues https://github.com/rancher/dashboard/issues/5966

https://xd.adobe.com/view/1453b6a5-ca9c-4794-8fbf-34f1349e45a3-4100/ has the proposed view.


converge_by is not needed when only platform resources are used

(http://docs.chef.io/lwrp_custom_provider.html). converge_by always
marks the custom resource as updated which may send spurious
notifications when nothing actually changed.

Yeah, it needs to be fixed. Without the converge_by you don't get as nice output from chef-client now that formatters are enabled by default :-/


Segmentation fault in compute_dist_mat.py

Hi,
I keep getting a segmentation fault when I run compute_dist_mat.py:
$ python compute_dist_mat.py
Using TensorFlow backend.
Segmentation fault

All the previous steps in the README seem to complete without error. I've tried this on two different computers with python 2.7 and tensorflow 1.4 and 1.9, respectively. Any ideas on how to get around this? Or if someone could share the output of this script, that would also be appreciated. Thanks!

Hi @robinjia ,
I have just been able to run the mentioned command without any trouble and generate the output file (using TF 1.8.0 and Python 3.6).
I think the source of error may be due to the fact the output matrix is too large and may not fit into the memory of your machine (the output file size is ~20 GB), but this pre-computation would save a lot of time while generating the attack.


1.0.0-beta1 breaks compatibility with oh-my-posh

System Details

posh-git version/path: 1.0.0-beta1 C:\Program Files\WindowsPowerShell\Modules\posh-git\1.0.0
PowerShell version: 5.1.16299.98
git version 2.15.0.windows.1
OS: Microsoft Windows NT 10.0.16299.0

Issue Description
Not sure if this is an issue with posh-git or oh-my-posh but version 1.0.0.-beta1 breaks compatibility with oh-my-posh whereby the prompt disappears and it can even hang. I know that 1.x contains breaking changes but this change was maybe not intended since 0.7.2-pre1 integrates without problems with oh-my-posh

@JanDeDobbeleer Were you ever able to verify that the updated oh-my-posh works correctly on Windows PowerShell?

Please close the issue as everything seems to be working like a charm.

Excellent!

Thanks, I can confirm this works now well with the latest version 2.0.0-beta1 of PSReadline and 1.0.0-beta1 of posh-git (and 2.0.193 of oh-my-posh) on Windows 10 1803 using ConEmu 161206 [64] {Stable}


TypeError in IE 10

When i use this component in IE 10, Error is occurred on console like below.
TypeError :  Unable to get property 'max' of undefined or null reference
at Rheostat ( xxxxxx bundle.js :5 : 667 )
at g._constructComponentWithoutOwnner..
.
.
.
also  bundle.js:5:667 is
Object.getPrototypeOf(Rheostat).call(this,e)),n=t.props,r=n.max,i=n.min,o=n.values;
I check the code 5:667 to bold text

How are you shimming Object.getPrototypeOf? You should be using https://npmjs.com/airbnb-browser-shims or the equivalent.

Thank you. I solved this problem. !!


Bug: ContentState.prototype.getEntityMap() does not return map of entities

What is the current behavior?
ContentState.prototype.getEntityMap() does not return map of entities, currently, it returns just the DraftEntity object, whose methods are already attached to ContentState.prototype.
  getEntityMap(): any {
    // TODO: update this when we fully remove DraftEntity
    return DraftEntity;
  }

so this is what I get from getEntityMap

To reproduce, please use this fiddle and open your console
What is the expected behavior?
It should return a map of the entities

Aha, I saw that this is actually getting fixed here in 0.11

@juliankrispel is there a fix for this in DraftJS .10.2? I need to view my entityMap but I can't access it

@lightninglu10
convertToRaw(editorState.getCurrentContent())

If you're looking for the logic behind how convertToRaw gets the entitymap  - you can see this here  - it basically searches for all entities in the tree and builds a map...
Not super nice :D

Is there a work around to update entities? I am able to access the entityMap through convertToRaw , but the keys in this map are different from the ones created by Draft and as such I am unable to update the right entity

hey @moontails - the way you'd update entities is by using Modifiers. You shouldn't need a workaround or access the entities directly. Here's a link to the docs on modifiers.
Also feel free to join the draft js community on slack https://draftjs.herokuapp.com


Non-deterministic behavior for environment markers in requirements.txt

Package ecosystem: pip
Package manager version: pip 22.1.2
Language version: python 3.7–3.10
Manifest location and content before the Dependabot update: https://github.com/microsoft/torchgeo/blob/main/requirements/required.txt
dependabot.yml content: https://github.com/microsoft/torchgeo/blob/main/.github/dependabot.yml
Updated dependency: numpy from 1.22.4 to 1.23.0
What you expected to see, versus what you actually saw: I have a single requirements.txt file listing all required dependencies. Unfortunately, newer versions of many of these dependencies have dropped support for older versions of Python which I still test. To handle this, I've added environment markers such as:
numpy==1.22.4;python_version>='3.8'
numpy==1.21.6;python_version=='3.7'

When the numpy 1.23.0 release came out, I would have expected dependabot to update the first line, since 1.23.0 does not support Python 3.7. However, dependabot instead updated the second line. Interestingly, dependabot made the right decision for a different dependency in https://github.com/microsoft/torchgeo/pull/621/files. I'm unsure how dependabot is deciding which line to update, but I would like more control (either always update the first line, or actually use pip to check to see if a new version is compatible with that environment marker).
Native package manager behavior: N/A
Images of the diff or a link to the PR, issue, or logs: https://github.com/microsoft/torchgeo/pull/622
🕹 Bonus points: Smallest manifest that reproduces the issue

One possible workaround would be to keep a separate requirements.txt for older versions of Python that dependabot would not update. Unfortunately this isn't possible due to #5299.

👋 sorry for the slow reply.
This appears to be a duplicate of:

https://github.com/dependabot/dependabot-core/issues/3343


[REGRESSION]: SSHv0 no longer working on self-hosted Windows 10 x64 agents

New issue checklist

[X] I searched for existing GitHub issues
[X] I read pipeline troubleshooting guide
[X] I checked how to collect logs

Task name
SSHv0
Breaking task version
0.236.2
Last working task version
0.231.3
Regression Description
SSHv0 previously worked on self-hosted Windows 10 Agents, now does not as of most recent update - log explicitly states that whatever it's attempting to do is not available on Windows(x64). We have a workaround as we have some linux agents as well, but far fewer in number due to the nature of our company's products so i'd like to get these people off of the linux ones if possible.
Pretty sure it's not our agents as the most recent success was yesterday, on and agent that; on Attempt 1 downloaded the newer version of the task, failed; then on Attempt 2 downloaded the older version of the task for some reason (or maybe it already had it? didn't have system.debug on for that one - not sure how this all happened lol) and succeeded (verified same agent / same machine both times).
I did see that the ssh2 library was updated as part of the most recent update a couple weeks ago, though that should have been a minor change from what i can see - though the sftp lib was updated significantly but i would think that wouldn't matter here.
Also seeing that the node10 executor was removed in #19532, was this done early in error? I'm unaware of the current state of node10 deprecation, documentation seems to still place node10 as the default executor without mention of deprecation of it yet in favor of node16 or node20. Would it be simple enough to update the agent (would that install the 16 executor it might be missing)? Or would something like a NodeToolInstaller task @ node 16 do the job?
Other things that might not matter but included for completion:

Agent version 2.204.0
Task was running an inline script
Targeting bash on remote linux machine (not sure of OS but again probably doesn't matter)
All other fields seemed set at defaults

Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Windows 10 (for self-hosted agent)
Relevant log output
company info [omitted]

2024-03-13T12:12:37.2931033Z ##[section]Starting: [OMITTED]
2024-03-13T12:12:37.3932610Z ==============================================================================
2024-03-13T12:12:37.3933777Z Task         : SSH
2024-03-13T12:12:37.3934542Z Description  : Run shell commands or a script on a remote machine using SSH
2024-03-13T12:12:37.3934988Z Version      : 0.236.2
2024-03-13T12:12:37.3935996Z Author       : Microsoft Corporation
2024-03-13T12:12:37.3936822Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/deploy/ssh
2024-03-13T12:12:37.3937459Z ==============================================================================
2024-03-13T12:12:37.5927557Z ##[error]A supported task execution handler was not found. The task does not carry an implementation that is compatible with your current operating system 'Windows(X64)'. Contact the task author for more details.
2024-03-13T12:12:37.5948155Z ##[section]Finishing: [OMITTED]

Full task logs with system.debug enabled
Do not have a successful run with system.debug on as newer runs (now, i guess) are downloading the newer task version, and the previous success had it off; but i've included that success regardless to illustrate the task version. The success is also the one that appeared out of order.
company info [omitted]

UNSUCCESSFUL RUN

2024-03-14T13:28:55.4742748Z ##[debug]Evaluating condition for step: [OMITTED]
2024-03-14T13:28:55.4812566Z ##[debug]Evaluating: succeeded()
2024-03-14T13:28:55.4820437Z ##[debug]Evaluating succeeded:
2024-03-14T13:28:55.4879262Z ##[debug]=> True
2024-03-14T13:28:55.4893476Z ##[debug]Result: True
2024-03-14T13:28:55.4955422Z ##[section]Starting: [OMITTED]
2024-03-14T13:28:56.6013212Z ==============================================================================
2024-03-14T13:28:56.6014659Z Task         : SSH
2024-03-14T13:28:56.6015878Z Description  : Run shell commands or a script on a remote machine using SSH
2024-03-14T13:28:56.6019328Z Version      : 0.236.2
2024-03-14T13:28:56.6020545Z Author       : Microsoft Corporation
2024-03-14T13:28:56.6021876Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/deploy/ssh
2024-03-14T13:28:56.6023048Z ==============================================================================
2024-03-14T13:28:56.6544760Z ##[error]A supported task execution handler was not found. The task does not carry an implementation that is compatible with your current operating system 'Windows(X64)'. Contact the task author for more details.
2024-03-14T13:28:56.6568923Z ##[debug]System.InvalidOperationException: A supported task execution handler was not found. The task does not carry an implementation that is compatible with your current operating system 'Windows(X64)'. Contact the task author for more details.
   at Microsoft.VisualStudio.Services.Agent.Worker.TaskRunner.RunAsync()
   at Microsoft.VisualStudio.Services.Agent.Worker.StepsRunner.RunStepAsync(IStep step, CancellationToken jobCancellationToken)
2024-03-14T13:28:56.6577176Z ##[section]Finishing: [OMITTED]

SUCCESSFUL RUN

2024-03-13T12:13:57.5508201Z ##[section]Starting: [OMITTED]
2024-03-13T12:13:57.6514823Z ==============================================================================
2024-03-13T12:13:57.6515853Z Task         : SSH
2024-03-13T12:13:57.6516624Z Description  : Run shell commands or a script on a remote machine using SSH
2024-03-13T12:13:57.6517085Z Version      : 0.231.3
2024-03-13T12:13:57.6517719Z Author       : Microsoft Corporation
2024-03-13T12:13:57.6518550Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/deploy/ssh
2024-03-13T12:13:57.6519277Z ==============================================================================
2024-03-13T12:13:59.6765585Z Trying to establish an SSH connection to [OMITTED]
2024-03-13T12:13:59.8829161Z Successfully connected.
[OMITTED]
2024-03-13T12:14:44.2068275Z Removing zip and package directory
2024-03-13T12:14:44.2069238Z
2024-03-13T12:14:44.2069622Z Finished
2024-03-13T12:14:44.2069844Z
2024-03-13T12:14:45.6250348Z ##[section]Finishing: [OMITTED]

Repro steps
1. have Windows 10 x64 agent (presumably self-hosted, at version 2.204.x)
2. attempt to use SSHv0 task at version 0.236.x
3. task fails due to not having compat execution tool

Hi @Metarract, thanks for the feedback.
The node10 runner was removed because dependent packages do not support node10.
We have a task when the runner is missing in the agent, but, unfortunately, it will not work with an old agent, because it works only for previous node versions (e.g. agent has node 16 handler and had node 10, the task could install node 10 for such agent).
One of the options is that you can pin the task version in the yml file (e.g<EMAIL_ADDRESS>to use older version but it may contain some bugs, or update your agent(you need agent's version at least 2.206.1 where node16 runner was introduced).
We apologise for the inconvenience

fair enough, cheers; i assume it would make sense to bump the minimum agent version to 2.206.1 in the task json then? i see a pr out to fix it in the docs at the very least

You're right, we'll bump minimumAgentVersion to 2.206.1, thank you!

@Metarract, thank you reporting this!
I'd like to add that we highly recommend you upgrade you agent to more modern version, [v2.x is not supported], with each agent iteration we deliver important security fixes & improvements (https://learn.microsoft.com/en-us/azure/devops/pipelines/agents/v3-agent?view=azure-devops#can-i-stay-on-2x-agents-if-im-not-working-on-any-changes-in-my-project-anymore)

@kirill-ivlev yeah previously we were just letting them update as-needed and didn't realize that they were so far behind as a result; went around updating them all for this reason but also since the team in question had several deployments waiting around.

Also i saw this got merged in last week and a new release was made just a couple days ago so i'm gonna just close this if that's fine; thank you very much everyone!


WIP: D3D12 Implementation

Sketching out a broad task list here, will convert this to specific function implementations as we narrow the work down.

[x] Command buffer pooling and acquisition
[x] Buffer creation
[x] Texture creation
[ ] Data transfer
[ ] Graphics resource binding
[ ] Compute resource binding
[ ] Command buffer submission and resource tracking cleanup
[x] Capability queries

Checklist for FNA3D that does not involve downloads, from most important to least:

[ ] Blit
[ ] SubmitAndAcquireFence
[ ] SetSwapchainParameters
[ ] GenerateMipmaps
[ ] BindVertexSamplers

As of latest push we can run the Wizorb trace with D3D12.
There's still a few issues to clear up - we're gonna need per-format blit pipelines and a command signature cache based on argument buffer stride. But it's very cool that we have a game's actual render loop working!


Adding Squared Hinge Loss

Build and test passes locally.
Reference : #127

@rxwei Is there a clip function? Or should I add it from here? The function I was thinking of adding was clipbyvalue would I have to define a vjp for it, given I will use it in a loss fn?

Not yet. Feel free to add one. I'd suggest naming it clipped(min:max:).

Yeah I'll add that in a while. Also, @rxwei this PR is good to go, The kokoro run was a success, the latest commit is just merging into master.


[IDEA] - !info or !stats command

shows stats of the bot

Servers online
Members registered
Something like this

Added in c7b6641f3d8f39048493f7035f1cfec7713308d0


Current LiteRT Android dependencies in documentation look broken

I think after the recent TensorflowLite rename to LiteRT some pages in documentation where renamed incorrectly and are currently very confusing.
For a particular example, see this: https://ai.google.dev/edge/litert/android/gpu

The docs say to add com.google.ai.edge.litert:litert-gpu and com.google.ai.edge.litert:litert-gpu-api with versions 2.X.Y, which do not exist (current latest version is 1.0.1), to a toml version catalog.
In the next paragraph it also switches into using gradle files instead of toml to declare other dependencies, which I found somewhat confusing.
Later, in standalone setup, it says to include com.google.ai.edge.litert:litert-gpu-delegate-plugin dependency, which does not exist and also follows with a code snipped supposedly showing how to include it, but it shows other dependencies.
Other places too include dependencies with incorrect versions, like com.google.ai.edge.litert:litert-gpu:2.3.0

I just happend to start working with LiteRT for Android right now and found it very difficult to distinguish which parts of documentation are outdated and which aren't.

Hi, @vykintazo
I apologize for the delayed response, thank you for bringing this issue to our attention I'll have look into this issue and will update you.
Thank you for your cooperation and patience.


Added accessor type to RemoteObject

The RemoteObject doesn't have a type accessor which I found a request to have resulting in the request being unable to deserialize.

Fixes issue


feat: use built-in dotnet format linter for csharp language

Fixes #4522

Proposed Changes

Add a test case on which dotnet format should pass
Update linter command for CSHARP language

Readiness Checklist
Author/Contributor

[ ] If documentation is needed for this change, has that been included in this pull request

Reviewing Maintainer

[ ] Label as breaking if this is a large fundamental change
[ ] Label as either automation, bug, documentation, enhancement, infrastructure, or performance

Hi! Thanks for this PR. You may need to tweak a few other places as well :)
https://github.com/search?q=repo%3Asuper-linter%2Fsuper-linter dotnet-format&type=code

Hi, @ferrarimarco
Thanks for pointing me to other places. I've updated my PR with touching them as well, unfortunately, cannot test them properly locally as not so familiar with super-linter infra

Looks like the install script is failing. Can you take a look @vhatsura?  https://github.com/super-linter/super-linter/actions/runs/6525095511/job/17717425694?pr=4627#step:6:4030


[SPARK-12094][SQL] Prettier tree string for TreeNode

When examining plans of complex queries with multiple joins, a pain point of mine is that, it's hard to immediately see the sibling node of a specific query plan node. This PR adds tree lines for the tree string of a TreeNode, so that the result can be visually more intuitive.

For example, the following snippet
val path = "file:///tmp/parquet/unsafe"
(sqlContext range 10 coalesce 1).write mode "overwrite" parquet path
sqlContext.read parquet path registerTempTable "u"
sqlContext.sql("select count(*) from (select * from u where id > 3) a join u b where a.id = b.id") explain true

gives the following query plan explanation, which can be (arguably) more intuitive:
== Parsed Logical Plan ==
'Project [unresolvedalias((count(1),mode=Complete,isDistinct=false))]
+- 'Filter ('a.id = 'b.id)
   +- 'Join Inner, None
      :- 'Subquery a
      :  +- 'Project [unresolvedalias(*)]
      :     +- 'Filter ('id > 3)
      :        +- 'UnresolvedRelation `u`, None
      +- 'UnresolvedRelation `u`, Some(b)

== Analyzed Logical Plan ==
_c0: bigint
Aggregate [(count(1),mode=Complete,isDistinct=false) AS _c0#13L]
+- Filter (id#0L = id#12L)
   +- Join Inner, None
      :- Subquery a
      :  +- Project [id#0L]
      :     +- Filter (id#0L > cast(3 as bigint))
      :        +- Subquery u
      :           +- Relation[id#0L] ParquetRelation[file:/tmp/parquet/unsafe]
      +- Subquery b
         +- Subquery u
            +- Relation[id#12L] ParquetRelation[file:/tmp/parquet/unsafe]

== Optimized Logical Plan ==
Aggregate [(count(1),mode=Complete,isDistinct=false) AS _c0#13L]
+- Project
   +- Join Inner, Some((id#0L = id#12L))
      :- Project [id#0L]
      :  +- Filter (id#0L > 3)
      :     +- Relation[id#0L] ParquetRelation[file:/tmp/parquet/unsafe]
      +- Relation[id#12L] ParquetRelation[file:/tmp/parquet/unsafe]

== Physical Plan ==
TungstenAggregate(key=[], functions=[(count(1),mode=Final,isDistinct=false)], output=[_c0#13L])
+- TungstenExchange SinglePartition, None
   +- TungstenAggregate(key=[], functions=[(count(1),mode=Partial,isDistinct=false)], output=[count#16L])
      +- Project
         +- BroadcastHashJoin [id#0L], [id#12L], BuildRight
            :- Filter (id#0L > 3)
            :  +- Scan ParquetRelation[file:/tmp/parquet/unsafe] PushedFilter: [GreaterThan(id,3)] [id#0L]
            +- Scan ParquetRelation[file:/tmp/parquet/unsafe] PushedFilter: [] [id#12L]

Test build #47053 has started for PR 10099 at commit 90b46d9.

This is pretty cool, thanks! LGTM

@cloud-fan Thanks! comments addressed.

Test build #47062 has started for PR 10099 at commit 10eb09e.

Test build #47053 has finished for PR 10099 at commit 90b46d9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47053/
Test PASSed.

Test build #47063 has started for PR 10099 at commit 31a1c22.

Test build #47062 has finished for PR 10099 at commit 10eb09e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47062/
Test PASSed.

Merged build finished. Test PASSed.

Test build #47063 has finished for PR 10099 at commit 31a1c22.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47063/
Test PASSed.

Merged build finished. Test PASSed.

Merging to master and branch 1.6.


x/build,cmd/link: darwin/amd64 builders unexpectedly skipping cmd/link.TestBuildForTvOS

In #58221 we observed consistent test failures in cmd/link.TestBuildForTvOS on darwin-amd64-11_0, but not on the other darwin/amd64 builders. The root cause of the failure is not OS-dependent, which suggests that the test is being skipped on all of the other darwin/amd64 builders. That is unexpected.
We should probably:

adjust those builders so that they can run the test (attn @golang/release), and
adjust the test so that it fails instead of skipping if GO_BUILDER_NAME is non-empty (attn @golang/compiler, @golang/ios).

(The discussion on #58221 covers more detail about the skips.)

Yeah, the test requires tvOS SDK installed on the machine. Not sure they are...

cc @prattmic for the next time we build macOS images?

@golang/ios How much do we care about this test?

cc @eliasnaur

The different builders have slightly different versions of XCode installed (generally the latest that happens to be supported by that OS version). It wouldn't surprise me if one of those XCode installations isn't quite right. Exact installation steps seemed to vary slightly from version to version, so it wouldn't surprise me if something went wrong along the way.

Xcode completes its installation on its first launch, showing a prompt that looks like this (in a recent version):

So it'd be a matter of not forgetting to check the box next to tvOS and pressing continue next time.

I may have done the installation incorrectly, but FWIW I never saw that window on any of the Xcode installations I did for Darwin builders. IIRC, the installation process is completely GUI-free other than accepting the license.


[Bug] Unpublished page

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

On an unpublished app's view mode, we have a scroll
On an unpublished apps' view mode, the color of the icon doesn't adhere to the Appsmith design system

Steps To Reproduce
Launch an app that is not published
Environment
Production
Version
Cloud

@marks0351 Could you mention the approach to resolve the scrolling issue?

I looked into this, I think if we don't render the AppPage component when there are no widgets, can solve this problem

.

And the reason this problem is arising is in AppPage component we are setting the height of the container, equal to a factor of bottomRow.


test_models: check openpilot can cancel cruise

Checks we can cancel cruise in a timely manner

[ ] Make sure check is reliable
[ ] Check for accidental re-enables
[ ] Move to new test

Test models only checks things that we can fully reproduce just from the CAN messages. This belongs in FindInteresting.


[WIP] clicrud pack

clicrud
Status

new pack, work in progress

Description
WIP - creating pack for clicrud (a library for working with Brocade ICX, VDX and MLX devices).
read action works, but needs more testing
configure action not working quite right yet, need to handle command list better
TODO:

Add aliases.
Figure out how to add tests

Checklist (tick everything that applies)

[X] Metadata: pack.yaml, icon, structure (required for new packs)
[ ] Version bump (required for changed packs)
[ ] Code linting (required, can be done after the PR checks)
[ ] Tests (not required but really recommended)

Closing this in favour of upcoming PR from @DavidJohnGee for his clicrud pack https://github.com/DavidJohnGee/clicrudST2


CA Certificate

I use this docker container with custom ssl certificates (SSL_TYPE=custom). To use my own certificates I create a pem file from crt and key like this:
cat mail.domain.tld.crt mail.domain.tld.key > config/ssl/mail.domain.tld-full.pem

Is there a way to add the CA Certificate?
The option for postfix in /etc/postfix/main.cf is smtpd_tls_CAfile = path_to_cert.

Hi,
I never tried this myself, this feature begin added by a contributor but:

have a look to https://github.com/tomav/docker-mailserver/blob/master/target/start-mailserver.sh#L135-L153. It should do the trick except is something is missing (should be a good test)
if it doesn't work, best thing is to try to fix and document this part
last option is to configure it manually using what I explained here

Let me know.

Perhaps @emazzotta can help. (cf. this PR)

Hi @skaren
The idea with the custom "full" certificate, is that it contains everything necessary, so you don't need to reference anything else.
I use startssl and my script to generate the "mail.domain.tld-full.pem" looks something like this:
cat << EOF > private.key
-----BEGIN RSA PRIVATE KEY-----
my_key...
-----END RSA PRIVATE KEY-----
EOF
cat << EOF > my_cert.crt
-----BEGIN CERTIFICATE-----
my_cert....
-----END CERTIFICATE-----
EOF

echo "$(cat my_cert.crt)\n$(curl --silent https://www.startssl.com/certs/class2/sha2/pem/sub.class2.server.sha2.ca.pem)" > unified.crt
openssl x509 -in unified.crt -out unified.pem -outform PEM
openssl rsa -in private.key -out private_unlocked.key
echo "$(cat private_unlocked.key)\n$(cat unified.pem)\n$(curl --silent https://www.startssl.com/certs/ca-bundle.pem)" > mail.domain.tld-full.pem

So the first curl request gets the intermediate certificate and the second request gets the whole ca-bundle. Depending on the CA you get your certificates from, this may vary.
If this helps, you're free to put this in the readme where appropriate @skaren / @tomav

I use StartSSL too. I get the following error, if I create the certificate with your script and test it on https://www.checktls.com/TestReceiver.
[001.175] Certificate 1 of 3 in chain:
          subject= /CN=mail.domain.tld
          issuer= /C=IL/O=StartCom Ltd./OU=StartCom Certification Authority/CN=StartCom Class 1 DV Server CA
[001.201] Certificate 2 of 3 in chain:
          subject= /CN=mail.domain.tld
          issuer= /C=IL/O=StartCom Ltd./OU=StartCom Certification Authority/CN=StartCom Class 1 DV Server CA
[001.243] Certificate 3 of 3 in chain:
          subject= /CN=mail.domain.tld
          issuer= /C=IL/O=StartCom Ltd./OU=StartCom Certification Authority/CN=StartCom Class 1 DV Server CA
[001.244] Cert NOT VALIDATED: unable to get local issuer certificate
[001.244] this may help: What Is An Intermediate Certificate
[001.244] So email is encrypted but the domain is not verified
[001.244] Cert Hostname DOES NOT VERIFY (123.456.789.12 != mail.domain.tld)
[001.245] So email is encrypted but the host is not verified

I get another problem with your script. It write the character \n instead of a line break. For the tests I removed this manualy.
-----END CERTIFICATE-----\n-----BEGIN CERTIFICATE-----

Kind regards
Skaren

That's interesting, I tested it and it works fine for me. Are you sure you're using the correct intermediate certificate?
With the line breaks, try replacing the two echo-statements with echo -e

echo -e works well.
I used the class 1 intermediate certificate from https://www.startssl.com/certs/class1/sha2/pem/sub.class1.server.sha2.ca.pem

Clearly something is wrong with your certificate, are you sure you ran this correctly?
Try this again, I already changed it to be class 1:
cat << EOF > private.key
-----BEGIN RSA PRIVATE KEY-----
my_key...
-----END RSA PRIVATE KEY-----
EOF
cat << EOF > my_cert.crt
-----BEGIN CERTIFICATE-----
my_cert....
-----END CERTIFICATE-----
EOF

echo "$(cat my_cert.crt)\n$(curl --silent https://www.startssl.com/certs/class1/sha2/pem/sub.class1.server.sha2.ca.pem)" > unified.crt
openssl x509 -in unified.crt -out unified.pem -outform PEM
openssl rsa -in private.key -out private_unlocked.key
echo "$(cat private_unlocked.key)\n$(cat unified.pem)\n$(curl --silent https://www.startssl.com/certs/ca-bundle.pem)" > mail.domain.tld-full.pem

I don't know where the problem lies.
Now I use let's encrypt. It's realy simple.
Thanks for the help!


Enhanced github functionality to check for personal repos if none are…

… found in the defined organization
For example:
If invoked with --organization <github personal userid>, it will first check for repos belonging to an org of that name, and if none are found, it will check for user Public Repos under the name instead.
This is so that we don't have to require repos we're interested in to belong only to a github organization.
Fixes #7

Missed this, sorry.
Could that just be a separate flag instead?

Closing this PR in order to not have multiple things in one pull request. Will resubmit


Stackblitz examples not working

Bug, feature request, or proposal:
Bug
What is the expected behavior?
This works https://material.angular.io/components/menu/overview.
What is the current behavior?
Stackblitz variant is not. https://stackblitz.com/angular/emekjleyejm?file=app%2Fmenu-overview-example.ts
What are the steps to reproduce?
Open the stackblitz example.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Browser: Chromium Version 64.0.3282.167 (Developer Build) (64-bit)
OS: Archlinux

@barni2000 I assume you're mentioning Google Chrome. It's working okay for me on macOS latest version of High Sierra. Do you see any errors in the console?

Yes:

If I export also not working.

Firs i got this after npm install:
><EMAIL_ADDRESS>start /home/work/emekjleyejm.angular
> ng serve

module.js:557
    throw err;
    ^

Error: Cannot find module '@angular-devkit/core'
    at Function.Module._resolveFilename (module.js:555:15)
    at Function.Module._load (module.js:482:25)
    at Module.require (module.js:604:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/home/work/emekjleyejm.angular/node_modules/@angular-devkit/schematics/src/tree/virtual.js:10:16)
    at Module._compile (module.js:660:30)
    at Object.Module._extensions..js (module.js:671:10)
    at Module.load (module.js:573:32)
    at tryModuleLoad (module.js:513:12)
    at Function.Module._load (module.js:505:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `ng serve`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/work/.npm/_logs/2018-02-21T11_56_22_632Z-debug.log

Afer I install @angula-devkit/core i got this with npm start:

><EMAIL_ADDRESS>start /home/work/emekjleyejm.angular
> ng serve

** NG Live Development Server is listening on localhost:4200, open your browser on http://localhost:4200/ **
Date: 2018-02-21T11:58:05.453Z
Hash: 7d8137d0a69c57926f5f
Time: 2024ms
chunk {inline} inline.bundle.js (inline) 5.79 kB [entry] [rendered]
chunk {main} main.bundle.js (main) 2.93 kB [initial] [rendered]
chunk {polyfills} polyfills.bundle.js (polyfills) 636 bytes [initial] [rendered]
chunk {styles} styles.bundle.js (styles) 150 kB [initial] [rendered]
chunk {vendor} vendor.bundle.js (vendor) 853 kB [initial] [rendered]

ERROR in Error: Tried to find bootstrap code, but could not. Specify either statically analyzable bootstrap code or pass in an entryModule to the plugins options.
    at Object.resolveEntryModuleFromMain (/home/work/emekjleyejm.angular/node_modules/@ngtools/webpack/src/entry_resolver.js:131:11)
    at Promise.resolve.then.then (/home/work/emekjleyejm.angular/node_modules/@ngtools/webpack/src/angular_compiler_plugin.js:239:54)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:160:7)

webpack: Failed to compile.

If I split AppModule to app/app.module.ts from main.ts its works.

Can you provide your own reproduction case as the example you linked to seems to be working as expected.

I create some changes it works after export in local but in stackblitz also not  loading.

npm install
npm start

https://angular-hlrhkp.stackblitz.io

I've done some testing on other web browsers. The original examples works with Firefox, i think this issue Chromium related.

I have installed Google-Chrome instead of Chromium and its working, but the examples still not exportable.


Unhelpful commitlint comment

commitlint comments are unhelpful. At a minimum they should include links to documentation of what is a valid commit type and message. Currently it just says it's invalid with info on how to fix it. This is a poor experience for new contributors.
Example comment: https://github.com/open-wc/open-wc/pull/1450#issuecomment-601750809

We've been having some discussions in the team about this, we agree the current setup isn't great especially for contributing docs changes. We are evaluating whether we should remove the commitbot for now.
@daKmoR @thepassle @Westbrook @bennypowers

I recommend retaining the bot, upstreaming UX improvements

So the action here is to improve the commitlint bot message to explain the committer what they should do.

@abraham would you be willing to file this on https://github.com/conventional-changelog/commitlint?

I currently don't have the time to own upstreaming this.


Limitations of the method

Just wondering if any limitations of the Infini-attention like inference speed and model performance. Not too much discussions in the paper.

Since infini attention uses segmentation which is mainly focused on reducing memory usage and computational cost into O(N) so if you use very long seq such as seq len = 1M, then you have to wait 5min per batch on 1x H100 gpu.
In this implementation I used for loop to make it work so you'll get exactly N times more time to inference or to train.
e.g.,
segment 2048 & block 2048 ==> 1 segments, just 1 attention computation
segment 2048 & block 32K ==> 16 segments, slower 16 times than block 2048
Model performance check was done at the paper with PPL but we may need more tests on very long sequence based tests.

Since infini attention uses segmentation which is mainly focused on reducing memory usage and computational cost into O(N) so if you use very long seq such as seq len = 1M, then you have to wait 5min per batch on 1x H100 gpu.
In this implementation I used for loop to make it work so you'll get exactly N times more time to inference or to train.
e.g., segment 2048 & block 2048 ==> 1 segments, just 1 attention computation segment 2048 & block 32K ==> 16 segments, slower 16 times than block 2048
Model performance check was done at the paper with PPL but we may need more tests on very long sequence based tests.

I agree with you that more tests on very long sequence input are needed. It's not unique to think about using the segmentation to break down the long sequence input by for loop, e.g. RingAttention. Just curious about the impact of compressive memory, how does it influence the computation of attention when dealing with the long input?

Since infini attention uses segmentation which is mainly focused on reducing memory usage and computational cost into O(N) so if you use very long seq such as seq len = 1M, then you have to wait 5min per batch on 1x H100 gpu.
In this implementation I used for loop to make it work so you'll get exactly N times more time to inference or to train.
e.g., segment 2048 & block 2048 ==> 1 segments, just 1 attention computation segment 2048 & block 32K ==> 16 segments, slower 16 times than block 2048
Model performance check was done at the paper with PPL but we may need more tests on very long sequence based tests.

I also noticed during various tests that it slows down with time even before generating more tokens than segment dimension. For example, in my small 4b quantized model, it starts with 35tk/s, after about 1k token it's at 4tk/s.


[PL-58076]: SMP 0.22.0 RN

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: SMP 0.22.0 RN
Jira/GitHub Issue numbers (if any): PL-58076
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6718b9c8e99ca7760e6beff0--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://671bbeb58fce5c6919acdc36--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://671e7a0974664d92b0b72273--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6720cacc82b109f4a415c3c8--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6720ebc947a35f1fbe0df439--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6720f76218615d1f879b413e--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6721073f95ec6f12ff1a60d6--harness-developer.netlify.app


vSphere CPI/CSI options omitted when selecting vsphere cloud provider in Rancher 2.6.7 or newer

Setup

Rancher version: >= 2.6.7
Browser type & version: Firefox 104.0

Describe the bug
vSphere CSI options are omitted when selecting the respective cloud provider in rke2 provisioning. vSphere CPI options however remain
To Reproduce
Create cluster -> RKE2 -> Select vSphere as the cloud provider > Navigate to addon config
Result
Expected Result
vSphere CPI options presented, vSphere CSI options are omitted.
Screenshots
In 2.6.7/2.6.8:

in 2.6.6

Additional context
In 2.6.6 the latest (non experimental) version of RKE2 is 1.22.13
In 2.6.8 the latest version of RKE2 is 1.24.4
The CSI chart has a version restriction:
https://github.com/rancher/charts/blob/36de515042fb94888c751aeedd645f6ffae6a854/index.yaml#L8784
  rancher-vsphere-csi:
  - annotations:
      catalog.cattle.io/certified: rancher
      catalog.cattle.io/display-name: vSphere CSI
      catalog.cattle.io/kube-version: '>= 1.20.0-0 < 1.24.0-0'

Even in 2.6.8, if a valid version is specified for the RKE2 version, the CSI options are not present.
The CPI chart does have a version restriction, but is higher:
https://github.com/rancher/charts/blob/36de515042fb94888c751aeedd645f6ffae6a854/index.yaml#L8576
  rancher-vsphere-cpi:
  - annotations:
      catalog.cattle.io/certified: rancher
      catalog.cattle.io/display-name: vSphere CPI
      catalog.cattle.io/kube-version: '>= 1.18.0-0 < 1.25.0-0'

What I suspect is happening:

in 2.6.8 when an RKE2 version is created, it defaults to the latest version (1.24.4) which is outside the range of the CSI chart but not the CPI chart. However, if you were to select a different version that does fall within this range, this is not re-evaluated.

in 2.6.6 when an RKE2 cluster is created, it defaults to the latest version (1.22.13) which is inside the range of both charts, therefore both chart values are seen

Therefore, I believe an action needs to be performed in the UI whenever a specific k8s version is selected to re-evaluate respective addon charts for the vSphere cloud provider, assuming the version restriction is for a legitimate reason.

@MKlimuszka can you work with your team and Prachi to see if we can figure out what's happening with the chart? It might be something @samkulkarni20 can look at with his team.

I'm adding my two cents if I may. I've noticed that somehow the display of the vsphere csi config options seem to be related to the K8S version that the Rancher server itself is running on not the version that I would like Rancher to deploy for me.
For example:

Running Rancher 2.6.9 on K3S v1.24.6+k3s1 and trying to deploy RKE2 cluster v1.24.4+rke2r1 -- no vsphere csi config options shown
Running Rancher 2.6.9 on K3S v1.23.13+k3s1 and deploying RKE2 cluster v1.24.4+rke2r1 -- all good, vsphere csi options present.

@mihkel , thank you for reporting this observation.
This actually makes sense as CSI chart is only compatible with k8s 1.20 through 1.23, not 1.24.
However, in this case it seems to be wrong it's evaluated against Rancher cluster version and not downstream cluster version. Plus the UX could definitely be better to handle case where it's not supported by selected downstream cluster version.
The other good question is why it's not filtering out on v2.6.6 or on v2.6-head versions.
This is not a resolution, just some thoughts for @rayandas @mitulshah-suse who are working on this issue.
cc: @MKlimuszka @prachidamle

Thanks for all the pointers.
There's a tracking issue https://github.com/rancher/rancher/issues/38188 to update CSI chart to include all the bug fixes that happened for CSI, this includes vSphere CSI 2.6.1 (latest version) which supports K8s 1.24 So I believe this (updated CSI chart to the latest) will fix the issue vSphere CSI configs getting omitted.
The milestone of the mentioned tracking issue  is 2023-Q1-v2.7x. but this issue milestone is 2.7.0 and is release blocker right now.
cc: @prachidamle @snasovich @mitulshah-suse

Investigated this one a little more and TLDR is that the immediate issue should indeed be fixed by https://github.com/rancher/rancher/issues/38188 and we can wait for it to be done for the next minor feature release after 2.7.0 (which is 2023-Q1-v2.7x milestone) as this issue is not actually a regression.
Per https://github.com/rancher/rancher/issues/39462#issuecomment-1297597317 it's already established that the UI doesn't render CSI configuration form if request to v1/catalog.cattle.io.clusterrepos/rancher-charts?link=index doesn't return rancher-vsphere-csi entry.
This is an issue in itself as whether there is an appropriate version of CPI/CSI chart to be installed on the downstream cluster depends on the selected RKE2 version. I.e. it depends on if the versions of CPI/CSI charts bundled in the release supports that release.
@brandond @rancher-max , is it possible for RKE2 release to bundle versions of CPI/CSI charts that don't support it? For example, I believe this PR adds CSI 2.6.0 that is required for k8s 1.24.x, but I'm sure we've released 1.24.x versions before then. Does it mean whose versions had CSI chart bundled within them that wasn't working or it wasn't bundled at all?
I understand that once CSI base chart is updated and pulled into RKE2 it's no longer an issue for the subsequent patch releases, but for those first few releases on new minor k8s version we may not have charts ready yet.
Circling back on UI, @codyrancher , is there something that UI uses from that payload (e.g. to generate form automatically) or it's simply exists/not exists check?
Now, onto the explanation of why the issue showed up in v2.6.7 and is not reproducible on v2.6-head.
v2.6.7 is the first version with 1.24.x support and since that release it's the default for embedded k3s version when running Rancher as Docker container (https://github.com/rancher/rancher/blob/v2.6.7/package/Dockerfile#L45) - and that's how most testing was done. So since 2.6.7+ was generally tested on 1.24 k8s then, the issue showed up. As per https://github.com/rancher/rancher/issues/39462#issuecomment-1302132086 even 2.6.9 running on 1.23 doesn't exhibit this issue. So it's not really a regression - just the fact that CSI chart currently bundled in Rancher doesn't support k8s 1.24+.
The reason why the issue is not reproducible on v2.6-head (or any non-release version for that matter) is because the filtering logic used by v1/catalog.cattle.io.clusterrepos/rancher-charts?link=index endpoint (https://github.com/rancher/rancher/blob/08bf3fedd660c884fab1780a172593d5ad59f7cf/pkg/catalogv2/content/content.go#L200) simply short-circuits for non-release versions and returns unfiltered list. Now, I can see a rationale in not filtering based on Rancher version chart annotations, but I don't see a good reason to skip filtering based on k8s version and suggest fixing it.
@prachidamle @MKlimuszka , I believe it's something that will be for your team.
Pending the responses to some pending questions, we will repurpose this issue to fix charts filtering for non-release versions and probably open another one to properly use downstream cluster information to determine whether CSI/CPI charts are available (or assume they are as a stop-gap measure, if possible).

note that on the latest (2.7.0) the behavior seems to have flipped: now the UI shows only CSI options, but no CPI options (see linked ticket above for details)

note that on the latest (2.7.0) the behavior seems to have flipped: now the UI shows only CSI options, but no CPI options (see linked ticket above for details)

I've seen it also on 2.6.9 when I upgraded one rke2 cluster from 1.24.4 to 1.24.7. After the upgrade the CPI options were missing. Downgrade to 1.24.4 made them show up again.

note that on the latest (2.7.0) the behavior seems to have flipped: now the UI shows only CSI options, but no CPI options (see linked ticket above for details)

I can confirm this. I was trying out Rancher for the first time and this caused a lot of confusion. Running vSphere 8.0 with Rancher v2.7.0.

I face similar issues when upgrade rancher and then upgrade my cluster via GUI.
When I open the configuration page and select a supported version of RKE, this wipes my CSI/CPI settings.
This seems to be because when the current version is selected, there are no settings shown in the addon page.
Workaround for me is now to edit the yaml

I re-tested this behaviour on rancher v2.6.9 and v2.7.0 for RKE2 type cluster.
On rancher v2.6.9 both the CPI and CSI options are not visible.

On rancher v2.7.0 CPI option is not visible but the CSI option is visible as mentioned here by @slickwarren

On rancher v2.6.9 for k8s cluster v1.23.14+rke2r1 only CSI configuration is visible.

On rancher v2.6.9 for k8s cluster v1.22.16+rke2r1 only CPI configuration is visible along with Calico configuration.

/backport 2023-Q1-v2.6x

Tested the CSI/CPI cloud configuration visibility on rancher v2.7.0 while pointing KDM to Dev branch.


gitlab: 7.4.2 -> 8.0.5

Fixes #10476 .

Right, the biggest hurdle here is how to apply migrations for users on 7.4.2?

Hmm, yeah. @domenkozar should the migrations be applied by the service ? Or should the users apply it themselves. If applied by the service, I guess I could run the migration command bundle exec rake db:migrate RAILS_ENV=production in the preStart of the service. Would that work ?
Otherwise, I think users can use gitlab-runner rake db:migrate RAILS_ENV=production.
Another problem for current 7.4.2 users is that the secret file is now required to be entirely made of UTF8 characters (no random binary, hence https://github.com/NixOS/nixpkgs/pull/10535/files#diff-a8187202c46aeec728e5b8818fdce3c8L250). If we want to fix this, we'll probably want to check if the secret contains only ASCII (for convenience) and if not, regenerate it. I'll need to check what the consequences of changing the secret is first though...

I think we must provide migration automatically.

This commit should run the database migrations automatically upon starting gitlab now. I suppose that's enough (seems to be, looking at the update guides), but since I don't have a gitlab to migrate, I'm not sure, and since I can't easily install an older version, it ain't really possible for me to test it.
I found out the gitlab_shell_secret file is recreated each time the gitlab service is started, so that's not an issue, the binary secret will be changed to an ascii one automatically upon upgrading.

Bump. This is ready on my end, just waiting for a merger.

@offlinehacker @fmap @cstrahan @abbradar @teh anyone of you using gitlab 7.4.x and would like to try out these migrations?

nope

I don't have a GitLab instance at the moment, but I can try to spin one up tonight and see if the migrations work.

Bump ?

Bump

bump 2

Bump. Squashed the commits into one and gave it the proper commit message as per CONTRIBUTING.md.

Tested?
Sorry, I can't test myself :(

The old version (7.4.2) fails to install, one of the dependency is broken (therubyracer). I don't even know if anyone currently runs the gitlab server currently in the repo, given it's uninstallable right now...
The current version, freshly installed, works perfectly well on my end.

Found old nixos server with gitlab there. Tested. It works and it migrates.

Could this be backported to 15.09?


Update pr-docu-checks.yml

Switch to md-check-link
Description
Changes proposed in this pull request (what was done and why):

Use md-check-link that supports docsify and anchors properly

Changes refer to particular issues, PRs or documents:
Traceability

[ ] The PR is linked to a GitHub issue.
[ ] New features have a milestone set.
[ ] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[ ] The corresponding GitHub issue has a respective area and kind label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] Adjusted the documentation if the change is user-facing.
[ ] The feature is unit-tested
[ ] The feature is e2e-tested

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[Error: regeneratorRuntime is not defined]

Hi
Below is the list of packages installed
accounts-google                 1.0.6  Login service for Google accounts
accounts-password               1.1.4  Password support for accounts
accounts-ui                     1.1.6  Simple templates to add login widgets to an app
angular                         1.3.1  Everything you need to use AngularJS in your Meteor app
dotansimha:accounts-ui-angular  0.0.2  AngularJS wrapper for Meteor's Account-UI package
es5-shim                        4.1.14  Shims and polyfills to improve ECMAScript 5 support
flemay:less-autoprefixer        1.2.0  The dynamic stylesheet language + Autoprefixer
harrison:babyparse              1.0.1  BabyParse is a fast, reliable CSV parser based on PapaParse
jquery                          1.11.4  Manipulate the DOM using CSS selectors
meteor-base                     1.0.1  Packages that every Meteor app needs
mobile-experience               1.0.1  Packages for a great mobile user experience
mongo                           1.1.3  Adaptor for using MongoDB and Minimongo over DDP
pbastowski:require              0.0.3  Simple require stub
semantic:ui                     2.1.7  Official Semantic UI Integration for Meteor
service-configuration           1.0.5  Manage the configuration for third-party services
session                         1.1.1  Session variable
standard-minifiers              1.0.2  Standard minifiers used with Meteor apps by default.
tracker                         1.0.9  Dependency tracker to allow reactive callbacks
universe:modules                0.6.7  Use ES6 / ES2015 modules in Meteor with SystemJS!

In my server side code, I cannot create generators for my custom classes.
  *[Symbol.iterator]() {
    for (let item of this.items) {
      yield item;
    }
  }

As it throws [Error: regeneratorRuntime is not defined]. I couldn't test yet with the ecmascript package since I'm dependent on angular-meteor, so I'd like to know if there is a workaround considering the dependencies listed above.
Thanks!

It seems I found a workaround for this. First, install meteorhacks' npm package:
meteor add meteorhacks:npm
A package.json should be generated in the app's root folder; then, add "babel-polyfill": "6.3.14" and run meteor again.
Finally, import the package:
let polyfill = Meteor.npmRequire('babel-polyfill');
If there are better workarounds, I'll be glad to know.

Hi @webpolis ,
This looks like an issue with https://github.com/pbastowski/angular-meteor-babel/, that maintained by @pbastowski .

@webpolis I will look into this issue.

@webpolis Could you please provide an actual code sample that I can work with or otherwise a link to a repo that I can test with?

@pbastowski  yes, feel free to clone https://github.com/webpolis/qfx - it's a prototype -.
And this is the file where I implement an iterator: https://github.com/webpolis/qfx/blob/master/server/lib/importers/importer.import.js

@webpolis
I have a branch of angular-babel that should fix the issue you're experiencing. The problem was with babel helpers, such as the browser-polyfill, only being loaded on the client, because, well, the file was called "browser-...", so, I didn't think to include it on the server. Stupid, I know, but I have even funnier stories to tell over a drink :)
To test the new branch do this:

create a packages folder in your app's root folder (if you don't already have one)
clone my angular-babel repo https://github.com/pbastowski/angular-meteor-babel.git into your packages folder
checkout branch feature/server-systemjs-sourcemaps

Restart Meteor and see if the problem is fixed. I tested with you example file and everything compiled properly for me. I hope you get the same result.
BTW, this new branch has server side SystemJS support also. If you are interested in that then please make sure to read the readme carefully, as you will have to make some simple changes to your code to make things work.

@pbastowski ping me when you release a new version so I will update our dependencies as well, thanks!

@urigo sure thing.

you rock @pbastowski

hi,
When I use<EMAIL_ADDRESS>with angular-babel 1.0.8 my app doesn't start anymore here is the error I got :
W20160103-11:28:44.842(1)? (STDERR)
W20160103-11:28:44.843(1)? (STDERR) /Users/Thoms/.meteor/packages/meteor-tool/.1.1.10.r3ykmi++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20160103-11:28:44.843(1)? (STDERR) 						throw(ex);
W20160103-11:28:44.843(1)? (STDERR) 						      ^
W20160103-11:28:44.843(1)? (STDERR) Error: Match error: Failed Match.OneOf or Match.Optional validation in field maxSize
=> Exited with code: 8

Everything works fine with<EMAIL_ADDRESS>with angular-babel 1.0.7.
Here is the list of package installed:
accounts-base                       1.2.2  A user account system
accounts-password                   1.1.4  Password support for accounts
alanning:roles                      1.2.14  Authorization package for Meteor
angular                             1.3.2* Everything you need to use AngularJS in your Meteor app
angular-meteor-auth                 0.0.2  Angular-Meteor authentication module
angular:angular-animate             1.4.8  AngularJS (official) release. For full solution: http://angular-meteor.com/
angular:angular-aria                1.4.8  AngularJS (official) release. For full solution: http://angular-meteor.com/
angular:angular-material            0.11.4* Google Material design written for AngularJs (official repo)
angular:angular-messages            1.4.8  AngularJS (official) release. For full solution: http://angular-meteor.com/
angularui:angular-ui-router         0.2.15  angular-ui-router (official): Flexible routing with nested views in AngularJS
angularui:ui-calendar               1.0.1  AngularJs directive for Arshaw FullCalendar
cfs:ejson-file                      0.1.4  CollectionFS, FS.File as EJSON type
cfs:filesystem                      0.1.2  Filesystem storage adapter for CollectionFS
cfs:s3                              0.1.3  Amazon Web Services S3 storage adapter for CollectionFS
cfs:standard-packages               0.5.9  Filesystem for Meteor, collectionFS
check                               1.1.0  Check whether a value matches a pattern
chrismbeckett:toastr                2.1.2_1  Gnome / Growl type non-blocking notifications
edgee:slingshot                     0.7.1  Directly post files to cloud storage services, such as AWS-S3.
es5-shim                            4.1.14  Shims and polyfills to improve ECMAScript 5 support
fourseven:scss                      3.4.1  Style with attitude. Sass and SCSS support for Meteor.js (with autoprefixer and sourcemaps).
jquery                              1.11.4  Manipulate the DOM using CSS selectors
matb33:collection-hooks             0.8.1  Extends Mongo.Collection with before/after hooks for insert/update/remove/find/findOne
meteor-base                         1.0.1  Packages that every Meteor app needs
meteorhacks:aggregate               1.3.0  Proper MongoDB aggregations support for Meteor
meteorhacks:kadira                  2.26.3  Performance Monitoring for Meteor
meteorhacks:npm                     1.5.0  Use npm modules with your Meteor App
meteorhacks:zones                   1.6.0  Zone.Js integration for meteor
mizzao:user-status                  0.6.6  User connection and idle state tracking for Meteor
mobile-experience                   1.0.1  Packages for a great mobile user experience
mongo                               1.1.3  Adaptor for using MongoDB and Minimongo over DDP
msavin:mongol                       1.6.2  In-App MongoDB Editor
npm-container                       1.2.0+ Contains all your npm dependencies
percolate:synced-cron               1.3.0  Allows you to define and run scheduled jobs across multiple servers.
planettraining:jquery-fullcalendar  2.5.0  Arshaw FullCalendar jquery plugin original files.
session                             1.1.1  Session variable
standard-minifiers                  1.0.2  Standard minifiers used with Meteor apps by default.
thoms:angular-datepicker-720kb      0.0.2  angular-datepicker-720kb
thoms:angular-i18n-locale-fr        0.0.1  angular-i18n-locale-fr
thoms:angular-moment                0.0.3  angular-moment-fr
thoms:material-design-icons         1.0.1  material-design-icons
thoms:moment-timezone-fr            0.0.3  moment-timezone-fr
thoms:ng-draggable                  0.1.8  ngDraggable
tracker                             1.0.9  Dependency tracker to allow reactive callbacks
underscore                          1.0.4  Collection of small helpers: _.map, _.each, ..
```

May the error comes from a package?

Thanks

@ThomsC
Are you on windows or os x
What browser are you using chrome/firefox/safari?
What browser version?
I have just pushed v1.0.9 of pbastowski:angular-babel, with an attempt to fix the problem. Could you please add the package directly to your project:
meteor add pbastowski:angular-babel

and then try again and let me know the result?
Thanks.

@pbastowski
I'm on os x
With<EMAIL_ADDRESS>I got no error.
Thanks!


Server API plugin support

It would be great to have support for the Server API plugin. Is there any plans to add it?

Yeah, I would like to be able to implement it into the script. The issue is that they don't have a Linux build. Additionally, we run ASA under Proton because the ASA devs don't have a Linux build for ASA. So, it would take some work to get the API working under Linux using Proton or Wine.

I saw that Mr. Mschnitzer managed to implement the server API in his ASA docker image. I was just curious, Don't want to migrate to another image because yours is really good :) Thanks.

Thanks for letting me know. A lot of people have been asking if I will implement it. Now that I see others have gotten it working, I might be able to implement it in my script as well. I'll give it a shot and see if I can get it working with my version of the script.


feat(bridge/assets): include Propy token addresses

References: https://github.com/base-org/web/pull/243
The PRO token has been added to the Superchain Token List so I wanted to reopen this pull request according to the feedback that I received from the Coinbase team.
Here is where PRO was added: https://github.com/ethereum-optimism/ethereum-optimism.github.io/pull/626

Please follow the PR template!


Add script to invoke E2E tests on CI using ProdCon results

Part of https://github.com/aspnet/Universe/issues/706

:up: :date:


Add API endpoint to recreate a backup

To update the image of a CronJob and to apply the new values, it's necessary to recreate backups. Currently, this happens via deletion & new creation of a backup pipeline. As side effect, all pipeline runs are deleted. A recreation (PUT) endpoint should preserve the pipeline runs, but also recreate the CronJob.

Instead of an API endpoint, this should be done automatically: https://github.com/DSD-DBS/capella-collab-manager/issues/1749


Refactor MiqRequest factories

A small refactoring that does two things:

removes the "abstract" factory for MiqRequest because it's not used/needed and fails Factory Girl's built-in linter
gives the "concrete" factories a requester, because it's required

@miq-bot add-label test, refactoring

@imtayadeway We usually keep STI subclass factories in the same file as the base class factory.  Also, I would expect required things to be defined in the factory for the base class even if that base class factory isn't used.  It provides a single place to update when those things change.

@bdunne can I move the already-separate :service_reconfigure_request factory in with the others, if that's the case?


Update pWS chart version to 3.0

Checked the https://helm.soketi.app/index.yaml to look for the 0.3.0 version and is not there.
Can you push it up?
Have the server running pretty good on local trying to have it running on my k8s :)
Thanks,
Francisco

Keep the version set to 2.1.1 until I'll upgrade the values and the chart to 0.3 😁
Can you share the configuration for your run? It seems like you activated Prometheus, but I want to know what other options you set.

I did without any configs and the error happened, for what I checked on the code the prometheus driver is always imported:
export class Metrics implements MetricsInterface {
    /**
     * The Metrics driver.
     */
    protected driver: MetricsInterface;

    /**
     * Initialize the Prometheus exporter.
     */
    constructor(protected server: Server) {
        if (server.options.metrics.driver === 'prometheus') {
            this.driver = new PrometheusMetricsDriver(server);
        } else {
            Log.error('No metrics driver specified.');
        }
    }
}

directly on the Docker it seems to give no issue, so I'm guessing is something related to this vars, not having the correct value:
protected registerMetrics(): void {
    let port = this.server.options.port;
    let prefix = this.server.options.metrics.prometheus.prefix;
}

and give that validation error. looks pretty strait forward but probably related to that.
Thanks for the quick response.
Francisco

I have updated the pWS chart to 0.4.0, it seems like for some reason I misspelled the version number.
I still don't know where your error comes from, I use it internally for a project without any configurations and it just works???

Thanks Will try it out, and get back to you.

Got it. Thanks, will check it later today and get back to you.

besides that I run the repo directly and got the same issue:
There is a print:

Hey @rennokki any luck checking this out?
Thanks,
Francisco


rtl languages issue

i want to use this widget with rtl direction.. can i find the solution

What problem with rtl direction? Do you mean that you need to have (0, 0) coordinate to be at top right corner?

when i change the page direction to rtl
i found 2 problems

when i drag an item, it appears in another place "solved"
when i resize item, the placeholder appears in far place

Letting the grid use RTL is irrelevant. Only the content of your widgets need to be RTL.
On the gridstack DIV set CSS to be direction: ltr; while the rest of you page may be rtl.
Then on each widget content box set the CSS to be  direction: rtl;

It's fixed now


useGETForQueries: It doesn't pick up query variables at all

A big thanks for helping to get the feature in, hope I'm not too annoying by reporting all these bugs ;)
Sandbox to reproduce:
https://codesandbox.io/s/nostalgic-moore-ie2j1?file=/.meshrc.json
I'd expect to be able to provide query variables for GET queries, but it seems it currenly is skipping those variables.
0.2.13 - useGETForQueries=false:

0.2.13 - useGETForQueries=true
Doesn't start at all, as expected, you fixed this in #633 thanks a lot!
0.2.14 - useGETForQueries=false

0.2.14 - useGETForQueries=true

We'd love to get those bug reports from the community, thank you very much for every contribution you made so far including this bug report :) All those are making Mesh better day by day.
I have a question. How do you send variables to this GraphQL endpoint? Could you send an example GET request? Maybe there is a different use case for GET with GraphQL.

Thanks for the response @ardatan!
For the query:
query urlResolver($url: String!){
  urlResolver(url: $url) {
    id
    __typename
  }
}

variables:
{
  "url": "/"
}

The query variables should be send as a query parameter as well for a GET request:
curl $'https://develop.pwa-venia.com/graphql?query=query%20urlResolver(%24url%3A%20String\u0021)%20%7B%0A%20%20urlResolver(url%3A%20%24url)%20%7B%0A%20%20%20%20id%0A%20%20%20%20__typename%0A%20%20%7D%0A%7D%0A&operationName=urlResolver&variables=%7B%22url%22%3A%22%2F%22%7D' \
  -H 'content-type: application/json'

The current result (as you can reproduce in the sandbox):

@paales Thank you for reproduction. I fixed it in GraphQL Tools's URL Loader. You can try it with the canary version of that PR;
https://github.com/Urigo/graphql-mesh/pull/806

Available in the latest version!

@ardatan Just to confirm, it is working as expected!


chore(site): add press kit

Changes

#1722  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add the label merge-queue to this PR to add it to the merge queue.
You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.


make saml auth request option forceauthn configurable

Description
This PR...

Fixes: #9701

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[x] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan test

@DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests

[SF] Trillian test result (tid-11600)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 60218 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9756-t11600-kvm-ol8.zip
Smoke tests completed. 136 look OK, 2 have errors, 3 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_secure_vm_migration
Error
134.12
test_vm_life_cycle.py

test_01_secure_vm_migration
Error
134.12
test_vm_life_cycle.py

test_01_redundant_vpc_site2site_vpn
Error
542.26
test_vpc_vpn.py

test_01_vpc_site2site_vpn_multiple_options
Error
339.74
test_vpc_vpn.py

test_01_vpc_site2site_vpn_multiple_options
Error
339.75
test_vpc_vpn.py

ContextSuite context=TestVPCSite2SiteVPNMultipleOptions>:teardown
Error
446.23
test_vpc_vpn.py

ContextSuite context=TestVpcRemoteAccessVpn>:setup
Error
580.99
test_vpc_vpn.py

ContextSuite context=TestVpcSite2SiteVpn>:setup
Error
654.50
test_vpc_vpn.py

all_test_webhook_delivery
Skipped
---
test_webhook_delivery.py

all_test_webhook_lifecycle
Skipped
---
test_webhook_lifecycle.py

all_test_host_maintenance
Skipped
---
test_host_maintenance.py

errors seem unrelated retrying

@blueorangutan test

@DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests

@vishesh92 can this be part of 4.19.2 milestone?

@vishesh92 can this be part of 4.19.2 milestone?

It can be

[SF] Trillian test result (tid-11615)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 73080 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9756-t11615-kvm-ol8.zip
Smoke tests completed. 139 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_secure_vm_migration
Error
404.97
test_vm_life_cycle.py

test_02_redundant_VPC_default_routes
Failure
450.38
test_vpc_redundant.py

test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers
Failure
438.32
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Failure
514.84
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Error
514.86
test_vpc_redundant.py

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11296

@blueorangutan test

[SF] Trillian test result (tid-11628)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 47068 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9756-t11628-kvm-ol8.zip
Smoke tests completed. 129 look OK, 4 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestRouterServices>:setup
Error
0.00
test_routers.py

ContextSuite context=TestCpuCapServiceOfferings>:setup
Error
0.00
test_service_offerings.py

ContextSuite context=TestServiceOfferings>:setup
Error
0.37
test_service_offerings.py

ContextSuite context=TestSetSourceNatIp>:setup
Error
0.00
test_set_sourcenat.py

test_10_attachAndDetach_iso
Error
12.93
test_vm_life_cycle.py

@blueorangutan test

@DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests

[SF] Trillian test result (tid-11637)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 42518 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9756-t11637-kvm-ol8.zip
Smoke tests completed. 133 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

cc @JoaoJandre fwd-merging this likely security enhancement on main


Staging next

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@FRidh I put gobject-introspection in here as well.

Some gtk issues https://github.com/NixOS/nixpkgs/pull/63874#issuecomment-513288337

gtk issues hopefully solved with https://github.com/NixOS/nixpkgs/pull/64982/commits/3b24d6e187f6c59cd33003aeeea54cdadc16f180
Open issues:

darwin bootstrapping fails https://github.com/NixOS/nixpkgs/issues/65136
docschema5 fails due to unzip cve patch https://github.com/NixOS/nixpkgs/commit/0238946872c1c63709790f6285bf9cc5585a6704

Known regressions caused by unzip: CVE-2019-13232 (#64909) should be fixed by https://github.com/NixOS/nixpkgs/commit/5e9f1bb343fd7ec4bf3e5541c87dca9d0ba1d82a and https://github.com/NixOS/nixpkgs/commit/16dec3974696dd2a83865f017402ac9fdfb8cf8a.

Pushed a fix for all occurrences of pyyaml 3 and merged master again. Let's hope this is the last evaluation.


[NVIDIA XLA] Add a new debug flag to turn off mlir pretty print form

Introduce a debug flag "xla_dump_enable_mlir_pretty_form" to control pretty print form when dumping mlir.
Disabling pretty form will make the dumped mlir parsable. LLVM states the pretty form is not parsable by any optimizer tool.
With pretty form:
func.func @main.3(%arg0: memref<4xi8> {lmhlo.params = 0 : index} "Arg_0.1"
The string at the end is not parsable
With xla_dump_enable_mlir_pretty_form=false:
func.func @main.3(%arg0: memref<4xi8> {lmhlo.params = 0 : index} loc("Arg_0.1")

@akuegel I think this was merged manually?

hi @akuegel , could you take another look please? Thanks.


a bug in counting total_comparisons/prompt_tokens/completion_tokens

https://github.com/ielab/llm-rankers/blob/4f632a41f688ce2ad81040dd66327ae67d85a877/run.py#L193C2-L195C66
I may have found a bug in counting total_comparisons/prompt_tokens/completion_tokens. ranker.total_compare represents the cumulative number of calls to the ranker compare function, so there is no need to set total_comparisons in the run.py file to re-accumulate.

Ok, I see the rerank function code with self.total_compare = 0. There doesn't appear to be any bugs in it.

Thanks @Ziyang1060 for checking this out.


Merge /test-sync branch from main repo for development

Merge /sync-test changes to master for development


add cloudinary adapter to ActiveStorage

Linked Issue(s)

closes #83

Description
Feature
Describe the changes made by this Pull Request.
Checklist

[x] I have read the Contributing Guide and Code of Conduct

Review app tested with automatically deployed Cloudinary. Successful.
Gives me cause to think about going back and seeing if I can get SendGrid addon to actually work. It would be excellent to have that configured for automatic deployment... Might need to setup a dummy Heroku account to test it though.
Note: Cloudinary adds only a CLOUDINARY_URL config var.


[boost-variant2] Build error on x64-windows

Package<EMAIL_ADDRESS>Host Environment

Host: x64-windows
Compiler: MSVC 19.39.33521.0
vcpkg-tool version: 2024-06-10-02590c430e4ed9215d27870138c2e579cc338772
vcpkg-scripts version: 52be198ec 2024-06-19 (10 hours ago)

To Reproduce
vcpkg install
Failure logs
-- Downloading https://github.com/boostorg/variant2/archive/boost-1.85.0.tar.gz -> boostorg-variant2-boost-1-1a5fc662.85.0.tar.gz...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Trying to load bundleconfig from C:\src\vcpkg\vcpkg-bundle.json
[DEBUG] Failed to open: C:\src\vcpkg\vcpkg-bundle.json
[DEBUG] Bundle config: readonly=false, usegitregistry=false, embeddedsha=nullopt, deployment=Git, vsversion=nullopt
[DEBUG] Metrics enabled.
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Feature flag 'dependencygraph' unset
Downloading https://github.com/boostorg/variant2/archive/boost-1.85.0.tar.gz
[DEBUG] Trying to hash C:\src\vcpkg\downloads\boostorg-variant2-boost-1-1a5fc662.85.0.tar.gz.41268.part
[DEBUG] C:\src\vcpkg\downloads\boostorg-variant2-boost-1-1a5fc662.85.0.tar.gz.41268.part has hash f4f79ab208c534a7e7a285e2722dbe14c7d810a8ad9e73dd2a6de51862b3b08928802251e8661815e5bf9f82eefd10125a984724ae4670823307edc93cd679a8
error: Failed to download from mirror set
error: File does not have the expected hash:
url: https://github.com/boostorg/variant2/archive/boost-1.85.0.tar.gz
File: C:\src\vcpkg\downloads\boostorg-variant2-boost-1-1a5fc662.85.0.tar.gz.41268.part
Expected hash: 1a5fc662247f3e8437dfeec552fc3c5c380112db4a2e6b0c9e7abad1c06ec0d07dcbb528c244884f2507ac14174f4db3fadfd211e84d9bcd26a5ab5f33cb4d97
Actual hash: f4f79ab208c534a7e7a285e2722dbe14c7d810a8ad9e73dd2a6de51862b3b08928802251e8661815e5bf9f82eefd10125a984724ae4670823307edc93cd679a8
[DEBUG] D:\a\_work\1\s\src\vcpkg\base\downloads.cpp(997):
[DEBUG] Time in subprocesses: 0us
[DEBUG] Time in parsing JSON: 6us
[DEBUG] Time in JSON reader: 0us
[DEBUG] Time in filesystem: 576us
[DEBUG] Time in loading ports: 0us
[DEBUG] Exiting after 817 ms (797555us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):

      Failed to download file with error: 1
      If you are using a proxy, please check your proxy setting. Possible causes are:

      1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
         to `https://address:port`. This is not correct, because `https://` prefix
         claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
         , etc..) is an HTTP proxy. Try setting `http://address:port` to both
         HTTP_PROXY and HTTPS_PROXY instead.

      2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
         set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
         The value set by your proxy might be wrong, or have same `https://` prefix issue.

      3. Your proxy's remote server is out of service.

      If you've tried directly download the link, and believe this is not a temporary
      download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
      to report this upstream download server failure.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
  scripts/cmake/vcpkg_from_github.cmake:106 (vcpkg_download_distfile)
  buildtrees/versioning_/versions/boost-variant2/71aeea8a9326c7b978940c83970790090ddcf8f9/portfile.cmake:3 (vcpkg_from_github)
  scripts/ports.cmake:191 (include)

Additional context
vcpkg.json
{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg-tool/main/docs/vcpkg.schema.json",
  "name": "chatterino",
  "version": "2.0.0",
  "builtin-baseline": "01f602195983451bc83e72f4214af2cbc495aa94",
  "dependencies": [
    "boost-asio",
    "boost-circular-buffer",
    "boost-foreach",
    "boost-interprocess",
    "boost-signals2",
    "boost-variant",
    {
      "name": "qt5compat",
      "default-features": false,
      "features": [
        "textcodec"
      ]
    },
    {
      "name": "qtbase",
      "default-features": false,
      "features": [
        "concurrent",
        "gui",
        "jpeg",
        "network",
        "openssl",
        "png",
        "widgets"
      ]
    },
    {
      "name": "qtimageformats",
      "default-features": false,
      "features": [
        "webp"
      ]
    },
    "qtsvg"
  ],
  "features": {
    "benchmarks": {
      "description": "Build benchamarks",
      "dependencies": [
        "benchmark"
      ]
    },
    "tests": {
      "description": "Build tests",
      "dependencies": [
        "gtest"
      ]
    }
  }
}

This seems to be the same problem as in #39409
It seems to try to compare the hash of a .part file instead of the complete download
Really frustrating that there isn't a "ignore hashes" option

I am seeing a similar build failure with a dependency on libsodium -- suddenly, SHA512SUM mismatch.

This issues is already fixed here. boost moved existing git tags instead of making a new patch release. Now you need to move, too.

Closed as duplicate of #39409.

I have no idea how to fix this.
This is a fresh install of vcpkg.
I have git pulled and ran bootstrap multiple times.
Still getting this error