Update S3/Azure TypeScript definition file

This updates the S3 and Azure properties to be optional in the typescript definition file for properties with default values.

@rnicholus Any idea why Travis build is failing on this one?
Doesn't seem like anything in this PR would break the build.

looks like a temporary Travis issue, somehow the server port was already in use. Restarting the build fixed it.

@rnicholus Just wanted to know out of 3 merging options, what is your preferred way of merging PR's?
I'll do the same for any future merges.

I'm a fan of squashing all commits into one with a single descriptive commit message.
Unfortunately I haven't fully automated releases yet or setup anyone other than myself to publish to npm, but I'd like to change that...

@jleider thanks for your contribution.

probably need to cut a new release, push to npm, github releases, etc. @SinghSukhdeep are you interested in becoming more involved in this project? If so, I'd like to give you the power to do all of these things.

I'll be happy to help as much as I can.
I've never done something like this before, so please let me know the exact steps to follow.

So as much as I'd like to release this, the build is failing due to something Travis-CI "engineers" broke on their end (which tends to happen far too often). So I'm going to have to hold off until both this is fixed and I get another block of time to release a new version. Sorry.

honestly don't know when i'll be able to come back to this, so i was able to work around Travis' latest bug, and released 5.14.3.


JAX-RS 2.1 TCK issues

In testing the JAX-RS 2.1 TCK, we've found a few issues:

The Headers.toHttpLanguage(...) method will return things like "en-" in some cases - I believe this is because there is an empty (but non-null) country passed in as part of the Locale.  This seems like an error condition to me, but it seems to be tested by the TCK, so the fix is pretty simple - just checking whether the country is non-empty rather than just non-null.
The javadoc for Invocation.Builder.headers(...) method says that when called, all pre-existing headers must be removed.  Currently, we're just adding the new headers, but not removing the old ones.

I provided test cases for these conditions, but I won't have confirmation that these fixes actually pass the TCK until tomorrow, but I wanted to get the PR out now to ensure that the fixes get into the 3.2.2 release.  If the fixes are unsuccessful, I'll abort this PR and try again after 3.2.2 is released.  If they are successful (and assuming no glaring issues uncovered during review), then I'll plan to merge ASAP.
Thanks, Andy

@sberyozkin I agree with your update.  I'm not sure whether the clone is required for somewhere else though.  I figured the safest thing was a new method to clear the headers, so as to not disrupt other code flows.
I did make a couple more changes though - the spec also says that the user can pass in a null headers map, which means that we need to just clear the headers - so I added a null check.  I also changed the removeAllHeaders method to clear the headers map rather than creating a new one.
Thanks for the review!

I added a new commit.  Some SSE systests were failing because the SSE code was calling builder.headers(...) which was clobbering some required headers.  This commit changes it to only add the one header rather than replacing them all.  With this last commit, the SSE systests pass for me locally.  Assuming they pass Jenkins here, I think this PR should be good to merge.
Thanks All!

And one more TCK fix.  CXF was failing with a ClassCastException when the TCK added a new non-String object header in a ClientRequestFilter.  CXF converts these header objects to Strings earlier in the flow, but does not convert them after filters have been invoked.  When CXF goes to put those non-String headers on the wire, it fails because it cannot cast them a String.  This fix converts the headers after invoking the filter method.


Add ufsAccess metrics

What changes are proposed in this pull request?
Add metrics to count file nums which access metadata
Why are the changes needed?
We want to add a metric MASTER_UFS_METADATA_ACCESS_COUNT and now we can not get ufs metadata access num info
Does this PR introduce any user facing changes?
Users can get ufs metadata access num metrics in webui
fix #13734

Codecov Report

Merging #13953 (7f9309e) into master (430e4a8) will decrease coverage by 25.82%.
The diff coverage is 27.65%.

@@              Coverage Diff              @@
##             master   #13953       +/-   ##
=============================================
- Coverage     43.24%   17.41%   -25.83%
+ Complexity     9136     2685     -6451
=============================================
  Files          1411     1413        +2
  Lines         81723    81897      +174
  Branches       9900     9912       +12
=============================================
- Hits          35344    14266    -21078
- Misses        43429    66377    +22948
+ Partials       2950     1254     -1696

Impacted Files
Coverage Δ

...va/alluxio/underfs/UnderFileSystemWithLogging.java
20.73% <20.93%> (+0.02%)
:arrow_up:

...ommon/src/main/java/alluxio/metrics/MetricKey.java
97.38% <100.00%> (-0.34%)
:arrow_down:

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/alluxio/wire/AlluxioMasterInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/job/meta/JobIdGenerator.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/stress/BaseParameters.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/client/UnderStorageType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...main/java/alluxio/underfs/options/ListOptions.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 671 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 430e4a8...7f9309e. Read the comment docs.

@kuszz @maobaolong , we are testing https://github.com/Alluxio/alluxio/pull/13671 using a newly added stressbench on RPC , to understand if #13671  is introducing any performance overhead. If not, we will merge that PR and you can totally follow the same pattern there


More information available?

Hi Jonathan,
Is there a webpage with more information regarding this project? I'm looking for a xmpp server running in Azure. This project came up but would like some more information if that's possible.
Thanks in advance!
Regards.
Guido

Sorry Guido, it was a pet project that I simply ran out of time for - I'll delete this repository to avoid any further confusion.
Give this a go: https://hub.docker.com/r/rroemhild/ejabberd/ https://azure.microsoft.com/en-us/overview/containers/


Map imp-orts and Wind empowerment display incorrectly

In zFarm, whenever I import my save, it sets my map imp-orts to my maximum imps purchased, not my number of map imps purchased. It also sets my Wind empowerment to 1, however my highest zone reached is 72, so I haven't gotten to nature empowerments at all.

That sounds like a cache issue. Try force-refreshing the page (Ctrl + F5).

Seems like you were right, upon loading it up today and plugging my save in, wind empowerment was automatically set to zero, the only thing that I have to change is map imp-orts back to 1, but I only do that once and it doesn't change it back, thanks


Getting "Access Denied" message when trying to connect to microbit

Describe the bug
With a single browser, single tab open, when I try to connect to the microbit, I get an "Access Denied" error. In the dev console I see this exception:
{message: 'Access denied.', stack: undefined, name: 'SecurityError', jacdacName: undefined}
Repros in both Edge and Chrome.
OS: Windows 11
To Reproduce
Steps to reproduce the behavior:

Close all browsers
Open a single browser, single tab
Navigate to https://aka.ms/jacdac
Plugin your microbit to USB
In the browser click "Connect"
Go through the flow to connect your microbit
Notice it fails!

Expected behavior
It would connect successfully.
Screenshots
So far so good, it sees the microbit:

Microbit appears in the device selection dialog:

After clicking "Connect", the error appears:

Dev console output:

Did you flash a microbit without jacdac in it by any chance?
Sent from Outlookhttp://aka.ms/weboutlook

From: Eric Anderson @.>
Sent: Monday, October 11, 2021 11:32 AM
To: microsoft/jacdac @.>
Cc: Subscribed @.***>
Subject: [microsoft/jacdac] Getting "Access Denied" message when trying to connect to microbit (#786)
Describe the bug
With a single browser, single tab open, when I try to connect to the microbit, I get an "Access Denied" error. In the dev console I see this exception:
{message: 'Access denied.', stack: undefined, name: 'SecurityError', jacdacName: undefined}
Repros in both Edge and Chrome.
OS: Windows 11
To Reproduce
Steps to reproduce the behavior:

Close all browsers
Open a single browser, single tab
Navigate to https://aka.ms/jacdachttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Faka.ms%2Fjacdac&data=04|01|jhalleux%40microsoft.com|58bffd139c104597dc1008d98ce57b07|72f988bf86f141af91ab2d7cd011db47|1|0|637695739513536884|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=RZacY6OxFMwwSZ0DpXxHIJl4GhSSP2idAL28Ll7yg9c%3D&reserved=0
Plugin your microbit to USB
In the browser click "Connect"
Go through the flow to connect your microbit
Notice it fails!

Expected behavior
It would connect successfully.
Screenshots
So far so good, it sees the microbit:
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F12176807%2F136837369-085456d1-03eb-4a26-a6e0-e97f33bd38fc.png&data=04|01|jhalleux%40microsoft.com|58bffd139c104597dc1008d98ce57b07|72f988bf86f141af91ab2d7cd011db47|1|0|637695739513536884|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ewr7k32BGXDzgUp3VqRekhMNxweJsKvDCTGnA5uOSCM%3D&reserved=0
Microbit appears in the device selection dialog:
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F12176807%2F136837409-fbdd7ef4-6572-425a-838a-4d660a4888e0.png&data=04|01|jhalleux%40microsoft.com|58bffd139c104597dc1008d98ce57b07|72f988bf86f141af91ab2d7cd011db47|1|0|637695739513546876|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EoJrNA8lIAPg3hqcHzyFErDIBlqwkwilQjG8DIHcYyg%3D&reserved=0
After clicking "Connect", the error appears:
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F12176807%2F136837460-65ff80c6-70e5-4256-a186-116126b6654e.png&data=04|01|jhalleux%40microsoft.com|58bffd139c104597dc1008d98ce57b07|72f988bf86f141af91ab2d7cd011db47|1|0|637695739513546876|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=aJz30zWz5uuVyQKPHU4a3GvsB4fbHSfB9K%2B09LxjwFU%3D&reserved=0
Dev console output:
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F12176807%2F136837650-062583cb-7616-4d8d-9a9c-131105da509c.png&data=04|01|jhalleux%40microsoft.com|58bffd139c104597dc1008d98ce57b07|72f988bf86f141af91ab2d7cd011db47|1|0|637695739513556867|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=yOwsHPntVEzrSG5l2RMR7eIVbxA3gwa7%2FKF1DT8hueo%3D&reserved=0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fjacdac%2Fissues%2F786&data=04|01|jhalleux%40microsoft.com|58bffd139c104597dc1008d98ce57b07|72f988bf86f141af91ab2d7cd011db47|1|0|637695739513556867|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=d61tqCrUCVK9CInFKGDmel73HrW1d1YdroB39cLIJ5o%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKJGOKYABEX6M2BAW6LUGMUTZANCNFSM5FY2KJRA&data=04|01|jhalleux%40microsoft.com|58bffd139c104597dc1008d98ce57b07|72f988bf86f141af91ab2d7cd011db47|1|0|637695739513566861|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7pYHmOTZLYW0tB4uWK8EmAdFCDxQuAah6YMXFeFBYKM%3D&reserved=0.

I think this might be due to the program on micro:bit not having the Jacdac stack. Need to investigate.

The message for missing jacdac is different

We still haven't found why this happened on Eric's machine. Note that Eric sent me his micro:bit and things worked fine here too. This is a generic micro:bit issue. closing here.

I had a kind of similar error & found that resetting the Browser Permissions for the Site woked for me!


Add support for more targeting operators

Add support for new comparison operator on experiment targeting.
Supports gt, ge, lt, le, and ne operators
👓 @spladug @mattknox
Recreation of https://github.com/reddit/baseplate/pull/215

🔕 le gem

@fruchtose ?


Temperature slide responsive time too long

The temperature slide responds very slowly after the user slides it.

Fixed.


atom crashes if you try to open any files in webkit.js repo

If you clone https://github.com/trevorlinton/webkit.js, start atom in it (with or without the --safe flag), it appears to start up fine and lists files in the left file list, but as soon as you open any of them, Atom freezes for a few seconds, and then a dialog appears "The editor has crashed. Please report this issue to https://github.com/atom/atom. [Keep It Open] [Reload] [Close]".

Same thing happens for me with http://github.com/servo/servo/

There's an issue with libgit and large repo.
https://github.com/atom/atom/issues/9339
it's on the process of being fixed.

Thanks @jeancroy!


Bug fixes

Fixes bugs created by updates to the official theme

It looks like there's an issue when you manage an instance. It shows the original stuff in the background still.

i have not been able to reproduce this issue, but i fixed some other things i'd missed previously and some new things from new updates.

it just occurred to me what is causing your issue. There is a supplied background image which you don't seem to have. I'll add a solid background colour as a failsafe in my next commit

Oof. That's my bad. I grabbed the files in the PR without thinking about a separate image.

i was going to do like this,

but instead, i just base64 encoded it so now the image isn't required. 🤓

I like that idea. Looks good.

Looks like there's conflicts that need resolved in your branch. It won't let me merge until they're dealt with.

Did you want to close this and make a new PR due to the conflicts?


Support st_contains using H3 index

This is a follow up PR for : https://github.com/apache/pinot/pull/7252 since that one seems doesn't get update for long time.
The idea is to convert input geometry into a list of h3 cells by using polyfill. But h3 polyfill only fills with the hexagons whose centers are contained by the geometry.
so creating a method coverGeometryInH3 to return the set of H3 cells at the specified resolution which completely cover the input shape.
Instructions:

The PR has to be tagged with at least one of the following labels (*):

feature

(**) Use release-notes label for scenarios like:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

Codecov Report

Merging #8498 (0034808) into master (7dd374f) will decrease coverage by 44.74%.
The diff coverage is 9.09%.

:exclamation: Current head 0034808 differs from pull request most recent head c70e677. Consider uploading reports for the commit c70e677 to get more accurate results

@@              Coverage Diff              @@
##             master    #8498       +/-   ##
=============================================
- Coverage     70.69%   25.95%   -44.75%
=============================================
  Files          1672     1661       -11
  Lines         87476    87187      -289
  Branches      13240    13214       -26
=============================================
- Hits          61843    22631    -39212
- Misses        21332    62397    +41065
+ Partials       4301     2159     -2142

Flag
Coverage Δ

integration1
?

integration2
25.95% <9.09%> (-0.13%)
:arrow_down:

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...perator/filter/H3InclusionIndexFilterOperator.java
0.00% <0.00%> (ø)

.../org/apache/pinot/segment/local/utils/H3Utils.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...ava/org/apache/pinot/core/plan/FilterPlanNode.java
59.45% <38.88%> (-29.12%)
:arrow_down:

.../java/org/apache/pinot/spi/utils/BooleanUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/NoOpRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/config/table/FSTType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/data/MetricFieldSpec.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/spi/utils/BigDecimalUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/common/tier/TierFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/spi/config/table/TableType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1222 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7dd374f...c70e677. Read the comment docs.

High-level comment is that using H3 index can speed up query evaluation, but we need to ensure it does not change query results. H3's Polyfill returns a set of H3 for approximation, but we need one more round of evaluation to check the borderlines.


duplicate projects

When using the tool i noticed that there are a bunch of duplicate entries showing up which makes me worry that that last push from the previous capstone or sometime between there resulted in duplicate project data. See exported data.
[

Uploading Community Engagement Partnership Initiative-Projects Report (8).csv…
](url)

@MansiKamthan @SachinPawaskarUNO

Deployed in production


[GHSA-pf59-j7c2-rh6x] Exposure of Sensitive Information to an Unauthorized Actor and Insertion of Sensitive Information Into Sent Data in Calico

Updates

Affected products

Comments
The previously mentioned version "0.8.6" didn't appear to exist for the calico project at all. I pulled the corrected affected and patched versions from https://www.tigera.io/security-bulletins-tta-2020-001/.

@darakian - it seems this update also hasn't made it through to the published record https://github.com/advisories/GHSA-pf59-j7c2-rh6x yet.  Are you able to nudge it along?

@westonsteimel just kicked it and it looks updated to me. Thanks for the ping and sorry about that! 👍


ComboBox indices are not updated well when bound collection is updated

Describe the bug
My combobox is bound to a collection that implemets INotifyCollectionChanged. if I clear the collection, add items more than the previous count, the combobox menu updates with all the new items, and the old ones cleared as expected. But selecting any of the items which indices exceed the last index in the previous collection will set the selected index to -1 and clear the box text, even though the item exist in the collection and appears in the menu
To Reproduce

Create a Combobox, in code or Xaml
Set the Items property to an observerable collection with items
Clear the items, and add more items than what was in before
Select any of the new items with indices greater that the last item in the previous collection
Selected Index is set to -1

Expected behavior
Selected Index should be set to the correct index
Desktop (please complete the following information):

OS: Windows 10 20H2
Version: 0.10.0

@emmauss i was wondering if you can provide a minimal repro for this bug, i cant seem to reproduce this bug here

It should have been fixed since 0.10.0

@emmauss i was wondering if you can provide a minimal repro for this bug, i cant seem to reproduce this bug here

Ok, https://github.com/emmauss/combobox_test
The issue occurs with an implementation of ObservableDictionary. I tested it with two implementations, one on nuget, and a second one, ObservableDictionary2 in the ui class. They both have issues with the combobox.

Does this video demostrate the issue well?
https://user-images.githubusercontent.com/16554748/128321506-ca6366e6-8601-4db2-8a31-65655b06e8ed.mov

okay i can confirm this bug on Avalonia master. This is a big bug indeed.

Added a check to https://github.com/AvaloniaUI/Avalonia/pull/6485 which throws when we encounter which a collection which implements INotifyCollectionChanged but not IList.

I suppose folks will just have to dynamically transform their lists to IList compatible types.


Fix Xcode 12.5 compilation

As per Xcode 12.5 Release Notes, the TestingProcedureKit target which uses
XCTest needs to specify ENABLE_TESTING_SEARCH_PATHS in order to be able to find
XCTest symbols.
See https://developer.apple.com/documentation/xcode-release-notes/xcode-12_5-release-notes

Thanks for the PR, would love to see it merged in a new release, especially for use with Carthage 🙏


Use dynamic memory allocation for coordinate stoage?

Reported by Ansley B Manke on 20 Jan 2005 20:53 UTC
The array line_mem stores irregularly-spaced coordinates for all lines, plus
their cell boundaries.  (See bug 767).  Does it make sense to use dynamic memory
allocation for this?  The problem is that when we find we need more coordinate
storage, we need to keep existing coordinate data that may already have been
loaded.  This is unlike pplus dynamic memory, where we allocate memory before
loading data and sending it to pplus, so any existing data can be wiped out by a
free command.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1158

Comment by<EMAIL_ADDRESS>on 17 Feb 2005 18:11 UTC
No, this would require wiping out existing coordinates whenever we wanted to
allocate more storage...


Add substring support for binary

Which issue does this PR close?

Closes #1593 .
Rationale for this change
What changes are included in this PR?

Add substring support for (Large)BinaryArray
Add some tests
fix a bug in the test of string array
update doc

Are there any user-facing changes?
Update some docs

Codecov Report

Merging #1608 (c503c53) into master (fd9cb23) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1608      +/-   ##
==========================================
+ Coverage   82.95%   83.00%   +0.04%
==========================================
  Files         193      193
  Lines       55435    55571     +136
==========================================
+ Hits        45988    46124     +136
  Misses       9447     9447

Impacted Files
Coverage Δ

arrow/src/compute/kernels/substring.rs
100.00% <100.00%> (ø)

arrow/src/datatypes/datatype.rs
66.40% <0.00%> (-0.40%)
:arrow_down:

parquet_derive/src/parquet_field.rs
66.21% <0.00%> (+0.22%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fd9cb23...c503c53. Read the comment docs.


docs(admin) clarify PUT endpoints behavior

As a follow-up to:
Kong/kong#3007 (PR)
Kong/kong#2774 (issue)
This commit adds clarifications as to the current behavior of PUT
endpoints on the Admin API. As much as we acknowledge that their
behavior is not compliant with the HTTP specifications, we do wish to
clarify their behavior to avoid confusion as of today, until a different
implementation sees the light of day.

We could also add this note to previous versions of the Admin API docs, if said note resonates with everybody here.

Merging this - adding it to previous docs is ok but could be done as part of a separate PR (or if someone raises the issue on the previous versions of kong)


Grant Milestone: Boot server and app locker sync support

[ ] #46

[x] API models

[x] Boot response
[x] Auth responses
[x] Locker response

[x] Boot support

[x] Support requesting boot config
[x] Use boot response to configure API classes via riverpod providers

[x] Auth support

[x] OAuth flow

[x] Support opening OAuth login url in embedded webview
[x] Support token creation via code grant and PKCE

[x] Handle callback uri and send back to flutter from platform

[x] iOS
[x] Android

[x] Token riverpod provider
[x] Support requesting user profile

[x] Locker sync

[x] Support storing apps from request into locker sync db
[x] Request initial locker on login
[x] Re-sync on an interval
[x] Request removal when app is removed locally

[x] PBW downloading and installation

[ ] Grant completion validation

OAuth flow logs in
Locker is contacted (based on boot endpoint config) and adds current apps
Apps and watchfaces show on watch and install when opened
Removal of app removes from remote locker

Blocked by https://github.com/pebble-dev/rebble-appstore-api/pull/24 and https://github.com/pebble-dev/rebble-boot/pull/12


Check if category contains unread posts

Hello,
I don't if this something you've planned to do, or not, but it would be very nice to be able to check if a category contains any unread posts.
It would allow to display a colored icon or something in front of the category that has one or more unread post.
This feature exists on almost every forum so I guess it's something a lot of people would be interested in ?
I hope my explanations are understandable :)
Thanks in advance

yeas its nice its like notification on facebook.
i think comshar.com has implement samething like that.you can check out their site https://comshar.com
so on comshar.com if you create discussion then if someone post on it ,it will on notification page.
i can give method to you if you are interesting

I like the idea of unread/read tracking per category. Notifications are a little different as I imagine them only being applicable in a "subscribe to category" kind of way, which is a whole different problem.
I need to look at how threads are tracked currently to see if this is feasible without impacting performance too much.
Thanks for the suggestion :)

Yeah it's not related to notifications and category "subscription" (which I think is easily doable yourself in your app using Events).
I guess that this unread check is doable in one query per category but I don't know how to handle sub level categories :/

@Riari I think that it should be solveable by joins and counts, something like:
if($category->threads()->count() > $category->threads->readed()->count()) {
    // category has unread threads/posts
}

class Thread {
    ....
    public function scopeReaded($query) {
        $thread = $this;
        $query->whereHas('readers', function($subquery) use ($thread) {
            $subquery
                ->where('user_id', \Auth::id())
                ->where('updated_at', '>=', $thread->updated_at);
        });
    }
    ...
}

If I haven't missed something this should work!?

I've written the query to find it, if anyone can eloquentise it for a getUnreadThreadsAttribute() function on the Categories model:
`select t.id, t.title, t.updated_at
from forum_threads t
where t.id not in
(
select t.id
from forum_threads t
join forum_threads_read r
on r.thread_id = t.id
where t.updated_at <= r.updated_at
and r.user_id =
)
and t.updated_at > `

t.category_id = $this->id, obvs ;(

This appears to give me the expected results:
        $time = time();
        $age = strtotime(config('forum.preferences.old_thread_threshold'), 0);
        $cutoff = $time - $age;

        $select = DB::table('forum_threads')
                        ->whereNotIn('forum_threads.id', function($query) {
                            $query->select('forum_threads.id')
                                  ->from('forum_threads')
                                  ->join('forum_threads_read', 'forum_threads_read.thread_id', '=', 'forum_threads.id')
                                  ->where('forum_threads_read.user_id', '=', \Auth::id())
                                  ->where('forum_threads_read.updated_at', '<=', 'forum_threads.updated_at');
                        })
                        ->where('forum_threads.updated_at', '>', date('Y-m-d H:i:s', $cutoff))
                        ->where('forum_threads.category_id', '=', $this->id)
                        ->get();

Needs a little adjustment, only getting ID will speed things up, or I am sure you can do some kind of count() type thing in laravel, still working on it but definitely possible with the information available in just one query per category.

This is how I did it in the end:
public function getHasNewPostsAttribute()
    {
        if(!\Auth::id()) {
            return false;
        }

        $time = time();
        $age = strtotime(config('forum.preferences.old_thread_threshold'), 0);
        $cutoff = $time - $age;

        // Get list of read threads for the user
        $read = DB::table('forum_threads_read')
                    ->where('user_id', '=', \Auth::id())
                    ->where('updated_at', '>=', date('Y-m-d H:i:s', $cutoff))
                    ->pluck('updated_at', 'thread_id');

        $new = DB::table('forum_threads')
                    ->whereNotIn('id', $read)
                    ->where('updated_at', '>', date('Y-m-d H:i:s', $cutoff))
                    ->where('category_id', '=', $this->id)
                    ->pluck('updated_at', 'id');

        $unread = 0;

        foreach($new as $key => $thread) {
            if(array_key_exists($key, $read)) {
                if($new[$key] > $read[$key]) {
                    $unread++;
                }
            } else {
                $unread++;
            }
        }

        return $unread;

    }

I'm still reasonably sure there's a single query that will do it without any post processing, but this is working for us without any issues and little overhead for the 9 forum categories we have.


next is undefined when using sub-apps for routing

This code will have the next argument defined:
const app = polka();

app.post('/api', (req, res, next) => {
  console.log('next is', next); // next is err => err ? this.onError(err, req, res, next) : loop()
});

However, isn't the following code where a sub app is used for routing also supposed to have the next argument defined?
const app = polka();
const subApp = polka();

subApp.post('/', (req, res, next) => {
  console.log('next is', next); // next is undefined
});

app.use('/api', subApp);

... or am I missing something here?

Hey, Yes this is true of the current stable release.
It's been fixed in @next (which you can start using now) and will remain fixed going forward.
This is technically a duplicate of #77 and so I will be closing this.
Thanks for reporting though :)


realsense2-all

Today, a client needs to link with many libraries in order to have librealsense work: realsense2 realsense-file rsutils
That gets worse with DDS: realsense2 realsense-file rsutils fastcdr fastrtps foonathan_memory realdds
And the list gets longer if we add more libraries.
And it gets worse in Linux, where we also have dependencies on udev libusb-1.0 pthread and ssl crypto rt with fastdds.
This adds a realsense2-all target which bundles most of the above into one single library.
Obviously, this is only when compiling static.

added .a/.lib to our output folder, for easy finding (and since they're actually targets)
a new target, realsense2-all is now dependent on realsense2

you have to specifically target it or compile everything

with DDS, foonathan_memory was misbehaving and not getting picked up as a library

upgraded it to version 0.7-3 from -2
removed compilation of the configurations prior to inclusion in FastDDS (!!) [LRS-801]
turned off TOOLS for it (they are creating another target in our project and we've never once used them)
it's now a target and is picked up as a library

added GHA stage to the static Windows & Linux builds to compile & run an actual client that is not part of our regular makefile, to show that no additional libraries are needed

It does not yet address the Linux dependencies -- those are still outside of realsense2-all.
Tracked on [LRS-931]

Had to rebase because of conflict

Question: which is better?
realsense2-all
Or
realsense2-bundle
??

Question: which is better? realsense2-all Or realsense2-bundle ??

Not sure... your call
I think all is fine


[Question] Tagging Repositories

Dear @mateodelnorte ,
i am currently evaluating this package for future use in my projects as i want to provide an easy to maintain approach for splitting up my packages into different modules and then building it all together.
I have a question, which i as not able to answer so far:
Is it possible to tag all sub-packages from the main repository? e.g., if i release a new version of my main application, can i tag all sub-packages at the same time?
Consider the following example:
I will release a 2.0.0 version of my main application and, therefore, adapt all my other packages. However, i would like to automatically tag them as well with 2.0.0 to have a consistent state across my overall application.
Is this possible? If yes, how?
All the best

Yup. Quite easy. meta git tag [tagname]
https://github.com/mateodelnorte/meta-git/blob/master/bin/meta-git-tag

hahahaha.. ok, sorry for bothering you with this issue..
Awesome support! Thank you very much!


Multiple documents on split view

The vertical/horizontal view available now is very usefull, although it would be great to make it possible to open more than 2 documents at the same time. To split the window in N active documents, as much as desired by the user.

Surprisingly, I don't see this on our feature backlog yet, but we should add it there so others can vote on this for prioritization...  Tagged accordingly.


display week number

Hi,
Is there any way / option that I can display the week number on the calendar?
Thank you!
Mickey

Hi Mickey.
Looks like it's possible.
We'll consider it.
Thanks for the suggesstion.


One-Click Install update

One-Click Install update

"description": "Enables the use of advance rolls and functions in Natha's Numenera Tabbed character sheets (both english and french) version 4+. Read the [Wiki page)(https://wiki.roll20.net/Script:Numenera_Natha) for help about chat commands and macros.",
Should be:
"description": "Enables the use of advance rolls and functions in Natha's Numenera Tabbed character sheets (both english and french) version 4+. Read the [Wiki page](https://wiki.roll20.net/Script:Numenera_Natha) for help about chat commands and macros.",
The closing ] on [Wiki page was a ).

Thanks @shdwjk !

Would it be possible for you to expand the description section of your two scripts. With one-click install the user isn't going to the repo, won't see your help.txt file. Can you move the information necessary to use the script into the description. You can use markdown syntax to format it. Let me know if you have any questions.

@SteveKoontz Concerning the "Porte Monstre Tresors" script, everything's said (one command). Concerning the Numenera script, I removed the help.txt and added a link to the wiki page (which is fully detailed) in the description . Is it enough?
If it's ok, I'll add a wiki page for the cypher script and a link.

That sounds fine and the wiki looks good. Let me know when the cypher script is documented and I'll merge these in.

@SteveKoontz The cypher script is documented in a wiki page and the link has been added to the package.json file, in the description.
https://wiki.roll20.net/Script:Cypher_System_Sheet


Editor-Max zoom 40 with "double-clicking"

How to reproduce:
0.a. Using the Editor and a visualization
0.b. Basemap: Positron

Double click on +

You can get very high zoom (until 40), but the basemap doesn't allow so much zoom.
Gif:

Note: In the embed map, you can't get so much high level of zoom. It stops correctly (I mean: according to basemap).

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Create a binary build target for the voyager resolver

Currently when building the project, only a single binary starknet-contract-verifier is produced which runs the flow of the entire verification process. However, when testing the CLI, sometimes we want to able to test the CLI logic without calling the backend API.
We should create a binary named starknet-contract-resolver with the API calls stripped out, and only performs the dependency resolving + compilation part. We can either park it under the verifier crate or park it under the resolver crate.

working on it


Error: identifier idents no member "rdx"

I'm pretty new to Lazarus/Pascal. What's going on here?

you need x64 of Lazarus_IDE, right now i am only supporting x64 for v2, to reduce detection ratio


Some suggestions to enrich the experience

Markdown support
Several notes per day
Choose folder to store my database. With this I can share my diary with other computer.
tags

Congratulations, great work!

Hi @texcalac,
Thanks for your feedback!

Since v2, diary entries are stored in the Markdown format. You can import from and export to .md files from the menu. I decided to only support a subset of the Markdown features (bold, italics, bulleted and numbered lists) because this app shouldn't be a full-blown text editor, but rather provide a simple way to write your diary. Are there any specific Markdown features you are missing?
The same goes for tags and the ability to have multiple entries per day. Its simplicity and intuitive design are what make Mini Diary different from other apps. I'm afraid adding too many features (like tags) would move it away from that philosophy. There are many other great diary apps if you need a tool with more advanced features.
You can already change the folder where your diary file is stored: Unlock your diary and change the setting under File → Preferences. That way, you can e.g. store your file in Google Drive or Dropbox to keep it synced between devices. (Please note that this does not work with the Mac App Store version due to sandboxing. If you need the feature on a Mac, you can download the DMG of the non-sandboxed version from the releases page.)


ukvm: Clean up error messages

Ensure all error messages go to stderr.
Consistently use err()/errx()/warn()/warnx() everywhere.
Slightly less terse error messages for users.
Cosmetic cleanups for usage errors.
Remove some more commented out/debug code.
Remove "useless" success messages (CPUID, blk/net setup).

No functional change intended with the exception of the last point (silent operation).
@djwillia I think this addresses "ukvm host feature detection" in #82. #9 talks about a few other things:

TSC stability: that's a guest issue, I may add some detection to Solo5 for this if needed
guest page size support which is moot for the moment since 2MB pages are supported on all x86_64 processors)

Looks good to me, I think as far as ukvm host feature detection, this is good (and I agree that the TSC thing is a guest issue.


🧹 Resolve build warning: C4068 unknown pragma 'GCC'

Is there an existing issue for this?

[X] I have searched the existing issues

Suggestion
Resolve C4068 error that presents itself for 'GCC'
These can be seen in this example run under windows-2019-gcc-9.2.0 or windows-2019-msvc-latest.
Example:

warning C4068: unknown pragma 'GCC' [D:\a\electionguard-cpp\electionguard-cpp\build\libs\msvc\Win32\src\electionguard.vcxproj]

Possible Implementation
Resolve the C4068 errors shown for GCC.
Anything else?
MSDocs
https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4068?view=msvc-160
Replaces #188

@avinal you got it!


Don't write empty header frame if there are no trailers

Motivation
Fixes https://github.com/netty/netty-incubator-codec-http3/issues/134
The empty header written fails client-side validation
Modification
Only write header frame if there are any trailing headers
Result
Http3FrameToHttpObjectCodec generated responses don't fail netty's validation

Thanks @valerauko for the PR!

The empty header written fails client-side validation

Can you explain more about which validation is this? Also, can you add a test that verifies the behavior?

@NiteshKant When using the example client (Http3ClientExample) the response generated will throw an error,
io.netty.incubator.codec.http3.Http3HeadersValidationException: Not all mandatory pseudo-headers included.

I see some tests are failing too, I'll fix that and add a new case too if none of the fixed ones will cover this.

When using the example client (Http3ClientExample) the response generated will throw an error

Hmm that sounds similar to the issue (https://github.com/netty/netty-incubator-codec-http3/commit/67254b9307dbe9133923f7fa2c5bac12577c05c8) I fixed in the client example. I do not see this error on main anymore. Are you sure your branch was rebased when you saw this error?

I don't think this is an issue in the example client code.
The server was responding with an empty trailing header frame last (without any of the pseudo-headers of course), which would fail the client's validation of header frames.

@NiteshKant what is the status of this one ?

I'll update the tests asap if the direction of this fix is okay

@valerauko yes it does make sense to omit writing header frame if there are no trailers.
Any idea why some tests are failing after this change?

I haven't looked closely yet but I think some of the tests actually expect an empty header frame right now.
I'll push fixes in a day or two!

@NiteshKant I've fixed the test errors. There are still two failures which are assertions that expect an empty header frame to be written when you write an EMPTY_LAST_CONTENT.
I don't know what's the ideal way to deal with this situation. The failing test cases explicitly expect an empty header frame to be written, but such a frame fails validation. Should I just delete the two tests that assert this? (L190 and L501 of Http3FrameToHttpObjectCodecTest).
This change also means that since no empty header frame is written, there's nothing to attach the WRITE_FIN listener to. What's ideal here?

Thanks @valerauko

Should I just delete the two tests that assert this?

Yes the asserts that expect a frame to be written can be removed.

This change also means that since no empty header frame is written, there's nothing to attach the WRITE_FIN listener to. What's ideal here?

It may be a good time to remove the usage of WRITE_FIN listener as it is deprecated. You can instead just call shutdownOutput() on the channel like:
((QuicStreamChannel) ctx.channel()).shutdownOutput();

Thanks for the guidance. Pushed fixes.

Thanks so much @valerauko


[BUG]: Missing Node tolerations for resiliency module

Bug Description
Missing node tolerations required by the resiliency module in csm-operator.
Logs
NA
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
NA
Expected Behavior
when resiliency is enabled the node tolerations are required else it will block the node pods being not scheduled on any worker nodes, up on respin of driver pods on a node failure.
CSM Driver(s)
All csiDrivers
Installation Type
csm-operator
Container Storage Modules Enabled
No response
Container Orchestrator
kubernetes 1.30
Operating System
RHEL

link: 28552


SQL optimisations for Embed Stat report

Coverage increased (+0.01%) to 79.75% when pulling dcc1965c3681e7ea5456d0655f4c23baf44e7be6 on fix-embed-stat-csv-generation into ccb616fc701136f8a4ba5e880947cae567834766 on staging.

Fixes #1047 (maybe)


Escoria 3.0 : Loading issues and crash of demo game with Godot 3.0.x

Scenario :

Windows 10 (64 bits)

Download the current escoria 3.0 branch (commit 7f43073  - 22nd June 2019)

Download Godot 3.0.6

Load the Escoria demo game into Godot (Scan then Edit)

With Godot 3.0.6 : in the console, several errors like " ERROR: load_interactive: Condition ' err != OK ' is true. returned: Ref()            **At: core/io/resource_format_binary.cpp:997** "
(Please note that this loading error does not happen with Godot 3.0)

Run the demo game without changing anything.

In the adventure scene, click a few times to move the character

Quickly you get a game crash with error:

In the script: Error at player.gd line 365 "Invalid get index 'scale' (on base: 'null instance')"
In the console: "Node not found: Collision. At scene/main/node.cpp:1524"
This runtime crash happens with any Godot 3.0.x (tested with 3.0 and 3.0.6)

?

I found a way to fix some things.
Open player.tscn

create walk_left and idle_left with flip_h checked then
open player_anims.gd and change
"walk_right", -1 to "walk_left", 1 and
"idle_right", -1 to "idle_right", 1

By doing this instead using mirror of walk_right and idle_right we are using walk_left and idle_left.
Any question feel free to ask.

FYI, while Escoria 3.x is in progress, Escoria 2 was ported to Godot 3.2 (branch esc2-godot3.2) and featured in the godot assets lib. The Escoria in Daïza was updated too.

@Beuc What do you think is more reliable branch of escoria with Godot 3.2.x. Can you please tell what do you think about using Escoria 3.x in its present state with Godot 3.2.x. Thanks for update.

I'm using Escoria 3.x from master (with a few minor changes) with Gotot 3.2 on daily basis.
Couldn't make my game on Escoria 2 - so I'm using Escoria 3.x :)

@ArturM I saw your fork of Escoria. That means I should use your 3.2-WIP branch. Am I correct. Thanks.

Hi,

Escoria 2.0 is stable and won't change significantly; that means you can upgrade it (for fixes and/or new Godot version) and it'll keep working; however don't expect new features

Escoria 3.x is under development and may change; if it works for you right now, you can use it, but if you upgrade Escoria 3.x to get new fixes/features, you may have to make changes to your game

@Beuc Trade-off between .... and .... Waiting for ... and .... never ending part of life. Thanks for saving my time :)


knife vsphere vm delete doesn't release the IP

Versions:

Version of knife-vsphere:
Version of chef:  Chef Development Kit Version: 1.2.22
chef-client version: 12.18.31

Platform Details

Version of vSphere:  6.5.0
Version of ESXi: ESXi 3

Scenario:
I am trying to delete and clone VM using the same static IP address; however , the IP address is not released
Please paste the command and output (run with -VV).
knife vsphere vm delete -VV host50 -f XXX_DEV --purge --vshost my-vcenter.nacr.com
INFO: Using configuration from C:/Users/yyyy/.chef/knife.rb
DEBUG: value for config item vsphere_host: onguard-vcenter.nacr.com
DEBUG: value for config item vsphere_path: /sdk
DEBUG: value for config item vsphere_port: 443
DEBUG: value for config item vsphere_nossl:
DEBUG: value for config item vsphere_user: yyyy
DEBUG: value for config item vsphere_pass: xxxx
DEBUG: value for config item vsphere_pass: xxxx
DEBUG: value for config item vsphere_pass: xxxx
DEBUG: value for config item vsphere_insecure: true
DEBUG: value for config item proxy_host:
DEBUG: value for config item proxy_port:
DEBUG: value for config item folder: XXX_DEV
DEBUG: value for config item vsphere_dc: Onguard Datacenter
Deleted virtual machine host50
DEBUG: Chef::HTTP calling Chef::HTTP::JSONInput#handle_request
DEBUG: Chef::HTTP calling Chef::HTTP::JSONOutput#handle_request
DEBUG: Chef::HTTP calling Chef::HTTP::CookieManager#handle_request
DEBUG: Chef::HTTP calling Chef::HTTP::Decompressor#handle_request
DEBUG: Chef::HTTP calling Chef::HTTP::Authenticator#handle_request
DEBUG: Signing the request as yyyy
DEBUG: Chef::HTTP calling Chef::HTTP::RemoteRequestID#handle_request
DEBUG: Chef::HTTP calling Chef::HTTP::ValidateContentLength#handle_request
DEBUG: Initiating GET to https://XXXX/organizations/xxxx/nodes/host50
DEBUG: ---- HTTP Request Header Data: ----
DEBUG: Accept: application/json
DEBUG: Accept-Encoding: gzip;q=1.0,deflate;q=0.6,identity;q=0.3
DEBUG: X-Ops-Server-API-Version: 1
DEBUG: X-OPS-SIGN: algorithm=sha1;version=1.1;
DEBUG: X-OPS-USERID: yyyy
DEBUG: X-OPS-TIMESTAMP: 2017-08-21T15:50:21Z
DEBUG: X-OPS-CONTENT-HASH: 2jmj7l5rSw0yVb/vlWAYkK/YBwk=
DEBUG: X-OPS-AUTHORIZATION-1: VKTq8ABsKOaB/VHw7PPY1XmcoWHN2xVNnVDOQD6zBRBuFkoHQnDAqMuIFPll
DEBUG: X-OPS-AUTHORIZATION-2: 3utr80RRbIrF7IR1dG6AizEupt/xB8Mp+z53R5JkzpbKJ+DhxtE25Kl4qQNL
DEBUG: X-OPS-AUTHORIZATION-3: VluyWBZJnP/mh1SBa5KCHF0NW+EY+Jp+NvK10UwFW5NBlXLqmqiOKveKkaPr
DEBUG: X-OPS-AUTHORIZATION-4: QQiPfA21yD0NCs6mB8Y1tICGGCmxdTJPAiXzfPe2pUQe0eoE0cDn0/v8DGo9
DEBUG: X-OPS-AUTHORIZATION-5: 5C4u2897NJfRKsfVaIYcfSPoE3MmuslJ/aIIeLncnpY9PLXhBBMjFIBHWjhs
DEBUG: X-OPS-AUTHORIZATION-6: MSzz2f2ETryUy2+IxmafVKI7s4xEu8Ji8gN87t8oLw==
DEBUG: HOST: abc.edf.com:443
DEBUG: X-REMOTE-REQUEST-ID: 4dbbc1bf-d9fc-460c-a1cf-22f744525af4
DEBUG: ---- End HTTP Request Header Data ----
DEBUG: ---- HTTP Status and Header Data: ----
DEBUG: HTTP 1.1 404 Object Not Found
DEBUG: server: ngx_openresty/<IP_ADDRESS>
DEBUG: date: Mon, 21 Aug 2017 15:50:22 GMT
DEBUG: content-length: 36
DEBUG: connection: close
DEBUG: x-ops-api-info: flavor=cs;version=12.0.0;oc_erchef=1.4.0
DEBUG: ---- End HTTP Status/Header Data ----
DEBUG: ---- HTTP Response Body ----
DEBUG: {"error":["node 'host50' not found"]}
DEBUG: ---- End HTTP Response Body -----
DEBUG: Chef::HTTP calling Chef::HTTP::ValidateContentLength#handle_response
DEBUG: Content-Length validated correctly.
DEBUG: Chef::HTTP calling Chef::HTTP::RemoteRequestID#handle_response
DEBUG: Chef::HTTP calling Chef::HTTP::Authenticator#handle_response
DEBUG: Chef::HTTP calling Chef::HTTP::Decompressor#handle_response
DEBUG: Chef::HTTP calling Chef::HTTP::CookieManager#handle_response
DEBUG: Chef::HTTP calling Chef::HTTP::JSONOutput#handle_response
DEBUG: Expected JSON response, but got content-type ''
DEBUG: Chef::HTTP calling Chef::HTTP::JSONInput#handle_response
INFO: HTTP Request Returned 404 Object Not Found: error
C:/opscode/chefdk/embedded/lib/ruby/2.3.0/net/http/response.rb:120:in `error!': 404 "Object Not Found" (Net::HTTPServerException)
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/lib/chef/http.rb:150:in `request'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/lib/chef/http.rb:115:in `get'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/lib/chef/node.rb:581:in `load'
        from C:/Users/yyyy/AppData/Local/chefdk/gem/ruby/2.3.0/gems/knife-vsphere-2.0.0/lib/chef/knife/vsphere_vm_delete.rb:42:in `destroy_item'
        from C:/Users/yyyy/AppData/Local/chefdk/gem/ruby/2.3.0/gems/knife-vsphere-2.0.0/lib/chef/knife/vsphere_vm_delete.rb:71:in `run'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/lib/chef/knife.rb:443:in `block in run_with_pretty_exceptions'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/lib/chef/local_mode.rb:44:in `with_server_connectivity'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/lib/chef/knife.rb:442:in `run_with_pretty_exceptions'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/lib/chef/knife.rb:219:in `run'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/lib/chef/application/knife.rb:156:in `run'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-12.18.31-universal-mingw32/bin/knife:25:in `<top (required)>'
        from C:/opscode/chefdk/bin/knife:68:in `load'
        from C:/opscode/chefdk/bin/knife:68:in `<main>'

Expected Result:
Expect IP is released so I can re-clone the same VM again
Actual Result:
From the vSphere Web Client, I see the host50 was deleted; however when I try to clone the same VM again,  IP didn't release from the delete VM , therefore the nclone VM doesn't have the same static IP.
Also, I notice the error:
{"error":["node 'host50' not found"]}
After running delete VM, do we have to run the command:
knife vsphere vm network delete VMNAME NICNAME
What are the actual sequences when we run the delete VM?

Hi, I'm not totally sure I understand this. I think the error you're getting is saying that the host50 node doesn't exist on the Chef server. If your node is called something different you can pass the -N flag with the name of the VM in Chef, or omit the --purge option so it just deletes the VM and doesn't talk to Chef at all.
IP -wise I don't expect that deleting the VM will release any IPs. All this command does is send an instruction to VMWare to delete the VM and, optionally, delete the node and client from Chef (that's the --purge flag)

I think I found the problem. It would take the VMWare more than 1 minute to release the IP address. So I have to wait for more than 1 minute before I can run the clone VM again.
Another issue I see is when running the "knfie vsphere vm delete VMNAME  -f DEV_XXX --purge --vhost myvhost-vcenter.com" I see the error:
ERROR: The object you are looking for could not be found
Response: node 'OEP225' not found
How do I get rid of this error?

Is that node in Chef? knife node show OEP225? When you use the --purge option, this gem will call Chef to ask it to remove the client and node records.
We store VMs by short name and Chef nodes by fqdn so when I use this command I normally do something like knife vsphere vm delete myvm --purge -N myvm.example.com

Which command can I use to list the Chef node associated with the VM short name?.

Try knife node list | grep SHORTNAME and see what pops up.

I run "knife node list" and couldn't find any Chef node associated with the VM short name

Was the node ever registered with Chef? It's quite possible to have a VM that never used Chef in which case you don't need the --purge option and that error will go away.
Despite this gem being a knife plugin most of the functions talk directly to vSphere. It's only the deletion and creation of VMs that needs to talk to Chef (and only if it's a Chef managed node)

If I take out the --purge then this is the error I got:
"Corresponding node and client for the Host225 server were not deleted and remain registered with the Chef Server."
Could we have the "knife vsphere vm list" also display the node name

Ah, I see... That message is misleading. It doesn't know if it's registered or not. It's just assuming that since you didn't ask to purge, it must still be registered.
vsphere doesn't have the information necessary to get the node name from Chef. I don't know of a way to link the two. We do it by convention. The node's short name is the name of the VM and the FQDN is in Chef.

when we use the knife vsphere vm clone --node-name MyNodeName so that when I delete VM with  --purge -N MyNodeName it should not error; however, I still see the error:
"ERROR: The object you are looking for could not be found
Response: node 'MyNodeName' not found"

When you cloned the VM, did you also have --bootstrap? If not, you'll get a VM that's not in Chef, and you'll continue to get the error if you try to --purge.
It's helpful to think of the VM management and Chef as separate things.


Update dependencies

I've changed the rpc dependency to the jcenter one, too. This update supports better presence persistence through discord restarts, especially on linux.

Closing this in favor of the rewrite.


feat: Add configMap to allow define parameters for SSH client

Since ssh-rsa has been deprecated since OpenSSH 8.2 but some git server are still using it like Azure DevOps, its necessary to bring back this feature.
Besisdes, since for internal networks, some people use not have too much concern about deprecated algorithms, this PR can give then hability to change the ssh client and make it compatible with their systems.
configMap example:
apiVersion: v1
kind: ConfigMap
metadata:
  name: argocd-image-updater-ssh-config
  namespace: argocd
data:
  config: |
    Host *
          PubkeyAcceptedAlgorithms +ssh-rsa
          HostkeyAlgorithms +ssh-rsa

This will be loaded in /app/.ssh/config as default user home path in image.

 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.


support for reading the value of a parameter from a file

Codecov Report

Merging #199 into master will increase coverage by <.01%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #199      +/-   ##
==========================================
+ Coverage   86.79%   86.79%   +<.01%
==========================================
  Files          50       50
  Lines        3051     3060       +9
==========================================
+ Hits         2648     2656       +8
- Misses        279      280       +1
  Partials      124      124

Impacted Files
Coverage Δ

openapi/rpc.go
97.82% <88.88%> (-2.18%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c92116c...437c831. Read the comment docs.


feat(learn): Add Tests to Issue Tracker Project, Remove InfoSec Tests

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Ref: #37185 - Proposed Definition Changes
Ref: #39435 - Remove InfoSoc Tests
Adds all tests to the Issue Tracker Project.  I have tested on my local install against my own project. Explicitly uses the fCC get-tests API to validate that all tests are both passing and have at least one assertion per test.
Note:  These tests assume some of the changes discussed in #39620 are implemented, so the current example project and boilerplate won't pass.  I'll get that pushed out soon.
If you want to grab my code, it's here:
https://github.com/SaintPeter/fcc-issue-tracker-project

This is blocked by #39024

Fixed merge conflict, and PR #39024 has been merged, so removing status: blocked label.

@SaintPeter You are going to need to rebase this to include the test related to the example project.

@Sky020

You are going to need to rebase this to include the test related to the example project.

I'm not sure that I understand.  Do you mean that I need to add a test the validates that they are not submitting the fCC sample project on the .rocks domain?  Looking at the code I don't see a test for the sample project at all, so this wouldn't be a change.

Oh, I just realized that the guy who said he was going to make these changes to the boilerplate flaked.  I'll try to figure out what needs doing and do it.  PRs to come soon.

Just to clarify: The test exists. It was just added after this PR was created:
https://github.com/freeCodeCamp/freeCodeCamp/blob/master/curriculum/challenges/english/06-quality-assurance/quality-assurance-projects/issue-tracker.md
So, you need to git pull --rebase upstream master to get it into this PR

@Sky020 There was a user responding to #39620 who said they would make the changes to the boilerplate and they did not.
I will update the boilerplate and sample project to match these updated user stories.  I think I'm going to ease back a bit on the user stories as well, to be more in line with the requirements of the Sudoku Solver and American/British Translator.
While I'm at it, I'll remove the user stories from the index.html.

Do we want to adjust this to add the new description, and so the test text uses you? or should we just keep this as a PR to add tests and make another for the issue of the user stories? I feel like we might as well add it here. What do you think @SaintPeter?

@moT01 I am planning to rework the boilerplate and sample project to match these updated tests.  I also will update these tests a bit. I should be able to do that in the next few days.

I'm fine with that for this one as well, but I think we should try and keep most of the rest of these back end PR's to the scope of the issues - They are difficult to review with PR's to boilerplates and demo-projects and the main repo.

They are difficult to review with PR's to boilerplates and demo-projects and the main repo.

@moT01 The issue is that all three are inter-related.  We need the updated tests, because that's the whole point.  We need the boilerplate to match the tests because when users start these projects they need something that is capable of matching and passing the tests on the live site.  Finally, we need the demo projects to pass because that's what we're testing the front-end tests against. It doesn't help that at the same time we're reformatting all of these and moving the user stories around.
I would love to do it all in one PR, but we've got 3 repos for good reasons.
If you have some suggestions on how to make these easier to review, I'm all ears.  Trust me, it's hard enough for me to get them up here and keep them up to date with the underlying repo-wide changes.

@nhcarrigan and I met and did a complete revamp of the user stories.  We changed the order, reworded many, and updated some of the requirements to be consistent with the other refactors we've done.  This makes for an overall better challenge.
We additionally rewrote the project description to be more standard and added the instructions block.
These tests can be checked against:
https://issue-tracker.pearlygatesoftware.com/
There is also a PR to update the Boilerplate, here:
https://github.com/freeCodeCamp/boilerplate-project-issuetracker/pull/13
And a PR to update the Demo Project, here:
https://github.com/freeCodeCamp/demo-projects/pull/34
Closes #39620

Ref: https://github.com/freeCodeCamp/freeCodeCamp/issues/39721

Thanks @nhcarrigan!


Remove fuse default_permission

What changes are proposed in this pull request?
Remove default_permission mount option.
Why are the changes needed?
Alluxio fuse needs comprehensive security story. Before that story come out, it does not make sense to enable default_permissions
Does this PR introduce any user facing changes?
NA

@ssz1997 can you help test this out?

By removing the flag, the owner/group of the mounted s3 dir is still 65534, and all users (root, alluxio) is able to access the dir and its files.

alluxio-bot, merge this please


[Feat] Feature Request: Notification System UI

Description
We need to design and implement a user-friendly UI for the notification system. This UI will be responsible for displaying alerts, messages, and updates to users in a clear and accessible manner across the application.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@mdazfar2  Please assign me this issue usder the label of gssoc and hactoberfest sir


Added C implementation of Dijkstra's shortest path.

Changed folder name to better describe the algorithm.
Added some more tags.

First time forking/branching/making a pull request.  I think I did it all correctly.  Let me know if it worked properly or if I could make any extra changes.

LGTM


allow double quote in escape sequences

Hello! 👋
I noticed this edge case in the escape sequences - if you're trying to capture " in a query and it isn't a named node you'll need to surround it in quotes but escape it with a backslash. tree-sitter-cli accepts that as a valid query but it trips up the parser here.

Wild. I literally just hit this today, came to report an issue, and low and behold found a PR filed 21 hours ago. 😂

<EMAIL_ADDRESS>


Initialize initializers at build time for GraalVM code removal

In order to be effective for GraalVM native code removal, the enabled flag introduced in 0b6427e95169c28d171f47694ffcebdeb8c9fdb7 should also defines build time init for initializer classes.
This can be done with a META-INF/native-image/org.springframework.boot/autoconfigure/native-image.properties file defining:
Args = --initialize-at-build-time=com.example.Initializer1,com.example.Initializer2

Added in 87bdff9


[move source language] Thread through source map

Motivation

Source map was already present from shared compiler. Threaded the information through into the source language CompiledUnit

Test plan
cargo test

@bors-libra r=wrwg

:pushpin: Commit 60bf523 has been approved by wrwg

:hourglass: Testing commit 60bf523c2bf62bc60149af8d1c02a5dd611502f3 with merge e9604575f502937721950942b2cff5e851729489...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: wrwg
Pushing e9604575f502937721950942b2cff5e851729489 to master...


[PAPI-86] Refactor Liquidity Pool Filtering and Pagination

market.stakingTokenId is no longer nullable

simplifies LP filtering and current implementation is inserting 0 instead of null anyways

liquidity pool filter changes at GET /liquidity-pools with cursor pagination
Adds indexed Name column to pool_liquidity, set during creation, for query simplification and improvements.

still need to add some tests
Related to: https://github.com/Opdex/opdex-db-scripts/pull/23

GetLiquidityPoolsWithFilter tests will be done as part of the assembler / response refactoring of pools right after this. Doing it now will be wasted time mocking pool snapshot details to turn around and switch to summary details.


Review timeout policy on SwellRT callbacks

Opened as answer to https://github.com/P2Pvalue/swellrt/issues/93

Do those timeout rise an error?

During invocation of createModel() or openModel() two networks operations occurs:

First a http get request. this can raise a timeout exception captured in the failure callback of the createModel() or openModel().

Secondly, wavelet's initial data must be delivered to the websocket. In this case, a time out exception on the websocket will be deliver through the global event handler as a SwellRT.events.FATAL_EXCEPTION.

This issue's intent is to provide consistency regarding this behaviour.

Caused by the currently removed request access operation


Preserve type keyword for import types

I am interested in helping provide a fix!
Yes
Which generators are impacted?

[X] All
[ ] Angular
[ ] HTML
[ ] Qwik
[ ] React
[ ] React-Native
[ ] Solid
[ ] Stencil
[ ] Svelte
[ ] Vue
[ ] Web components

Reproduction case
https://mitosis.builder.io/?outputTab=M4NwpgNgLmQ%3D&code=JYWwDg9gTgLgBAbzgVwM4FMAi6BmBDZAGxgAUoIxU4BfOHckOAcgAEAjZYQgE3SgDpgEAPQhgMCKmComAKFCRYcGAE8w6RHABCyGBIB2ZClVr0IjJv2Gr1c%2BeGjxLw1KsLo76AB6L4vfETwOMj6AMYwQvraugYAFGDklABc0XoQhomoAJSIsnAoGNgBxEaUADw6aRnGAHyxCHn5cKgAFnjqKUyoAI7IeFAeADSN1FmyjQMwyFBRsY35ZRxVcKGEeKioALwICcb8re3otDXzTTuZ%2FKEtXNwD%2BtSnZcJLBif5Yw9AA
Expected Behaviour
source
import { useDefaultProps } from<EMAIL_ADDRESS>import type { ButtonProps } from './type'

should preserve type keywords type
vue
import { defineComponent } from "vue";
import type { ButtonProps } from "./type";

react
import * as React from "react";
import type { ButtonProps } from "./type";

Actual Behaviour
vue
import { defineComponent } from "vue";
import { ButtonProps } from "./type";

react
import * as React from "react";
import { ButtonProps } from "./type";

Additional Information
Changed by versions
<EMAIL_ADDRESS>"^0.0.96",
<EMAIL_ADDRESS>"^0.0.55",
<EMAIL_ADDRESS>"^0.0.97",
<EMAIL_ADDRESS>"^0.0.56",

It is available before Feature: preserve types in all code blocks #1085
available link: https://mitosis-fiddle-git-fork-matthewsimo-fix-1047-4f5fa0-builder-io.vercel.app

Check importDeclaration: https://babeljs.io/docs/babel-types#importdeclaration
importKind: "type" | "typeof" | "value" (default: null, excluded from builder function)
Add importKind to store import type
export interface MitosisImport {
  path: string;
  imports: {
    [key: string]: string | undefined;
  };
+  importKind?: 'type' | 'typeof' | 'value' | null;
}

More detail check PR https://github.com/BuilderIO/mitosis/pull/1139


Issue#109. [Core] use entry processors for all operations on documents

Exceptions handling
DocumentRemoveProcessor return document accessor

well done!


Add FSDP docs

What does this PR do?
Adds docs for #7487, with usage instructions for this V1 case
cc @shuyingsunshine21 @ananthsub
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Test is failing atm, but let's keep it like that cos I want @ananthsub and @shuyingsunshine21 to take a look :)

lgtm overall, thanks @SeanNaren

Re-running the jobs!


Add equals and hashCode to BaseTableInfo subclasses

This fixes #564

LGTM

but I think that defining equals and hashCode also for abstract classes is a good practice in general

I would actually say the opposite. if you define it in the base class and override it only in some of the sub-classes then you can have a symmetric issues (B equals C but not C equals B) and can even get weird transitive issues. This is not an issue if you make the base class methods final.
In your case you do override all the sub-classes equals & hashCode methods, so right now we don't have an issue (other than the base-class equals is never used) but more likely to get broken in the future
if someone is not aware of it.


Allow to specify rpchost in sentinel config

This allows to specify rpchost in sentinel.conf
Needed when run inside docker.

I think we should default this similar to the rpcport argument, right (as opposed to the network)?

@nmarley The rpchost is the same for all networks, it's always localhost by default. Or did I misunderstand you?

I mean it can be extracted and defaulted just like the rpcport is currently, e.g. something like https://github.com/nmarley/sentinel/commit/8f5829c6623d302cf5d27629813fe7b9c036257d

Ah now I understand. That's actually how I implemented it in the beginning, but then realized that the rpchost is not specified in dash.conf. The dash.conf only contains the rpcport and rpcbind, but nothing like the hostname.
That's why I decided to not put this configuration value into dash.conf but into sentinel.conf instead.

Ah, that makes more sense. It's always localhost to the dashd process.

If I get it right rpchost is similar to -rpcconnect in dash-cli.

I have no preferences, rpchost looks a bit more common while rpcconnect is what it is in dash-cli (for whatever reason). I'm ok with both :)

Wherever I've seen configuration parameters for BTC/Dash RPC hosts, it was always called rpchost, rpc_host, host, ... (for example in multifaucet). dash-cli was the first one I encoutered a different naming. So I'd prefer to stick with rpchost. If you're ok with that, I'll merge after a :+1: :)

LGTM, :shipit:

Yep, in Bitcoin/Dash is most likely a legacy thing which came from connect/port for p2p by adding rpc prefix instead of following common practice...


chore(user-agent): add runtime info

Description
Add runtime info such as OS and .NET version to the User-Agent header.
Links
Testing
Checklist
Ensure the following things have been met before requesting a review:

[x] Follows all project developer guide and coding standards.
[x] Tests have been written for the change, when applicable.
[x] Confidential information (credentials, auth tokens, etc...) is not included.

I think your commits arent signed but I guess you can bypass this for now and just squash merge. What do we think @factset/enterprise-sdk


Class names in class browser should be sorted alphabetically

Webside connected to egg is showing classes without sorting them:


fix(artist): Fix jump bug

The type of this PR is: Fix
Description
This fixes an issue that was observed here, where scrolling after jump was blowing up scrolling on the page.
The issue has to do with useEffect dependencies. If we want something to execute once and only once, we shouldn't pass dependencies. Ignore that bloody react-hooks linter when it doesn't make sense 😄.

Pushed up the suggestion here for posterity, but closing this PR now.


[Compute] az vmss create: Add new parameter --max-surge to support enabling rolling upgrade policy max surge

Related command

az vmss create
Description

Add max surge in vmss rolling upgrade policy configurability
close https://github.com/Azure/azure-cli/issues/24846
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute


Question about boundary condition

Hi, I was going through your paper on Arxiv and trying out the solver. It's a great work.
I was wondering how do you deal with boundary conditions in PDE as there's only initial(terminal) condition s specified in the solver.
Thanks a lot.

Thanks for your interests in our work. The current paper and codes only consider the situation where the domain is R^d, without any boundary. So there are no space boundary conditions specified in the solver. There are BSDEs corresponding to PDEs with space boundary conditions and our idea should also be applicable. But it is beyond the scope of the current paper and solver.
I am open to discussion if you can provide more details of your problems.

Thanks for the reply.  I'm in the field of Materials Science and we were using the conventional method to solve Allen Cahn equation on R^3 domain with periodic space boundary mostly.  I was wondering as the periodic boundary is just a way to represent the infinite space, do I need to make modifications to your solver?


oUF_Neav: Targetframe debuffs

Hi again (sorry for the many reports),
are the following two options possible? (I have nothing found in the config or via rightclick on the target)

Show only my debuffs (was a standard setting in the Blizzard UI, nothing there yet, they have killed it)
Put the debuffs on top of the target frame as in the screenshot shown

Only if it's easy possible :)

Done, added to pull request. #180

Very cool, many thx to you guys!

Work fine now!


[SPARK-49399] Add pi-scala.yaml and pyspark-pi.yaml

What changes were proposed in this pull request?

This adds example Spark applications spec for Spark scala image, and adds an example for pyspark
Why are the changes needed?

As of now, Spark docker images are built with types of all / python / scala / r. While existing examples cover usage of Python image, this PR sets up examples for another image(scala). As R is being deprecated, operator now serves example for Python / scala images to avoid confusion.
Main example uses default image 4.0.0-preview1-scala2.13-java17-python3-ubuntu, 4.0.0-preview1-python3, 4.0.0-preview1 which includes Python:
examples/pi-scala.yaml uses scala image : 4.0.0-preview1-scala2.13-java17-ubuntu, 4.0.0-preview1-scala
Does this PR introduce any user-facing change?

No - new release
How was this patch tested?

CIs and integration tests
Was this patch authored or co-authored using generative AI tooling?

No.

If you are interested, could you add Apache Spark 3.5.2 examples, too?

Also, cc @TQJADE since you added Apache Spark 3.5.2 test case.
https://github.com/apache/spark-kubernetes-operator/blob/87920b57b4765f47028d6ec8c7b10d2eb89f1eef/tests/e2e/spark-versions/chainsaw-test.yaml#L35


Stamina regen while swimming

You can regen stamine while swimming if you use the "autowalk" keybind(Default Q), i guess stamina regen checks for WASD keys but not actual movement

Thank you. I will look into this issue.

if you dive into water activate the dive feature and just swim back to shore you just have infinite stamina on land
that is till you type in chat with /s
then its back to normal
might be because you are supposed to be able to regain stamina standing still in water
this was in version 101,,, havent updated server yet to 102

You can regen stamine while swimming if you use the "autowalk" keybind(Default Q), i guess stamina regen checks for WASD keys but not actual movement

I have implemented a fix for this that will be released in version 1.0.3. The resting state will now be triggered when the players velocity is below a certain threshold. This mean that large waves (like those during thunderstorms) will also be able to hinder the players ability to regain stamina in the water.

I have released v1.0.3 that includes the fix for this bug.


prepare config management

Let's move the CAPA app to the new config management.
There is going to be a separate PR in the config repo to add the Public key since it's not on there yet.
Checklist

[ ] Update changelog in CHANGELOG.md.

Only fixed this one small thing ssoPublicKey: 'ssh-rsa AAASbn3n fake@giantswarm', last back-tick was missing


Pythonise bootstrap_fraction_sauter() function in physics.f90 and integrate refactored PlasmaProfiles with it

In GitLab by @ym1906  on Mar 31, 2023, 09:58
The function bootstrap_fraction_sauter() calculates its own version of t and ne profiles and integrates them. For consistency and efficiency it should use the refactored Plasma Profiles which already hold this data.
Tasks:

Convert the function to python.
Determine whether a unique profile is required or whether this can be achieved with existing profiles

Note to future reader: this still needs doing!

Related issue #3263
Can hopefully do this with the refactor and docs


Date ranges

Many conferences (and thus, conference proceedings) have date ranges instead of dates, and I'd like to specify the start and end date of a conference. One option would be to follow the ISO 8601 standard (or at least what Wikipedia claims it is), and use <start-date>/<end-date>, like so:
[...]
  parent:
    type: proceedings
    publisher: IEEE
    date: '2022-05-16/2022-05-19'
    title:
      value: 2022 22nd IEEE International Symposium on Cluster, Cloud and Internet
        Computing (CCGrid)
      short: 2022 22nd IEEE Int. Symp. Cluster, Cloud and Internet Comput. (CCGrid)
    location: Taormina, Italy

Could there be an option for this?

Or subfields?
So normally:
date: 2022-05-16

Range:
date:
  start: 2022-05-16
  end: 2022-05-19

Or since page-range exists, it could also be called date-range. But that's probably more confusing.


Investigate slowness of DBS3Upload component under heavy load

Impact of the new feature
Improve throughput of DBS3Upload component.
Is your feature request related to a problem? Please describe.
Due to recent incident with resource pool meltdown we found slowness of DBS3Upload components in multiple agents.  In particular, one of the agents had 10K open blocks and individual block took 3-5 min to resolve while after restart of DBS3Upload component it processed such blocks in 3-5 sec. We should investigate the cause of this issue. So far preliminary suspects are the following:

slowness of SQL queries
network issue with upstream DBS server
etc.

Describe the solution you'd like
Understand and resolve slowness of DBS3Upload component
Describe alternatives you've considered
Do nothing and leave with it
Additional context
Add any other context or screenshots about the feature request here.

Some additional information. From submit3 agent using DBS3Upload/ComponentLog I see the following:
2023-10-08 20:25:40,210:140489754543872:INFO:DBSUploadPoller:Found 12432 open blocks.
2023-10-08 20:25:40,230:140489754543872:INFO:DBSUploadPoller:Loaded 0 blocks from the database.
2023-10-09 03:15:36,461:140489754543872:INFO:DBSUploadPoller:Queueing block for insertion: /ggH_Hto2Sto4B_MH-125-MS-40-cta
uS-10_PTH-80_TuneCP5_13p6TeV_powheg-pythia8/Run3Summer22EEMiniAODv4-130X_mcRun3_2022_realistic_postEE_v6-v2/MINIAODSIM#cd5
90279-b9f2-473a-9c82-6a5e27892ce2
2023-10-09 03:15:36,463:140489754543872:INFO:DBSUploadPoller:Queueing block for insertion: /ggH_Hto2Sto4B_MH-125-MS-55-cta
uS-0p1_PTH-80_TuneCP5_13p6TeV_powheg-pythia8/Run3Summer22MiniAODv4-130X_mcRun3_2022_realistic_v5-v2/MINIAODSIM#bfbd7c0f-5b
38-4046-810f-ea2ceae4fdc2
...

and then over the sudden we have the following conditions:
2023-10-10 11:29:05,933:140605748287232:INFO:DBSUploadPoller:Found 11651 open blocks.
2023-10-10 11:29:14,432:140605748287232:INFO:DBSUploadPoller:Loaded 11651 blocks from the database.
2023-10-10 11:34:00,798:140605748287232:INFO:DBSUploadPoller:Have 1 files for block /ggH_Hto2Sto4B_MH-125-MS-23-ctauS-10_P
TH-80_TuneCP5_13p6TeV_powheg-pythia8/Run3Summer22MiniAODv4-130X_mcRun3_2022_realistic_v5-v2/MINIAODSIM#63a59523-be7c-47f0-
9191-81cf60d743bc
2023-10-10 11:39:05,643:140605748287232:INFO:DBSUploadPoller:Have 4 files for block /JPsito2Mu_JPsiFilter_2MuFilter_TuneCP
5_13p6TeV_pythia8/Run3Summer22EEMiniAODv4-130X_mcRun3_2022_realistic_postEE_v6-v2/MINIAODSIM#5c5dbdbe-978d-4da9-a8fd-25704
6beffc0
...

The obvious questions are:

why do we have O(10K) open blocks in first case and loaded 0 blocks from database, while
in second case we have approximately the same amount of open blocks but the same amount of loaded blocks from database
Which conditions trigger to load at particular cycle all blocks from database? Do we need to constrain this amount of blocks to smaller number? Interestedly, once we load O(10K) blocks from database (which one) then each block record takes about 405 minutes to process. Looking further into top profile I found that we
have node saturation:

KiB Mem : 11507057+total, 16029780 free, 37839524 used, 61201268 buff/cache
KiB Swap: 33553404 total, 31724356 free,  1829048 used. 76602824 avail Mem

    PID USER      PR  NI    VIRT    RES    SHR S  %CPU %MEM     TIME+ COMMAND
  10391 cmsdata+  20   0   12.3g   4.0g   7136 S 421.2  3.6  85706:24 mariadbd
  10083 cmsdata+  20   0 7878632 179524   4692 S 164.2  0.2  72918:24 beam.smp
...

where we have quite some load on MariaDB and non-negligible RAM utilization. It seems to me that MariaDB consumes lots of node resources and slows down the entire process.

Just an update to this issue - because it's easier to lose this content in mattermost.
The FNAL agent submit5 was running only the DBS3Upload component yesterday (all the other components had been stopped) and it took around 15h to go through 10k blocks, as can be seen in this log snippet
2023-10-12 05:59:16,105:140059579315968:INFO:DBSUploadPoller:Starting the DBSUpload Polling Cycle
2023-10-12 05:59:40,377:140059579315968:INFO:DBSUploadPoller:Found 9656 open blocks.
2023-10-12 05:59:44,384:140059579315968:INFO:DBSUploadPoller:Loaded 9656 blocks from the database.
2023-10-12 05:59:49,409:140059579315968:INFO:DBSUploadPoller:Have 1 files for block /WJetsToLNu_HT-200To400_TuneCP5_13TeV-madgraphMLM-pythia8/RunIISummer20UL17NanoAODv9-106X_mc2017_realistic_v9_ext1-v1/NANOAODSIM#4f4b3037-e70b-4c30-84a9-d
aa1f714c01b
...
2023-10-12 14:39:34,487:140059579315968:INFO:DBSUploadPoller:Have 1 files for block /ZZto4L_TuneCP5_13p6TeV_powheg-pythia8/Run3Summer22EENanoAODv12-130X_mcRun3_2022_realistic_postEE_v6-v2/NANOAODSIM#5c7e2a64-6047-4ccb-8cdd-69eb20484c64
2023-10-12 14:50:30,128:140059579315968:INFO:DBSUploadPoller:Queueing block for insertion: /WJetsToLNu_HT-200To400_TuneCP5_13TeV-madgraphMLM-pythia8/RunIISummer20UL17NanoAODv9-106X_mc2017_realistic_v9_ext1-v1/NANOAODSIM#4f4b3037-e70b-4c30
-84a9-daa1f714c01b
...
2023-10-12 14:51:16,744:140059763263296:INFO:DBSUploadPoller:About to call insert block for: /TTtoBsto2Tau_BsFilter_TauTauFilter_TuneCP5_13p6TeV_pythia8-evtgen/Run3Summer22EEMiniAODv4-130X_mcRun3_2022_realistic_postEE_v6-v2/MINIAODSIM#d70e455e-7893-4cde-9e0b-8fbd4f4e3b32
2023-10-12 15:23:15,119:140059763263296:INFO:DBSUploadPoller:About to call insert block for: /ZZZ_TuneCP5_13p6TeV_amcatnlo-pythia8/Run3Summer22EENanoAODv12-130X_mcRun3_2022_realistic_postEE_v6-v2/NANOAODSIM#8e01f1c1-42bc-4229-839e-fb97b2b8811f
...
2023-10-12 21:53:10,613:140059579315968:INFO:BaseWorkerThread:DBSUploadPoller took 57234.507 secs to execute
2023-10-12 21:54:51,853:140059579315968:INFO:DBSUploadPoller:Starting the DBSUpload Polling Cycle
2023-10-12 21:55:02,432:140059579315968:INFO:DBSUploadPoller:Found 13577 open blocks.

we can make a few observations from this log:

the component takes < 30 secs to retrieve the open blocks from MariaDB
it took around 8h40min to load files and associate them to the relevant blocks
it takes around 10min to queue these blocks in a multiprocess call to insert them into DBS
then in this multiprocess execution of the dbs insert calls, it staled the logs for 6h30min (waiting for DBS server to reply to all of the insert requests?)

In addition, after inserting them into DBS, they should have been marked as closed and be gone from the next cycle. Yet, the component says Found 13577 open blocks. I checked a couple of block names and it looks like those are actually new blocks (they are not listed in the previous cycle log).

Alan, thanks for update. If I take your numbers here is my observations:

10K blocks in 6h30min for multiprocess execution is equal 21630 seconds
I have no idea what multiprocess execution means, i.e. I do not know if it is sequential process(es) or not, but if I divide total time to 10K blocks it gives us around 20sec on average to insert the block
if I look at DBS writer logs from vocms0750, here is my observation (please scroll horizontally to see how DBS log entry looks like, at the end it shows time of the request):

rg POST /cephfs/product/dbs-logs/dbs2go-global-w-*.log-20231012 | grep bulkblocks | grep prodsrv | head

/cephfs/product/dbs-logs/dbs2go-global-w-5c65bcc968-x82dd.log-20231012:[2023-10-12 13:32:12.407878011 +0000 UTC m=+222.670200331] HTTP/1.1 200 POST /dbs/prod/global/DBSWriter/bulkblocks [data: 1104 in 2 out] [remoteAddr: <IP_ADDRESS>:59248] [X-Forwarded-For: <IP_ADDRESS>] [X-Forwarded-Host: cmsweb-k8s-prodsrv.cern.ch] [auth: no-TLS cipher-none "/DC=ch/DC=cern/OU=computers/CN=tier0/vocms001.cern.ch"<EMAIL_ADDRESS>X509Proxy] [ref: "-" "DBSClient/Unknown/"] [req: 191.418241ms proxy-resp: 0]
/cephfs/product/dbs-logs/dbs2go-global-w-5c65bcc968-x82dd.log-20231012:[2023-10-12 13:32:18.205551097 +0000 UTC m=+228.467873417] HTTP/1.1 200 POST /dbs/prod/global/DBSWriter/bulkblocks [data: 42804 in 2 out] [remoteAddr: <IP_ADDRESS>:59250] [X-Forwarded-For: <IP_ADDRESS>] [X-Forwarded-Host: cmsweb-k8s-prodsrv.cern.ch] [auth: no-TLS cipher-none "/DC=ch/DC=cern/OU=computers/CN=tier0/vocms001.cern.ch"<EMAIL_ADDRESS>X509Proxy] [ref: "-" "DBSClient/Unknown/"] [req: 5.958781216s proxy-resp: 0]
/cephfs/product/dbs-logs/dbs2go-global-w-5c65bcc968-x6mmd.log-20231012:[2023-10-12 06:50:47.900312536 +0000 UTC m=+18.839267176] HTTP/1.1 200 POST /dbs/prod/global/DBSWriter/bulkblocks [data: 836 in 2 out] [remoteAddr: <IP_ADDRESS>:41138] [X-Forwarded-For: <IP_ADDRESS>] [X-Forwarded-Host: cmsweb-k8s-prodsrv.cern.ch] [auth: no-TLS cipher-none "/DC=ch/DC=cern/OU=computers/CN=tier0/vocms001.cern.ch"<EMAIL_ADDRESS>X509Proxy] [ref: "-" "DBSClient/Unknown/"] [req: 144.312138ms proxy-resp: 0]
/cephfs/product/dbs-logs/dbs2go-global-w-5c65bcc968-x6mmd.log-20231012:[2023-10-12 06:50:47.99190786 +0000 UTC m=+18.930862538] HTTP/1.1 200 POST /dbs/prod/global/DBSWriter/bulkblocks [data: 1184 in 2 out] [remoteAddr: <IP_ADDRESS>:41140] [X-Forwarded-For: <IP_ADDRESS>] [X-Forwarded-Host: cmsweb-k8s-prodsrv.cern.ch] [auth: no-TLS cipher-none "/DC=ch/DC=cern/OU=computers/CN=tier0/vocms001.cern.ch"<EMAIL_ADDRESS>X509Proxy] [ref: "-" "DBSClient/Unknown/"] [req: 233.80814ms proxy-resp: 0]
/cephfs/product/dbs-logs/dbs2go-global-w-5c65bcc968-x6mmd.log-20231012:[2023-10-12 06:52:29.09537099 +0000 UTC m=+120.034325638] HTTP/1.1 200 POST /dbs/prod/global/DBSWriter/bulkblocks [data: 1041 in 2 out] [remoteAddr: <IP_ADDRESS>:41146] [X-Forwarded-For: <IP_ADDRESS>] [X-Forwarded-Host: cmsweb-k8s-prodsrv.cern.ch] [auth: no-TLS cipher-none "/DC=ch/DC=cern/OU=computers/CN=tier0/vocms001.cern.ch"<EMAIL_ADDRESS>X509Proxy] [ref: "-" "DBSClient/Unknown/"] [req: 115.061983ms proxy-resp: 0]
/cephfs/product/dbs-logs/dbs2go-global-w-5c65bcc968-x6mmd.log-20231012:[2023-10-12 06:52:29.176204407 +0000 UTC m=+120.115159068] HTTP/1.1 200 POST /dbs/prod/global/DBSWriter/bulkblocks [data: 1093 in 2 out] [remoteAddr: <IP_ADDRESS>:41144] [X-Forwarded-For: <IP_ADDRESS>] [X-Forwarded-Host: cmsweb-k8s-prodsrv.cern.ch] [auth: no-TLS cipher-none "/DC=ch/DC=cern/OU=computers/CN=tier0/vocms001.cern.ch"<EMAIL_ADDRESS>X509Proxy] [ref: "-" "DBSClient/Unknown/"] [req: 195.894124ms proxy-resp: 0]
/cephfs/product/dbs-logs/dbs2go-global-w-5c65bcc968-x6mmd.log-20231012:[2023-10-12 06:53:56.640501486 +0000 UTC m=+207.579456192] HTTP/1.1 200 POST /dbs/prod/global/DBSWriter/bulkblocks [data: 736 in 2 out] [remoteAddr: <IP_ADDRESS>:48654] [X-Forwarded-For: <IP_ADDRESS>] [X-Forwarded-Host: cmsweb-k8s-prodsrv.cern.ch] [auth: no-TLS cipher-none "/DC=ch/DC=cern/OU=computers/CN=tier0/vocms001.cern.ch"<EMAIL_ADDRESS>X509Proxy] [ref: "-" "DBSClient/Unknown/"] [req: 58.04745ms proxy-resp: 0]
/cephfs/product/dbs-logs/dbs2go-global-w-5c65bcc968-x6mmd.log-20231012:[2023-10-12 06:54:10.265695032 +0000 UTC m=+221.204649688] HTTP/1.1 200 POST /dbs/prod/global/DBSWriter/bulkblocks [data: 965 in 2 out] [remoteAddr: <IP_ADDRESS>:41150] [X-Forwarded-For: <IP_ADDRESS>] [X-Forwarded-Host: cmsweb-k8s-prodsrv.cern.ch] [auth: no-TLS cipher-none "/DC=ch/DC=cern/OU=computers/CN=tier0/vocms001.cern.ch"<EMAIL_ADDRESS>X509Proxy] [ref: "-" "DBSClient/Unknown/"] [req: 77.922276ms proxy-resp: 0]

Then, if I filter to extract request time which DBS measures, I see the following
rg POST /cephfs/product/dbs-logs/dbs2go-global-w-*.log-20231012 | grep bulkblocks | grep prodsrv | head | awk '{print $31}'
5.593826514s
187.177023ms
191.418241ms
5.958781216s
144.312138ms
233.80814ms
115.061983ms
195.894124ms
58.04745ms
77.922276ms

So, the bulkblocks DBS API is taking far less (from the above printout 9 out of 10 calls took less than a second) according to DBS log. Here I do not know which DN is used by WMAgent, I also assumes WMAgent talks to cmsweb-k8s-prodsrv.cern.ch backend which is behind cmsweb-prod frontend. Sai that:

there is some discrepancy in terms of time spend in DBS request, according to WMA requests take on average 20sec while according to DBS log it only take less than a second (again out of first 5 entries)
20 sec is not that big either in terms of insert call since it really depends on complexity of underlying payload
therefore, 6h30min for DBS insertion for 10K blocks (or so) does not seem crazy.

But I have no idea what does it means that it took around 8h40min to load files and associate them to the relevant blocks. Where the files comes from, what this procedure looks like, how association is done?

As mentioned in this PR comment:
https://github.com/dmwm/WMCore/pull/11768#issuecomment-1763060939
most of the component latency is apparently coming from the RDMBS data update. It needs to be further analyzed in the future though and we might decide to make some larger refactoring of the component.
For now, we agreed to merge https://github.com/dmwm/WMCore/pull/11768, which is meant to improve the component logging and give us a better idea of critical sections of the component.


Enables packaging

Enable packaging for the project -> name is decm05

Check https://github.com/python-versioneer/python-versioneer to use versionning

See warning message:
main-ci-action / main-ci-action
The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

#60


chore(gemspec): don't include Gemfile.lock

I was surprised to find dozens of security vulnerability warnings for my application. The offending dependencies were: git, rack, nokogiri, stringio, and rdoc. The offending versions didn't make sense because my application is kept up-to-date and we're running the latest known-security versions of these, and we don't even use the git gem.
Upon closer inspection I realised that these were all coming from bundle/gems/rubyXL-3.4.22/Gemfile.lock.
Now, this is a false positive with our vulnerability scanner, but it expects that there's only one Gemfile.lock file for an application which is a reasonable assumption.
There's no reason to ship Gemfile.lock. Applications won't use it. I don't think there's even a reason to have Gemfile.lock committed to the repo here at all, but that's up to you.
Of 237 installed gems, only two include Gemfile.lock
$ find /usr/local/bundle/ -name Gemfile.lock
/usr/local/bundle/gems/os-1.1.4/Gemfile.lock
/usr/local/bundle/gems/rubyXL-3.4.22/Gemfile.lock

As you see, that remained from the very very original creator 13 years ago. I have no problem removing it.


Always getting Status: Hash sum exists. Skip download...

I seem to get this result when trying to download a file with the same name as one previously downloaded.  However I know that the file has changed, and the sha1sum does not match what is in the hash db.  The output even indicates the file size is different, but displays an incorrect sha1sum than if I download the file and run sha1sum filename

Hi,
Please read and apply SUPPORT.md page
Thanks

Closed due to inactivity


fix setting loglevel for the operand

Replacing https://github.com/openshift/cluster-kube-apiserver-operator/pull/284 as he is out.

/lgtm

/retest

/retest

/retest

/retest


Prefix ocaml-jupyter-opam-genspec with opam exec --

I don't know whether it is specific to my installation, but at first the original command would fail with:
$ ocaml-jupyter-opam-genspec
-bash: ocaml-jupyter-opam-genspec: command not found

ocaml-jupyter-opam-genspec is installed into ~/.opam directory. You need to run eval $(opam env) for updating $PATH.


Test 3.0

/close
superseded by https://github.com/openshift/prometheus/pull/237

/reopen
for tests

/testwith openshift/prometheus/master/e2e-agnostic-cmo openshift/prometheus-operator#321
/testwith openshift/prometheus/master/e2e-aws-ovn openshift/prometheus-operator#321
/testwith openshift/prometheus/master/e2e-aws-ovn-upgrade openshift/prometheus-operator#321

/testwith openshift/prometheus/master/e2e-agnostic-cmo openshift/prometheus-operator#321
/testwith openshift/prometheus/master/e2e-aws-ovn openshift/prometheus-operator#321
/testwith openshift/prometheus/master/e2e-aws-ovn-upgrade openshift/prometheus-operator#321

/testwith openshift/prometheus/master/e2e-agnostic-cmo openshift/prometheus-operator#321 openshift/thanos#154
/testwith openshift/prometheus/master/e2e-aws-ovn openshift/prometheus-operator#321 openshift/thanos#154
/testwith openshift/prometheus/master/e2e-aws-ovn-upgrade openshift/prometheus-operator#321 openshift/thanos#154

/retest-required

/testwith openshift/prometheus/master/e2e-agnostic-cmo openshift/cluster-monitoring-operator#2544
/testwith openshift/prometheus/master/e2e-aws-ovn openshift/cluster-monitoring-operator#2544
/testwith openshift/prometheus/master/e2e-aws-ovn-upgrade openshift/cluster-monitoring-operator#2544


move kmlExport ADE APIs to impexp core module

This PR moves all KmlExport-related APIs for ADEs into the impexp-core module. Thirdparty applications now just need the dependency of the core module to load ADE extension modules, when they implement the ADEKmlExportExtension interface.

Thanks for proposing this change, which I fully support. The only thing I'm not sure about is moving Queries.java into the org.citydb.ade.kmlExporter package. This class contains all the database queries required for the KML/COLLADA/glTF export and thus is a core component of the impexp-kml-collada-module. I therefore think the previous package  org.citydb.modules.kml.database is more appropriate for this class.
I understand that we run into circular dependencies if we just keep Queries.java in its old place but move all other classes into the impexp-core module. Does an ADE implemention really need access to all the queries in Queries.java, or could we just split the class and move some general helper queries such as getSurfaceGeometries or getImplicitGeometryNullColumns into the core module?

Good point. You are right, the Queries.java should be kept in the previous package org.citydb.modules.kml.database.
In fact, a number of methods in Queries.java are required by ADE implementation. We can abstract these methods into a dedicated helper interface which can be placed in the core module. see this proposal c6f918118bcea9fa7265f0185076fc6ab7319516

Approved by @clausnagel  offline. Merge it.


Poetry publish should support dev version

Right now we have workflows that allow to release the repo and publish to PyPI.
We would like to have something in between (Probably expanding the publish action/workflow) that allows to publish as a SNAPSHOT/dev version.

yes

I will create a template workflow for it