Can't run the action because: HttpError: Not Found

I'm getting the following error running this action:
HttpError: Not Found
     at file:///home/runner/work/_actions/CatChen/eslint-suggestion-action/v2/webpack:/eslint-suggestion-action/node_modules/@octokit/request/dist-node/index.js:86:1

Full debug log of the action run follows:
Debug log

2024-02-18T14:36:45.8405551Z ##[debug]Evaluating condition for step: 'Run CatChen/eslint-suggestion-action@v2'
2024-02-18T14:36:45.8408183Z ##[debug]Evaluating: success()
2024-02-18T14:36:45.8408860Z ##[debug]Evaluating success:
2024-02-18T14:36:45.8409613Z ##[debug]=> true
2024-02-18T14:36:45.8410182Z ##[debug]Result: true
2024-02-18T14:36:45.8411094Z ##[debug]Starting: Run CatChen/eslint-suggestion-action@v2
2024-02-18T14:36:45.8452874Z ##[debug]Loading inputs
2024-02-18T14:36:45.8455700Z ##[debug]Evaluating: github.token
2024-02-18T14:36:45.8456215Z ##[debug]Evaluating Index:
2024-02-18T14:36:45.8456649Z ##[debug]..Evaluating github:
2024-02-18T14:36:45.8457112Z ##[debug]..=> Object
2024-02-18T14:36:45.8457506Z ##[debug]..Evaluating String:
2024-02-18T14:36:45.8457930Z ##[debug]..=> 'token'
2024-02-18T14:36:45.8458708Z ##[debug]=> '***'
2024-02-18T14:36:45.8459284Z ##[debug]Result: '***'
2024-02-18T14:36:45.8464271Z ##[debug]Loading env
2024-02-18T14:36:45.8470837Z ##[group]Run CatChen/eslint-suggestion-action@v2
2024-02-18T14:36:45.8471372Z with:
2024-02-18T14:36:45.8471665Z   fail-check: true
2024-02-18T14:36:45.8472021Z   request-changes: true
2024-02-18T14:36:45.8472648Z   github-token: ***
2024-02-18T14:36:45.8473002Z   directory: ./
2024-02-18T14:36:45.8473330Z   targets: .
2024-02-18T14:36:45.8473735Z   eslint-lib-path: ./node_modules/eslint/lib/api.js
2024-02-18T14:36:45.8474359Z   eslint-bin-path: ./node_modules/.bin/eslint
2024-02-18T14:36:45.8474878Z env:
2024-02-18T14:36:45.8475288Z   MISE_TRUSTED_CONFIG_PATHS: /home/runner/work/bubu/bubu
2024-02-18T14:36:45.8475851Z   MISE_YES: 1
2024-02-18T14:36:45.8476167Z   MISE_EXPERIMENTAL: 1
2024-02-18T14:36:45.8476532Z ##[endgroup]
2024-02-18T14:36:46.0049798Z ::group::ESLint
2024-02-18T14:36:46.0050172Z ##[group]ESLint
2024-02-18T14:36:46.0053245Z Working directory is: /home/runner/work/bubu/bubu
2024-02-18T14:36:46.0054174Z Working directory is changed to: /home/runner/work/bubu/bubu
2024-02-18T14:36:46.0145418Z ##[notice]Using ESLint from: /home/runner/work/bubu/bubu/node_modules/eslint/lib/api.js
2024-02-18T14:36:47.0187242Z ##[notice]Using ESLint binary from: ./node_modules/.bin/eslint
2024-02-18T14:36:47.0373116Z [command]/home/runner/work/bubu/bubu/node_modules/.bin/eslint . --no-error-on-unmatched-pattern --format json
2024-02-18T14:36:48.4710736Z [{"filePath":"/home/runner/work/bubu/bubu/.eslintrc.cjs","messages":[],"suppressedMessages":[],"errorCount":0,"fatalErrorCount":0,"warningCount":0,"fixableErrorCount":0,"fixableWarningCount":0,"usedDeprecatedRules":[{"ruleId":"no-extra-semi","replacedBy":[]},{"ruleId":"no-mixed-spaces-and-tabs","replacedBy":[]}]},{"filePath":"/home/runner/work/bubu/bubu/index.ts","messages":[{"ruleId":"@typescript-eslint/no-unused-vars","severity":2,"message":"'server' is defined but never used. Allowed unused args must match /^_/u.","line":4,"column":28,"nodeType":"Identifier","messageId":"unusedVar","endLine":4,"endColumn":42}],"suppressedMessages":[],"errorCount":1,"fatalErrorCount":0,"warningCount":0,"fixableErrorCount":0,"fixableWarningCount":0,"source":"import type { Server } from \"bun\";\n\nconst server = Bun.serve({\n  fetch(_request: Request, server: Server): Response | Promise<Response> {\n    return Response.json({ message: \"ok\" });\n  },\n});\n\nconsole.info(`Server running at ${server.url}`);\n","usedDeprecatedRules":[{"ruleId":"no-extra-semi","replacedBy":[]},{"ruleId":"no-mixed-spaces-and-tabs","replacedBy":[]}]}]
2024-02-18T14:36:48.4960603Z ESLint exit code: 1
2024-02-18T14:36:48.4967120Z File name: .eslintrc.cjs
2024-02-18T14:36:48.4970505Z File name: index.ts
2024-02-18T14:36:48.4973981Z   [2] 'server' is defined but never used. Allowed unused args must match /^_/u. @ 4
2024-02-18T14:36:48.5956670Z ::endgroup::
2024-02-18T14:36:48.5957042Z ##[endgroup]
2024-02-18T14:36:48.5960318Z Event name: push
2024-02-18T14:36:48.5966521Z Owner: wolverian
2024-02-18T14:36:48.5969609Z Repo: bubu
2024-02-18T14:36:48.5972504Z Before SHA:<PHONE_NUMBER>000000000000000000000000000000
2024-02-18T14:36:48.5975880Z After SHA: a9c18308b31ea8fc1f4b2e930c65ac43e5891f5f
2024-02-18T14:36:48.5984102Z ::group::GitHub Push
2024-02-18T14:36:48.5984499Z ##[group]GitHub Push
2024-02-18T14:37:03.1415043Z HttpError: Not Found
2024-02-18T14:37:03.1416834Z     at file:///home/runner/work/_actions/CatChen/eslint-suggestion-action/v2/webpack:/eslint-suggestion-action/node_modules/@octokit/request/dist-node/index.js:86:1
2024-02-18T14:37:03.1418427Z     at processTicksAndRejections (node:internal/process/task_queues:96:5)
2024-02-18T14:37:03.1420236Z     at Job.doExecute (file:///home/runner/work/_actions/CatChen/eslint-suggestion-action/v2/webpack:/eslint-suggestion-action/node_modules/bottleneck/light.js:405:1)
2024-02-18T14:37:03.1536200Z ##[debug]Node Action run completed with exit code 1
2024-02-18T14:37:03.1544760Z ##[debug]Finishing: Run CatChen/eslint-suggestion-action@v2

I suspect the error comes from here:
https://github.com/CatChen/eslint-suggestion-action/blob/8fb7db4e235f7af9fc434349a124034b681d99a3/src/push.ts#L23-L27
Does this error happen if you retry? Does the workflow job have the right permission to fetch metadata about a push?


too many audit file or missing operator to change behavior

What happened:
too many audit.log and archive file

by default in the cfcr.yml and in our current deployment
 k8s-args:
        audit-log-maxage: 0
        audit-log-maxbackup: 0
        audit-log-maxsize: 0

at the end due to the number of all audit files, the syslog server have crashed:
2018/11/21 08:30:12 FATAL -- failed to create Watcher
goroutine 4071 [running]:
runtime/debug.Stack(0xc42166d590, 0x21, 0x0)
	/var/vcap/data/packages/golang/1798f26e8038da27c09433ca8194a50c05be3ea0/src/runtime/debug/stack.go:24 +0x79
github.com/hpcloud/tail/util.Fatal(0x6270f6, 0x18, 0x0, 0x0, 0x0)
	/var/vcap/packages/blackbox/src/github.com/hpcloud/tail/util/util.go:22 +0xc7
github.com/hpcloud/tail/watch.(*InotifyTracker).run(0xc421686d00)
	/var/vcap/packages/blackbox/src/github.com/hpcloud/tail/watch/inotify_tracker.go:231 +0x4ac
created by github.com/hpcloud/tail/watch.glob..func1
	/var/vcap/packages/blackbox/src/github.com/hpcloud/tail/watch/inotify_tracker.go:54 +0x233
was inside the blackbox log

What you expected to happen:
no audit log
How to reproduce it (as minimally and precisely as possible):
just run the cfcr with some application inside :-)
Anything else we need to know?:
Environment:

Deployment Info (bosh -d <deployment> deployment):
Using environment '<IP_ADDRESS>' as user 'obeyler'

Name  Release(s)           Stemcell(s)                                        Team(s)  Cloud Config
cfcr  bosh-dns/0.2.0       bosh-openstack-kvm-ubuntu-trusty-go_agent/3586.42  -        outdated
bpm/0.6.0
cfcr-etcd/1.5.0
docker/32.0.0
haproxy/8.6.0
kubo/0.24.0
nfs-volume/1.1.0
node-exporter/1.1.0
os-conf/19
routing/0.176.0
syslog/11
weave-scope/0.0.17
1 deployments

Environment Info (bosh -e <environment> environment):
Kubernetes version (kubectl version): 1.11.3
Cloud provider (e.g. aws, gcp, vsphere): openstack

I prepare a PR to solve it


Add a "Folder" column in the "Upload Status" tab of the "Manage Content" section of the Web App

Would it perhaps be possible to add an extra column called "Folder" in the "Upload Status" tab of the "Manage Content" section of the Web App? Sometimes, we upload the same file to multiple folders, but right now it is hard to track in the overview to which batch the file upload belongs.

@marcopostmovares, we are working on enhancing the "Manage Content" status page in our 1.1 release. We will add this to the backlog and likely include it as we are in there making changes. Thanks for the submission.

This has been included, along with many other new features, to the vNext-Dev branch which will be available in Main as part of the 1.1 release

resolved under imminent release of 1.1


Deprecate cents in favour of subunits

Why
cents are not currency aware, which is especially important for currencies that have no minor units like the japanese yen.
re https://github.com/Shopify/shopify/issues/121143
What

deprecate money.cents
modify allocation so it allocates subunits correctly
added tests to verify specifically JPY

@christianblais @SebastianSzturo This PR will now correctly round money objects. This might have an impact on JPY (for example: https://github.com/Shopify/shopify/issues/124792).


Add new worker identity

What changes are proposed in this pull request?

Add an abstract representation of a worker's identity.
The new identity is the same as a long based identity that used to be generated by the master. It decouples the concept of an identity and the actual implementation of an identity. It also includes a version field that allows backward compatible evolution of the worker identifier.

A new implementation for worker identities that uses UUID.
A worker will generate its own identity instead of acquiring one from the master, unless the user explicitly specifies one in the configuration. The generated identity is then persisted in the worker's working directory, and reloaded at the next worker restart.

Why are the changes needed?
For Dora workers, the identities are not generated by a centralized service, but by workers themselves. Therefore, we need something that provides randomness and avoids collision between workers, e.g. a UUID. The V1 parser implementation included in this PR uses a UUID as the worker identity.
Does this PR introduce any user facing changes?
Yes, a new configuration item alluxio.worker.identity.uuid is added that allows a user to specify the worker identity.

@dbw9580 Since we are storing identity in a file, I don't see the property key as necessary. Is there a specific reason behind?

There some cases where it's more convenient to set the identity via a environment variable, for example a user wants to centrally manage all worker identities e.g. in a database.

There some cases where it's more convenient to set the identity via a environment variable, for example a user wants to centrally manage all worker identities e.g. in a database.

who wants to do that? any user requesting specifically of maintaining their own management of the worker identities?

There some cases where it's more convenient to set the identity via a environment variable, for example a user wants to centrally manage all worker identities e.g. in a database.

who wants to do that? any user requesting specifically of maintaining their own management of the worker identities?

No actual user request so far. But it will come in handy if the user configuration directory is not writable for the identity file, and you just want to quickly debug certain things with a particular identity.

@ssz1997 @lucyge2022 I added a new config option for the path of the identity file. PTAL, thanks!

alluxio-bot, merge this please

alluxio-bot, merge this please


warning:couldn't write token cache to .cache-

Describe the bug
I try to run spotipy via a scheduled job on Windows 10.
Therefore I created a .bat file that calls the script.
When I run the script normally it works, but if I try to run it via a job or via the .bat file I get a popup of the command prompt with a url I have to go and insert the url I got redirected to.
Afterwars I get the error
Couldn't write token to cache at: .cache-XXXXXX
.bat file
"C:\Users\F\AppData\Local\Programs\Python\Spotify_Playlist_followed_artists\Scripts\python.exe" -i "C:\Users\F\PycharmProjects\Spotify_Playlist_followed_artists\main.py"
pause

In my Script it throws the error in my method where I initialize Spotipy:
def get_env():
    load_dotenv()
    SPOTIPY_CLIENT_ID = os.environ.get("SPOTIPY_CLIENT_ID")
    SPOTIPY_CLIENT_SECRET = os.environ.get("SPOTIPY_CLIENT_SECRET")
    SPOTIPY_REDIRECT_URI = os.environ.get("SPOTIPY_REDIRECT_URI")
    SPOTIPY_USER_ID = os.environ.get("SPOTIPY_USER_ID")
    scope = "user-follow-read,playlist-modify-private,playlist-modify-public"
    client_credentials_manager = spotipy.oauth2.SpotifyOAuth(
        scope=scope,
        username=SPOTIPY_USER_ID,
        client_id=SPOTIPY_CLIENT_ID,
        client_secret=SPOTIPY_CLIENT_SECRET,
        redirect_uri=SPOTIPY_REDIRECT_URI,
        open_browser=False)
    SP = spotipy.Spotify(client_credentials_manager=client_credentials_manager)

Expected behavior
I expect to enter the return url and that the .cache file could be written to.
Output
Go to the following URL: https://accounts.spotify.com/authorize?client_id=XXXXX&response_type=code&redirect_uri=http%3A%2F%XXXXX%3A8080&scope=playlist-modify-private+playlist-modify-public+user-follow-read
Enter the URL you were redirected to: http://XXXXXX/?code=XXXX-YYYY-ZZZZ-DDDD-WWWW
Couldn't write token to cache at: .cache-XXXXXX
Environment:

OS: Windows
Python version 3.10
spotipy version 2.20.0
your IDE PyCharm

Additional context
I know that there is already an issue #363 , but the workarounds in there didn't work for me

Given that the script works when run directly, but not when run from the .bat file, this sounds like a permissions issue, just like in #363.

I was getting a similar error on Linux, after enable logging the error detail I got this one:
[Errno 21] Is a directory: '.cache'
So, in my case, the script was looking for the file in "/.cache"
I've fixed the issue just by adding absolute file path cache_path=PROJECT_PATH + '/.cache'
sp = SpotifyOAuth(client_id=config.SPOTIFY_APP_CLIENT_ID, client_secret=config.SPOTIFY_APP_CLIENT_SECRET, redirect_uri=config.SPOTIFY_APP_REDIRECT_URL, cache_path=PROJECT_PATH + '/.cache')

Same issue, same error.  When I run my script on the command prompt directly, it works fine.  When I use the Windows launcher, "couldn't write token to cache at: .cache-username."  Error is originating from call spotipy.Spotify(auth=token), where token is similar to client_credentials_manager above.
Modified the save_token_to_cache as suggested by @Peter-Schorn and it does appear to be a permissions issue:
[Errno 13] Permission denied: '.cache-username'
I'll report back if I find a fix.
p.s. I also tried the fixes suggested on #363 -- including Windows Defender whitelisting, interactive mode, and changing cache_path -- with no success.  I think it's important to note that the dotfiles themselves aren't inherently problematic.  They are created and utilized just fine on direct execution.  It's the execution method that seems to be the source of the problem.
Also of note: I initially built/tested this script on Linux and am now modifying for Windows compatibility.  I never had this problem on Linux.

It's the execution method that seems to be the source of the problem.

How do you suggest we change it?

Source of the problem
The permissions issue is arising when using the launcher because cache_handler.py by default attempts to create a file in the current directory, i.e. the launcher directory, but it is protected in Windows (e.g. C:\WINDOWS\system32 in my case).  This can be fixed by specifying a full cache_path in the authorization call; see below.  (Note that this is not the cache_path fix suggested in #363, which just amounted to removing the dot.)
When executing a script directly from Windows command prompt, this will always (?) be done from a directory with read-write permissions, so no error.
Minimal example
import os
import errno

def main():
    print('Program started.')
    print('Current directory: ' + os.getcwd())

    try:
        # f = open(".cache-test", "w")  # problem line -- no error with direct execution in command prompt, error when using launcher
        # f = open(str(os.path.dirname(os.path.abspath(__file__))) + '\\' + '.cache-test', "w")  # solution line -- works with both direct execution and launcher
        f.write('test')
        f.close()
        print('Success.')
    except IOError as e:
        print(e)

    input('Press enter to exit.')

if __name__ == '__main__':
    main()

Uncommenting the problem line produces the error in question: [Errno 13] Permission denied: '.cache-test' when using the launcher.  Uncommenting the solution line successfully writes .cache-test to the directory containing the test script.  Both lines work on direct execution.
Solution using Spotipy
token = spotipy.util.prompt_for_user_token(username,scope,
 client_id=id,
 client_secret=secret,
 redirect_uri='http://localhost:8888/callback',
 cache_path=str(os.path.dirname(os.path.abspath(__file__))) + '\\' + '.cache-' + username)

spotify = spotipy.Spotify(auth=token)

Specifying a full cache_path to a writeable directory (in this case, the directory in which the script is located) resolves the issue.  This fix must also be included anywhere else a file is opened in the script.  (Otherwise you will get the same permission error with each open() call.)

@Peter-Schorn I agree with you -- this is a user/permissions issue, not actually a Spotipy issue.  I don't think a change is necessary.  It would be clunky and out of keeping with the rest of Spotipy and would probably lead to more unexpected behavior than helpful behavior.
The error is very specific.  Hopefully this solution has enough keywords for people to find it upon Googling their similar problem.
You might consider including a warning about using the Windows Python launcher in the documentation and/or providing a more descriptive error message upon encountering a permissions error.  It's not such a strange problem, but it was initially not at all clear that permissions were the reason for the failure.

I agree the error message should be displayed in the log message. I'm sure @stephanebruckert  would welcome a PR with that change.

Sure thing, I'll add one soon.


optimize:unify the format of configuration items in yml files

[x] I have registered the PR changes.

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

fixes #5117
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #5120 (6a78ffd) into develop (3e5af28) will decrease coverage by 0.38%.
The diff coverage is 50.00%.

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

@@              Coverage Diff              @@
##             develop    #5120      +/-   ##
=============================================
- Coverage      49.35%   48.96%   -0.39%
- Complexity      4145     4167      +22
=============================================
  Files            737      741       +4
  Lines          26190    26515     +325
  Branches        3232     3293      +61
=============================================
+ Hits           12925    12984      +59
- Misses         11881    12124     +243
- Partials        1384     1407      +23

Impacted Files
Coverage Δ

...igure/properties/config/ConfigNacosProperties.java
58.33% <25.00%> (-4.17%)
:arrow_down:

...e/properties/registry/RegistryNacosProperties.java
42.22% <25.00%> (-1.69%)
:arrow_down:

...java/io/seata/config/nacos/NacosConfiguration.java
11.62% <75.00%> (ø)

...overy/registry/nacos/NacosRegistryServiceImpl.java
14.89% <75.00%> (ø)

...o/seata/server/session/SessionStatusValidator.java
53.84% <0.00%> (-7.70%)
:arrow_down:

...n/src/main/java/io/seata/common/util/IdWorker.java
77.08% <0.00%> (-6.25%)
:arrow_down:

...r/console/impl/file/GlobalLockFileServiceImpl.java
94.87% <0.00%> (-2.57%)
:arrow_down:

...in/java/io/seata/server/session/GlobalSession.java
78.68% <0.00%> (-2.33%)
:arrow_down:

...io/seata/core/rpc/netty/AbstractNettyRemoting.java
12.98% <0.00%> (-1.30%)
:arrow_down:

...rage/redis/store/RedisTransactionStoreManager.java
74.67% <0.00%> (ø)

... and 14 more

 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.


Please specify EXTENT CRS

In the instructions it is unclear whether the EXTENT should be defined in 4326 (as it appears the case from my tests) or in the URL default CRS (mostly 3857).
It would be useful to add this to the README.
Thanks.

I'm sorry for late response, it seems good feedback! Thanks.

now enoughly documented extent as lat/lon.
optional arguments:
  -h, --help            show this help message and exit
  --extent EXTENT EXTENT EXTENT EXTENT
                        min_lon min_lat max_lon max_lat, whitespace delimited


Dog in Mobile Blocks View

It's hard to see the graph and some text boxes. Might be better to also use a smaller image for mobile, since it's very large.

I think we should just remove the dog from mobile and just small screens in general


Correct dl_dm term in student t inference

The dl_dm term in the gradient of the log-likelihood for the Student T process regressor seems to have been copied directly across from the Gaussian process regressor: however it should be slightly different: correction included here.
(My formatter has also adjusted a couple of other lines that are not functionality changes)

Hi @tangoed2whiskey
thanks for the contribution. Took me a while to find the time to check this.
I'll update the changelog and merge it.


List files in current folder

Is it possible to list the files and folders in the current directory?

Listing files and folders is currently not implemented but is planned for a future version.
If anyone is interested in implementing this, I'm open for pull requests.


Not Able to consume data from hashicorp vault version kv engine

Expected Behavior
We have got the secret data from hashicorp vault HTTP API through postman, teller also should be able to consume the same data.
Current Behavior
For same data, its not able to consume the data and giving panic: runtime error: invalid memory address or nil pointer dereference
Possible Solution
We need to homehow pass some key from config whihc field we want to decode from response.
From Curl we can get secret data like curl | jq -r '.data.data'. which is a map expected by teller
Steps to Reproduce

Create a new cluster in hashcorp cloud  with versioned KV2 engine enabled.
Store some secret key by creating a namespace and path.
Access the Same secret key data through curl, we will be able to get the data
Pass same URL path in teller config and try to access the data.

##Context
Response Data from API got changed and due to this  getting error. This can happen with any provider, so we need to pass some key which we want to consume in our teller application for creating map.
Specifications

Version: Teller 1.5.2 Revision 8c37a9df22f2f64ba10f71247cc8bcdf95f4ff51, date: 2022-06-21T06:34:51Z
Platform: MACOS

Hey @2dev2 thanks for reporting this issue, i'll check it

Do i am missing something, please correct me if its not bug.

Hey @2dev2, thanks for reporting this issue. i found a bug where we return secret and fixed it in pr #120
But didn't succeed in reproducing your case. I suspect the path you added in .teller.yaml is incorrect.
see my following steps to check your issue (you can check is as well on clean vault)
Run Vault locally
$ vault --version
$ Vault v1.11.0 ('ea296ccf58507b25051bc0597379c467046eb2f1+CHANGES'), built 2022-06-17T15:48:44Z
$ vault server -dev
$ export VAULT_ADDR='http://<IP_ADDRESS>:8200'
$ export VAULT_TOKEN=xxx

Create a secret json file
{
  "options": {
    "cas": 0
  },
  "data": {
    "foo": "bar",
    "secret-version": "test"
  }
}

Put secret:
curl \
    --header "X-Vault-Token: <SECRET>" \
    --request POST \
    --data @payload.json \
    http://<IP_ADDRESS>:8200/v1/secret/data/my-kv-secrets/test/demo

.teller.yaml file
project: vault
providers:
  hashicorp_vault:
    env_sync:
      path: secret/data/my-kv-secrets/test/demo

Run telelr
teller show

@2dev2, the panic releasee in v1.5.3.


Migrate from Laravel Mix to Vite

This pull request includes changes for migrating from Laravel Mix to Vite outlined in Migration Guide and automated by the Vite Converter.
Before merging, you need to:

Checkout the shift-67074 branch
Run composer update
Run npm install
Review all comments for additional changes
Thoroughly test your application (no tests?, no CI?)

Please send your feedback to<EMAIL_ADDRESS>or share the good vibes on Twitter.

:information_source: Previously, Laravel configured several npm scripts for building assets. For Vite, Laravel configures a dev script for generating and watching your assets (for development) and build script for generating your assets (for production).
Shift automated this change by replacing commands using these scripts with their new Vite script. However, you may still be referencing these scripts in your code or in deployment scripts.

:warning: Shift detected you are using Vue. As such, Vite recommends importing your CSS from your JavaScript entry point. This will improve you development experience by avoid flashes of unstyled content during hot reloading.
Shift removed your CSS entry point from your Vite configuration. However, you will need to import this CSS in your JavaScript entry point.

:information_source: Laravel renamed the environment variables used by Mix to use a VITE_ prefix, instead of MIX_. While Shift automated this change throughout your code, you should check for any additional references which may not be included in your Git repository.

:warning: Vite generates frontend assets to the public/build folder. This folder is not tracked by Git within a default Laravel project. As such, Shift added this folder to your .gitignore file.
If you wish to track the generated frontend assets within your project, such as for ease of deployment, you may undo this commit by running git revert c18472da.

:information_source: Vite requires JavaScript files containing JSX to have a .jsx file extension. Shift did not detect any JSX in files with a .js extension under the resources folder. However, if you are using JSX, be sure the files have a .jsx extension.

:warning: Shift detected you are using Inertia. You may need to update your require statement for Vue components as well as configure an entry point if you are using SSR.

:alembic: This Shift is still being refined. Please report any issues or suggestions to<EMAIL_ADDRESS>Your feedback is what helps improve the experience for everyone.

Will do after upgrading to Vue3.


change sign in button to say "log in/sign up"

hello @alexjball, can I work on this issue?

Hi @zeerakb1, that would be great! Let's change the button in the navbar to say log in/sign up and the button on the page to say Sign up.

@zeerakb1 Not sure if you already started that (if yes, I'm sorry for this situation), but I started this issue during last event on-site with the rest of the mobile changes: https://github.com/codeforboston/advocacy-maps/pull/652


fetchSize limited to 1000, higher value gives error: timeout

In a nutshell, I'm limited to reading only 1000 records from postgres, and unable to raise that limit.
Context: I was trying to read data out of postgres and write it to elsewhere (influx), using Juttle. My initial program was this (+2 years because I wanted to time-shift, irrelevant for the purposes here):
read postgres -table 'kb_out' -timeField 'time' -from :2014-02-22T20:52:36.000Z: -to :2014-02-23T22:52:34.000Z:
| put time = time + :2 years:
| write influx -db 'disks'

This resulted in only 1000 records written to influx, while there should have been 251573 (that's what I get from read postgres ... | reduce count() and is really what's in the db).
I then discovered the documented option -fetchSize which claims default of 10000, that doesn't match the 1000 i see in practice (documentation bug?).
So I thought, I'd just raise -fetchSize to get all the records, did this:
read postgres -table 'kb_out' -timeField 'time' -from :2014-02-22T20:52:36.000Z: -to :2014-02-23T22:52:34.000Z: -fetchSize 300000
| put time = time + :2 years:
| write influx -db 'disks'

And got Error: {"error":"timeout"}. The resulting count in influx is still 1000.
Just to be sure, I also tried with a very reasonable -fetchSize 3000, with the same result: timeout, and only 1000 records read from postgres / written out.

Nevermind! The issue is firmly on the side of write influx, closing this.


[DCOS-2993] Added --master-proxy option to dcos node ssh.  This enables `dcos

node ssh` in AWS.

one small nit and then LGTM when tests pass


Getting exception while fetching account details

java.lang.NoSuchMethodError: No static method encodeHexString([B)Ljava/lang/String; in class Lorg/apache/commons/codec/binary/Hex; or its super classes (declaration of 'org.apache.commons.codec.binary.Hex' appears in /system/framework/org.apache.http.legacy.boot.jar)
I am getting above exception.

I am getting the same error. Did you find a solution ?


feature extractor

I would like to use Moganet as a feature extractor with your package. Could you please assist me in achieving this?

Solution :
i find this way
feature_extractor = keras.Model(    inputs=moganet_model.inputs,    outputs=moganet_model.get_layer(name="avg_pool").output, ) 👍🏻 :)

Ya that's the solution. :)
Another is adding GlobalAveragePooling2D or other header blocks to a non-header model:
import kecam
mm = kecam.models.MogaNetTiny(num_classes=0)
print(mm.output_shape)
# (None, 7, 7, 256)
tt = keras.models.Model(mm.inputs, keras.layers.GlobalAveragePooling2D()(mm.outputs[0]))
print(tt.output_shape)
# (None, 256)


Fix linting2

this import something = require('something') syntax is really annoying. It is not sorted by fix :< it isn't really hard to solve manually tho, but still it's a pain. I hope everything passes now

But it works now @simondel 😄

The formatting is:
Built in modules

External modules

Index modules

Other internal modules```


Dockerfile mentions requirements.txt but it is not in the repo?

Maybe I am missing something here.
Trying to get it working with docker-compose

update: not blocked by this because I used poetry to generate it for me.


Broken single node selection

Godot version
4.0.dev (e312df0)
System information
Win 10 / 1080 Ti
Issue description
Selection of single node is broken after this commit.

Video:
https://user-images.githubusercontent.com/2698825/122100617-09938100-ce1c-11eb-84f2-b8a92739a9e8.mp4

Steps to reproduce
Create/open a scene and select a node.
Minimal reproduction project
Test2.zip

CC @unsignedFoo

Sorry I can't see it properly on the mobile phone. Once I have a moment I'll rollback.

I reverted the fix in master for now as the Inspector was unusable. But we can work on another attempt at solving #49451 in master.


Upgrade to v2.25.0 of the AzureRM Terraform Provider

Fixes: #678
Fixes: #680

Merged locally to add a comment for the size of the compute resource name


socket: add support for AF_VSOCK

This patch adds the support of AF_VSOCK in the socket module.
VSOCK is present since Linux 3.9.

I added an entry in the CHANGELOG, more description in the commit message and forced push.

The patch looks good so far. But could you please add some tests for the functionality?

Sure, I'll add some tests for VSOCK.

@asomers I add a test. Unfortunately, the current VSOCK implementation in Linux doesn't support loopback devices, so, for now, I add a simple test that tries to bind forbidden address and tries to connect, receiving an error.
For the 'AddressFamily' I think it is enough, what do you think?

All the test needs to do is ensure that Nix is binding correctly; we don't need a full regression test for vsock.  So if bind returns a distinctive error code then that's enough.

All the test needs to do is ensure that Nix is binding correctly; we don't need a full regression test for vsock. So if bind returns a distinctive error code then that's enough.

@asomers Okay, so what I tried in the second patch that I pushed on this PR is exactly what you said, because a bind on reserved address should return EADDRNOTAVAIL.

@asomers I removed the useless comment line that you pointed out and added few comments on VsockAddr to explain cid and port fields.

Please rebase to fix the merge conflict.

Please rebase to fix the merge conflict.

@asomers done!
Thanks,
Stefano

bors r+


Html workers

Hi, is html workers coming to this package ?
I've managed to make they work by copying the missing files from the demo pages to the same folders in my project.
It did great except for inline css and javascript and script tags which didn't work.

@gusdaud I am not sure what you mean. https://microsoft.github.io/monaco-editor/index.html uses the monaco-editor npm modules as it is shipped and has a HTML example:

Are you running your sample pages on a real protocol (such as http or https) or are you using the file protocol (the latter does not allow for web workers to be created).

[1] Chrome and Opera give an error "Uncaught SecurityError: Failed to construct 'Worker': Script at 'file:///Path/to/worker.js' cannot be accessed from origin 'null'." when you try to run a worker locally. It needs to be on a proper domain.
https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Using_web_workers#Browser_compatibility

Hey Alex. I'm running on http localhost and on google Chrome.
I've managed to make auto complete and other features provided by the workers to function by  manually copying some files to my project such as:
\min\vs\languages\html\common\htmlWorker.js
\min\vs\languages\html\common\htmlWorker.nls.js
\min\vs\base\worker\workerMain.js
That didn't come when I NPM INSTALL Monaco.
(I had to debug the online samples to figure which files were missing)

It looks like https://microsoft.github.io/monaco-editor/index.html is running something different than what's in NPM. The variable for setting the cross origin worker path should be GlobalEnvironment, not MonacoEnvironment. See this commit https://github.com/Microsoft/vscode/commit/70072cbedb7326f5dbcd316464e9fc14931d4235
In the NPM code its using GlobalEnvironment as can be seen here<EMAIL_ADDRESS>

brought back in https://github.com/Microsoft/vscode/commit/50fabe9c15774f51a5389681e96df6b7478e7407

The sample code for running cross origin is wrong. The variable for setting the cross origin worker url should be GlobalEnvironment, not MonacoEnvironment. See this commit Microsoft/vscode@70072cb

Nice! Fixed here!


Difference between instance and local variables

Simple question but just to confirm, local variables are those created inside a method, while an instance variable is available to the whole class and does not require instantiation. In my example below, the instance variables "a" and "b" are created, and "this.a" and "this.b" are local variables created in the Rectangle constructor. Does that sound right?
`double a;
double b;
public Rectangle(double a, double b) {
this.a = a;
this.b = b;
}
`

A couple things, member variables are those that a central to all instances of an object. In this case ALL rectangles have an a and b - the values are just not assigned.
The constructor allows for a new instance of a particular rectangle to be created. The constructor requires that a specific value be assigned for a and b (say 2 and 5).
2 and 5 then become the a and b values of that specific rectangle but act in the same way as all other a and b value. By adding the 'private' visibility to a and b we make sure that the scope of the variables remains within the the rectangle class
private double a;
private double b;

public Rectangle(double a, double b) {
this.a = a;
this.b = b;
}

Local variables are those that are declared within the body of a method. Only that method can use the variable. Say we wanted a method that always returned 1/2 of a rectangles area (cause I can't think of a better example...
public double getHalf(){

int scale = 2;
double full_area = this.a * this.b;
double half_area = full_area / scale;

return half_area ;

}

In this case, scale and full_area are local variables of the getHalf() method.


Add commit message standard with Husky

I would like to schedule a time during the day to talk about this
Here we can provide healthy development by maintaining a certain commit rule or first commit message, I think your suggestion is great


Swagger Header Constant Change

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[Y ] The title of the PR is clear and informative.
[ Y] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[Y ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[Y ] If applicable, the PR references the bug/issue that it fixes.
[ Y] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ Y] I have read the contribution guidelines.
[ Y] My spec meets the review criteria:

[ Y] The spec conforms to the Swagger 2.0 specification.
[ Y] The spec follows the guidelines described in the Swagger checklist document.
[ Y] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@ssankar1984,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

No modification for Ruby

No modification for Python

 https://github.com/Azure/azure-sdk-for-node/pull/2160


Uncap the value of the wounded condition

addresses #16534
uncaps the wounded value and replaces wounded pips with a number since there is no limit to the amount of pips there could theoretically be

How does it display on the character sheet?
Can you provide a screenshot?

Do you increase the value by clicking on it or by entering the number?

The previous functionality is still there so the condition can be raised and lowered by right and left clicking on the number or the condition tab

This issue was being sat on since it making a real difference is rare, and the main work would be coming up with better UI for it. I don't think I'd call this better.


ModuleNotFoundError: No module named 'molecule_docker'

When using the latest molecule_docker release (1.0.0), molecule reports a "No module named 'molecule_docker'" error:
❯ cat requirements.txt
ansible<2.10
ansible-lint
molecule[docker,lint]
yamllint

❯ python3 -m venv venv

❯ pip install pip setuptools wheel

❯ pip install -r requirements.txt

❯ molecule --version
Failed to load driver entry point Traceback (most recent call last):
  File "/home/jkirk/projects/misc/ansible-role-website/venv/lib/python3.9/site-packages/molecule/api.py", line 45, in drivers
    pm.load_setuptools_entrypoints("molecule.driver")
  File "/home/jkirk/projects/misc/ansible-role-website/venv/lib/python3.9/site-packages/pluggy/manager.py", line 299, in load_setuptools_entrypoints
    plugin = ep.load()
  File "/usr/lib/python3.9/importlib/metadata.py", line 77, in load
    module = import_module(match.group('module'))
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'molecule_docker'

molecule 3.4.0 using python 3.9
    ansible:2.9.25
    delegated:3.4.0 from molecule

❯ molecule init scenario -r ansible-role-website --driver-name docker
Failed to load driver entry point Traceback (most recent call last):
  File "/home/jkirk/projects/misc/ansible-role-website/venv/lib/python3.9/site-packages/molecule/api.py", line 45, in drivers
    pm.load_setuptools_entrypoints("molecule.driver")
  File "/home/jkirk/projects/misc/ansible-role-website/venv/lib/python3.9/site-packages/pluggy/manager.py", line 299, in load_setuptools_entrypoints
    plugin = ep.load()
  File "/usr/lib/python3.9/importlib/metadata.py", line 77, in load
    module = import_module(match.group('module'))
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'molecule_docker'

Usage: molecule init scenario [OPTIONS] [SCENARIO_NAME]
Try 'molecule init scenario --help' for help.

Error: Invalid value for '--driver-name' / '-d': 'docker' is not 'delegated'.

This used to work fine with molecule_docker 0.2.4.
My current workaround is to avoid 1.0.0:
❯ cat requirements.txt
ansible<2.10
ansible-lint
molecule[docker,lint]
molecule_docker<1.0.0
yamllint

The same here.


added backend code to detect beta user

@skhtet  - is this what you were thinking?
please do not merge this yet until I also have front-end logic in
@mannyrivera2010
@mleeBoeing

This was the wrong branch!


win: avoid uninitialized memory access in uv_getnameinfo

If GetNameInfoW fails, host and service are passed to WideCharToMultiByte that will then access uninitialized memory.
The result of WideCharToMultiByte is ignored. I don't know, if to_utf8(WCHAR host[NI_MAXHOST]) will always fit into char host[NI_MAXHOST] for this special case.

Ah, I see. WideCharToMultiByte already reports an error, if the supplied buffer is too small. No need to check the size manually first. I've changed it.

Landed in https://github.com/libuv/libuv/commit/76b873e8371be4d8b036ad78d1a9674f94655301. Thanks!


Migrate HttpUtils to use HttpClient

follow up on #8329 .
We still have several http utils in the codebase that we should migrate to use the new HttpClient.

Codecov Report

Merging #8390 (6ce7397) into master (6d2f749) will decrease coverage by 6.71%.
The diff coverage is 25.00%.

@@             Coverage Diff              @@
##             master    #8390      +/-   ##
============================================
- Coverage     70.79%   64.08%   -6.72%
  Complexity     4276     4276
============================================
  Files          1654     1609      -45
  Lines         86464    84654    -1810
  Branches      13037    12840     -197
============================================
- Hits          61216    54253    -6963
- Misses        21013    26470    +5457
+ Partials       4235     3931     -304

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
66.91% <0.00%> (-0.02%)
:arrow_down:

unittests2
14.25% <25.00%> (+0.12%)
:arrow_up:

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

Impacted Files
Coverage Δ

...org/apache/pinot/common/utils/http/HttpClient.java
13.21% <0.00%> (-32.70%)
:arrow_down:

.../controller/helix/ControllerRequestURLBuilder.java
76.51% <0.00%> (-7.46%)
:arrow_down:

...inot/controller/helix/ControllerRequestClient.java
48.00% <32.72%> (+30.22%)
:arrow_up:

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/pinot/common/metrics/ControllerMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/MinionQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...apache/pinot/common/helix/ExtraInstanceConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 383 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 6d2f749...6ce7397. Read the comment docs.


Store and expose userId in call state as structured object

What

Store userId as CommunicationUserKind in both StatefulCallClient and CallAdapter

Why

State uses backend DTOs. Using structured userId makes the type clear.
Brings StatefulCallClient on par with StatefulChatClient, which already stores userId as CommunicationUserKind.

How Tested
rush build
rush test
Process & policy checklist

[x] I have updated the project documentation to reflect my changes if necessary.
[x] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[x] This change causes current functionality to break.

This PR undoes #414
When trying to follow up and cleaning the CallContext field, I realized that while userId is optional in StatefulCallClient the calling-component-bindings package depends on it actually being the ACS user ID (Things like ParticipantList will break if some other arbitrary string is provided here).
So, instead of making this optional, I made it structured to signify that, no, this is intended to store the ACS user Id, and in fact our UI packages depend on it being the ACS user Id.


Remove unused symbols

What does this PR do?
Remove unused symbols
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Thanks for taking care of this; I didn't realize that my last PR made that type unused!


Add prettier as a CLI so we can use it to validate PRs

Prettier command is in place; needs to be wired up to actually run on each branch and be required to merge (deprioritizing this last piece)


Release v1.26.0

@haircommander volunteered to to the main work and I will support with that.

This includes updating the kubernetes API to v1.26.0, right ? (including cri-api v1)
So that it doesn't work with containerd 1.5 and cri-dockerd 0.2 (including dockershim)

yeah dockershim is correspondingly failing: https://github.com/kubernetes-sigs/cri-tools/pull/1040

I added a story to minikube, to move the installation of cri-tools/cni-plugins to the "bootstrapper":

https://github.com/kubernetes/minikube/issues/15460

Previously they were installed with the "provisioner", or most of the time not at all (for docker < 1.24)
So that usually meant that it had some old version like 1.21 and 0.8.5 installed, for all k8s versions.

When installing them with kubernetes, it should be possible to better match the expected versions:

https://github.com/kubernetes/minikube/issues/15495

note: https://github.com/kubernetes-sigs/cri-tools/pull/1043
there is a PR in progress updating k8s vendor in the containerd main branch ...
this containerd v1.6.12(LTS) will not have new v1 apis (for a while at least) so should be a good test for that.  We don't have user-namespace yet in main/1.7.. rata working on it.
we are doing another beta for containerd 1.7 ...hoping we get the last of the v1 cri api Enhancements in k8s 1.26 done before containerd 1.7 releases.. and when that happens we can add 1.7 to the matrix here to augment the main branch test
after you tag crictl for v1.26 we'll update containerd main to use it..

Release created: https://github.com/kubernetes-sigs/cri-tools/releases/tag/v1.26.0
Artifacts pushed to GCS: https://console.cloud.google.com/storage/browser/k8s-artifacts-cri-tools/release/v1.26.0

@haircommander wanna take the update for k/k, like: https://github.com/kubernetes/kubernetes/pull/112058 ?

Also make sure to update the kubeadm documentation, like: https://github.com/kubernetes/website/pull/36444

@haircommander wanna take the update for k/k, like: kubernetes/kubernetes#112058 ?

done https://github.com/kubernetes/kubernetes/pull/114548

Considering this as closed now, most of the stuff we need is in place.


Document OCI volume sources / KEP-4639

Adding a simple task to deploy image volumes. I assume most of the details will be generated from the API (https://github.com/kubernetes/kubernetes/pull/125660).
Refers to https://github.com/kubernetes/enhancements/issues/4639
Preview:

https://deploy-preview-46946--kubernetes-io-main-staging.netlify.app/docs/tasks/configure-pod-container/image-volumes
https://deploy-preview-46946--kubernetes-io-main-staging.netlify.app/docs/concepts/storage/volumes/#image

/retitle [WIP] Document OCI volume sources / KEP-4639

/sig node storage

Hello @saschagrunert :wave: please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday, July 16th, 2024 18:00 PST. Thank you!

Hi @saschagrunert, a gentle reminder that tomorrow is the deadline for having your Docs PR ready for review. Please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before tomorrow, Tuesday, July 16th, 2024 18:00 PST.

Hi @saschagrunert, a gentle reminder that tomorrow is the deadline for having your Docs PR ready for review. Please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before tomorrow, Tuesday, July 16th, 2024 18:00 PST.

Thank you for the hint, I started working on the docs which will be ready for review before the deadline.

/cc sftim

/approve

/approve
/lgtm

/lgtm

/approve

@sftim your approval is needed for this PR to be merged

Given concerns in https://github.com/kubernetes/website/pull/46946#pullrequestreview-2198828026:
/lgtm cancel
The main thing we should make clear is what fails when an image volume fails to pull: it's the whole Pod. It is technically also all of the containers, but volumes are a pod-level concept.

@saschagrunert do you want to tweak this further? If not, let us know; I'll be happy to approve it as-is.

@saschagrunert do you want to tweak this further? If not, let us know; I'll be happy to approve it as-is.

I'll give it one more push in the next hour :) Thank you for the detailed review!

Existing LGTMs
https://github.com/kubernetes/website/pull/46946#issuecomment-2249903095
https://github.com/kubernetes/website/pull/46946#issuecomment-2249903095
/lgtm
/approve


Improved Prometheus Documentation

Describe the feature request
The current Prometheus documentation is lacking in a number of areas which make it increasingly hard to set up (the documentation for Kiali and Jager isn't much better). I'll list the documentation areas in personal priority order below and hopefully as part of this someone could provide quick answers to help.

How metric merging actually works?

I assume Prometheus annotations but as a Prometheus Operator user this isn't something I have ever used
Clarification of the fact that mTLS isn't applied to the endpoint

How to use Prometheus Operator with istio?

Minimal Prometheus Operator configuration
Where the PodMonitor/ServiceMonitors can be found

Is Prometheus federation still required when using metric merging?

If so confirmation of recording rules and aggregation to move all merged metrics

For bonus points the following would be great.

Minimal Kiali Operator configuration (in CRD form)
Minimal Jager Operator configuration (in CRD form)

Describe alternatives you've considered
I've read all the relevant existing documentation, the source code and searched the internet.
I was asked to create a new issue when I commented on issue https://github.com/istio/istio/issues/27940.
[x] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

Is there anyone from the Istio team who can comment on this?


Implementing token-based authentication

Hey guys.

As i understand, the algorithm should look like this:

Check user input (username and password), return error if something goes wrong
Generate token via tokenizer.Tokenize and return it
Send the token as an HTTP header with each request

And this is how i do:
When user enters data, it's sent to the AuthModule via POST request.
And this is my AuthModule:
        public AuthModule(ITokenizer tokenizer)
        {
            Post["/signin", true] = async (ct, ctx) =>
            {
                var model = this.Bind<LoginViewModel>();
                var result = this.Validate(model);

                /* Check if user exists, return error if something happened wrong */

                var user = // ....

                var token = tokenizer.Tokenize(new UserIdentity
                {
                    UserName = user.Username,
                    Claims = new[] { user.Role }
                }, Context);

                return new
                {
                    Token = token,
                    redirect = "/"
                };
            };

So, as you can see the server returns a JSON response. Then a JS script parse this answer:
function verifyForm(id, url) {
    $(id).unbind('submit');

    $(id)
        .attr('action', url)
        .bind('submit', function () {
            var form = this;
            if ($(id).valid()) {
                $.post($(form).attr('action'), $(form).serialize(), function (data) {
                    if (data.error) {
                        // display errors
                        alert(data.error);
                    } else {
                        // redirect in case of successful sign in
                        document.location = data.redirect;

                        // BTW, probably need to handle token here via *data.token*
                    }
                });
            }
            return false;
        });
}

Token is generated successfully, but...

I am stuck with the last item. How to handle token properly and send it as an HTTP header. So for example i could log in and access to some pages where user needs to be authenticated. Judging by Google, i should use Ajax and XMLHTTPRequest in order to pass this token as an HTTP header. But i don't understand a few things, like how this code should look like or how to pass this token to XMLHTTPRequest from my JS script (maybe by using global variables, but i don't feel that it's the best approach).
Also i don't understand for example where to store it (maybe in database in case where we have long-term tokens) / (or in session for short-term tokens)  (Okay, docs say that tokens are not stored on the server, but i'm still interested in how for example expire this key immediately)

Maybe someone has a working example?
P.S. Unfortunately Nancy.Demo.Authentication.Token doesn't cover this part, and i always get RouteExecutionEarlyExitException on this.RequiresAuthentication() call.

Okay, i managed to deal with it ;)


Add an image showing the command palette to the README

Add an image showing the command palette to the README


[BUG] [remote store] FileAlreadyExistsException causes Replication failed

Describe the bug
Found a problem during running performance test against OpenSearch built from code in 2.x branch.
The index is red during indexing operation
$ curl -XGET "opens-clust-xxx.elb.us-west-2.amazonaws.com/_cat/indices?v"
health status index       uuid                   pri rep docs.count docs.deleted store.size pri.store.size
red    open   logs-1600gb qSnoT1VTSiuipllqbCJfTA  40   1<PHONE_NUMBER>            3    244.2gb        140.5gb

The below is error messages from OpenSearch log on data nodes:
[2023-08-25T17:23:39,684][ERROR][o.o.i.r.SegmentReplicationTargetService] [ip-10-0-3-162.us-west-2.compute.internal] [shardId 27] [replication id 655742] Replication failed, timing data: {INIT=0, GET_CHECKPOINT_INFO=112, FILE_DIFF=2, REPLICATING=0}

org.opensearch.indices.replication.common.ReplicationFailedException: Segment Replication failed
	at org.opensearch.indices.replication.SegmentReplicationTargetService$3.onFailure(SegmentReplicationTargetService.java:511) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.core.action.ActionListener$1.onFailure(ActionListener.java:88) [opensearch-core-2.10.0.jar:2.10.0]
	at org.opensearch.action.ActionRunnable.onFailure(ActionRunnable.java:104) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:54) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.OpenSearchExecutors$DirectExecutorService.execute(OpenSearchExecutors.java:343) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.ListenableFuture.notifyListener(ListenableFuture.java:120) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.ListenableFuture.addListener(ListenableFuture.java:82) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.action.StepListener.whenComplete(StepListener.java:95) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.indices.replication.SegmentReplicationTarget.startReplication(SegmentReplicationTarget.java:171) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.indices.replication.SegmentReplicationTargetService.start(SegmentReplicationTargetService.java:494) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.indices.replication.SegmentReplicationTargetService$ReplicationRunner.doRun(SegmentReplicationTargetService.java:480) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:908) [opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52) [opensearch-2.10.0.jar:2.10.0]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) [?:?]
	at java.lang.Thread.run(Thread.java:833) [?:?]

2023-08-25T10:23:39.928-07:00
Caused by: org.opensearch.common.util.concurrent.UncategorizedExecutionException: Failed execution
	at org.opensearch.common.util.concurrent.FutureUtils.rethrowExecutionException(FutureUtils.java:109) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.FutureUtils.get(FutureUtils.java:101) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.ListenableFuture$1.doRun(ListenableFuture.java:125) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52) ~[opensearch-2.10.0.jar:2.10.0]
	... 12 more

Caused by: java.util.concurrent.ExecutionException: java.nio.file.FileAlreadyExistsException: /home/ec2-user/opensearch/data/nodes/0/indices/qSnoT1VTSiuipllqbCJfTA/27/index/_i0y.cfe
	at org.opensearch.common.util.concurrent.BaseFuture$Sync.getValue(BaseFuture.java:286) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.BaseFuture$Sync.get(BaseFuture.java:260) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.BaseFuture.get(BaseFuture.java:82) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.FutureUtils.get(FutureUtils.java:94) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.ListenableFuture$1.doRun(ListenableFuture.java:125) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52) ~[opensearch-2.10.0.jar:2.10.0]
	... 12 more

Caused by: java.nio.file.FileAlreadyExistsException: /home/ec2-user/opensearch/data/nodes/0/indices/qSnoT1VTSiuipllqbCJfTA/27/index/_i0y.cfe
	at sun.nio.fs.UnixException.translateToIOException(UnixException.java:94) ~[?:?]
	at sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:106) ~[?:?]
	at sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111) ~[?:?]
	at sun.nio.fs.UnixFileSystemProvider.newByteChannel(UnixFileSystemProvider.java:218) ~[?:?]
	at java.nio.file.spi.FileSystemProvider.newOutputStream(FileSystemProvider.java:484) ~[?:?]
	at java.nio.file.Files.newOutputStream(Files.java:228) ~[?:?]
	at org.apache.lucene.store.FSDirectory$FSIndexOutput.<init>(FSDirectory.java:394) ~[lucene-core-9.7.0.jar:9.7.0 ccf4b198ec328095d45d2746189dc8ca633e8bcf - 2023-06-21 11:48:16]
	at org.apache.lucene.store.FSDirectory$FSIndexOutput.<init>(FSDirectory.java:387) ~[lucene-core-9.7.0.jar:9.7.0 ccf4b198ec328095d45d2746189dc8ca633e8bcf - 2023-06-21 11:48:16]
	at org.apache.lucene.store.FSDirectory.createOutput(FSDirectory.java:220) ~[lucene-core-9.7.0.jar:9.7.0 ccf4b198ec328095d45d2746189dc8ca633e8bcf - 2023-06-21 11:48:16]
	at org.apache.lucene.store.FilterDirectory.createOutput(FilterDirectory.java:75) ~[lucene-core-9.7.0.jar:9.7.0 ccf4b198ec328095d45d2746189dc8ca633e8bcf - 2023-06-21 11:48:16]
	at org.opensearch.index.store.ByteSizeCachingDirectory.createOutput(ByteSizeCachingDirectory.java:153) ~[opensearch-2.10.0.jar:2.10.0]
	at org.apache.lucene.store.FilterDirectory.createOutput(FilterDirectory.java:75) ~[lucene-core-9.7.0.jar:9.7.0 ccf4b198ec328095d45d2746189dc8ca633e8bcf - 2023-06-21 11:48:16]
	at org.apache.lucene.store.Directory.copyFrom(Directory.java:181) ~[lucene-core-9.7.0.jar:9.7.0 ccf4b198ec328095d45d2746189dc8ca633e8bcf - 2023-06-21 11:48:16]
	at org.opensearch.index.store.Store$StoreDirectory.copyFrom(Store.java:955) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.indices.replication.RemoteStoreReplicationSource.getSegmentFiles(RemoteStoreReplicationSource.java:119) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.indices.replication.SegmentReplicationTarget.lambda$startReplication$2(SegmentReplicationTarget.java:168) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.core.action.ActionListener$1.onResponse(ActionListener.java:80) ~[opensearch-core-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.ListenableFuture$1.doRun(ListenableFuture.java:126) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.OpenSearchExecutors$DirectExecutorService.execute(OpenSearchExecutors.java:343) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.ListenableFuture.notifyListener(ListenableFuture.java:120) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.common.util.concurrent.ListenableFuture.addListener(ListenableFuture.java:82) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.action.StepListener.whenComplete(StepListener.java:95) ~[opensearch-2.10.0.jar:2.10.0]
	at org.opensearch.indices.replication.SegmentReplicationTarget.startReplication(SegmentReplicationTarget.java:164) ~[opensearch-2.10.0.jar:2.10.0]
	... 7 more
	Suppressed: org.apache.http.ConnectionClosedException: Premature end of Content-Length delimited message body (expected: 495; received: 0)
		at org.apache.http.impl.io.ContentLengthInputStream.read(ContentLengthInputStream.java:178) ~[?:?]
		at org.apache.http.impl.io.ContentLengthInputStream.read(ContentLengthInputStream.java:198) ~[?:?]
		at org.apache.http.impl.io.ContentLengthInputStream.close(ContentLengthInputStream.java:101) ~[?:?]
		at org.apache.http.impl.execchain.ResponseEntityProxy.streamClosed(ResponseEntityProxy.java:142) ~[?:?]
		at org.apache.http.conn.EofSensorInputStream.checkClose(EofSensorInputStream.java:228) ~[?:?]
		at org.apache.http.conn.EofSensorInputStream.close(EofSensorInputStream.java:172) ~[?:?]
		at java.io.FilterInputStream.close(FilterInputStream.java:179) ~[?:?]
		at software.amazon.awssdk.services.s3.checksums.ChecksumValidatingInputStream.close(ChecksumValidatingInputStream.java:163) ~[?:?]
		at java.io.FilterInputStream.close(FilterInputStream.java:179) ~[?:?]
		at software.amazon.awssdk.core.io.SdkFilterInputStream.close(SdkFilterInputStream.java:83) ~[?:?]
		at org.opensearch.repositories.s3.S3RetryingInputStream.close(S3RetryingInputStream.java:243) ~[?:?]
		at org.opensearch.index.store.RemoteIndexInput.close(RemoteIndexInput.java:66) ~[opensearch-2.10.0.jar:2.10.0]
		at org.apache.lucene.store.Directory.copyFrom(Directory.java:180) ~[lucene-core-9.7.0.jar:9.7.0 ccf4b198ec328095d45d2746189dc8ca633e8bcf - 2023-06-21 11:48:16]
		at org.opensearch.index.store.Store$StoreDirectory.copyFrom(Store.java:955) ~[opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.indices.replication.RemoteStoreReplicationSource.getSegmentFiles(RemoteStoreReplicationSource.java:119) ~[opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.indices.replication.SegmentReplicationTarget.lambda$startReplication$2(SegmentReplicationTarget.java:168) ~[opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.core.action.ActionListener$1.onResponse(ActionListener.java:80) ~[opensearch-core-2.10.0.jar:2.10.0]
		at org.opensearch.common.util.concurrent.ListenableFuture$1.doRun(ListenableFuture.java:126) ~[opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.common.util.concurrent.OpenSearchExecutors$DirectExecutorService.execute(OpenSearchExecutors.java:343) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.common.util.concurrent.ListenableFuture.notifyListener(ListenableFuture.java:120) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.common.util.concurrent.ListenableFuture.addListener(ListenableFuture.java:82) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.action.StepListener.whenComplete(StepListener.java:95) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.indices.replication.SegmentReplicationTarget.startReplication(SegmentReplicationTarget.java:164) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.indices.replication.SegmentReplicationTargetService.start(SegmentReplicationTargetService.java:494) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.indices.replication.SegmentReplicationTargetService$ReplicationRunner.doRun(SegmentReplicationTargetService.java:480) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:908) [opensearch-2.10.0.jar:2.10.0]
		at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52) [opensearch-2.10.0.jar:2.10.0]
		at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) [?:?]
		at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) [?:?]
		at java.lang.Thread.run(Thread.java:833) [?:?]

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.

Finally the cluster had "cluster-manager not discovered exception" instead of red index.

2023-08-25 23:52:06.183,"[2023-08-25T23:52:06,051][ERROR][o.o.i.r.SegmentReplicationTargetService] [ip-10-0-5-34.us-west-2.compute.internal] [shardId [logs-5000gb][1]] [replication id 42376] Replication failed, timing data: {INIT=0, GET_CHECKPOINT_INFO=70, FILE_DIFF=5, REPLICATING=0}"
2023-08-25 23:43:02.184,"[2023-08-25T23:43:02,174][ERROR][o.o.i.s.IndexShard       ] [ip-10-0-5-34.us-west-2.compute.internal] [logs-5000gb][8] skip local recovery as failed to find the safe commit"
2023-08-25 23:42:54.819,"[2023-08-25T23:42:54,804][ERROR][o.o.i.s.IndexShard       ] [ip-10-0-4-14.us-west-2.compute.internal] [logs-5000gb][5] skip local recovery as failed to find the safe commit"
2023-08-25 23:41:53.717,"[2023-08-25T23:41:53,550][ERROR][o.o.i.s.IndexShard       ] [ip-10-0-4-14.us-west-2.compute.internal] [logs-5000gb][6] skip local recovery as failed to find the safe commit"
2023-08-25 23:41:23.416,"[2023-08-25T23:41:23,250][ERROR][o.o.i.r.SegmentReplicationTargetService] [ip-10-0-4-14.us-west-2.compute.internal] [shardId [logs-5000gb][5]] [replication id 41100] Replication failed, timing data: {INIT=0, GET_CHECKPOINT_INFO=84, FILE_DIFF=3, REPLICATING=0, GET_FILES=112}"
2023-08-25 23:41:23.416,"[2023-08-25T23:41:23,245][ERROR][o.o.i.r.RemoteStoreReplicationSource] [ip-10-0-4-14.us-west-2.compute.internal] File already exists?"
2023-08-25 23:41:23.199,"[2023-08-25T23:41:23,046][ERROR][o.o.i.r.SegmentReplicationTargetService] [ip-10-0-4-14.us-west-2.compute.internal] [shardId [logs-5000gb][5]] [replication id 41097] Replication failed, timing data: {INIT=0, GET_CHECKPOINT_INFO=109, FILE_DIFF=4, REPLICATING=0}"
2023-08-25 23:40:16.154,"[2023-08-25T23:40:15,908][ERROR][o.o.i.r.SegmentReplicationTargetService] [ip-10-0-5-34.us-west-2.compute.internal] [shardId [logs-5000gb][8]] [replication id 39920] Replication failed, timing data: {INIT=0, GET_CHECKPOINT_INFO=81, FILE_DIFF=5, REPLICATING=0, GET_FILES=104}"
2023-08-25 23:40:16.154,"[2023-08-25T23:40:15,904][ERROR][o.o.i.r.RemoteStoreReplicationSource] [ip-10-0-5-34.us-west-2.compute.internal] File already exists?"
2023-08-25 23:40:15.903,"[2023-08-25T23:40:15,713][ERROR][o.o.i.r.SegmentReplicationTargetService] [ip-10-0-5-34.us-west-2.compute.internal] [shardId [logs-5000gb][8]] [replication id 39914] Replication failed, timing data: {INIT=0, GET_CHECKPOINT_INFO=90, FILE_DIFF=3, REPLICATING=0}"
2023-08-25 23:40:14.651,"[2023-08-25T23:40:14,458][ERROR][o.o.i.s.IndexShard       ] [ip-10-0-5-34.us-west-2.compute.internal] [logs-5000gb][9] skip local recovery as failed to find the safe commit"


CSS discrepancies

Issue where we scribble down notes and things we discovered during #1846 we should look into.
### Tasks
- [ ] Maybe avoid classes for readonly and disabled and rely on native selectors instead.
- [ ] We should probably add css classes to our components for users to be able to target elements
- [ ] tooltip: Has no styling for content arrow. This is provided from floating-ui
- [ ] chip: clean up css variables
- [ ] Try to use aria-attributes (that needs to be set) to style as much as possible

Will be solved by #2295


Camera pressure issue

Resolve #1731
Issue
https://github.com/webaverse/app/issues/1731
We have 4 raycasts from camera corners.
Based on that, we move camera towards the player.

Here is the interface to our PhysX raycaster which can be used: https://github.com/webaverse/app/blob/1f1af692e449d536e2b72443eb379b9ccf6764e0/physics-manager.js#L198

I have implemented physx racyast insted of THREE.raycast

This has some conflicts to resolve before we can merge it.


Add devpy configuration

This is the most basic devpy config, using only default commands, but I've tested that ./dev.py build, ./dev.py test, and ./dev.py ipython work.
To add a custom command, you can do as we do in skimage:

Update pyproject.toml
Add custom command

Please note that the tool is very new, so any feedback is much appreciated.

Builds are made in build, installs go to build-install (Ralf recommended we separate those, based on their experience with SciPy.)

Still relevant, I presume?


Remove Metaclass

Motor-ODM currently uses a metaclass for its initialization mechanism. Following PEP 487 this mechanism should be implemented in __init_subclass__ instead, removing the need for a metaclass.

__init_subclass__ is more of a notification on subclass creation. Customizing subclass behavior is better fittet in an actual meta class.


Fix Kotlin DSL Reference docs for 8.2

Fixes

https://github.com/gradle/gradle/issues/25102

@bot-gradle test and merge

I've triggered a build for you.


feat: Parse HelmRelease with chartRef

This patch adds minimal support for parsing HelmRelease objects with
a chartRef field instead of a chart. There is no support for
processing such releases in any way, but at least flux-local won't abort
during parsing.

Codecov Report
Attention: Patch coverage is 80.48780% with 8 lines in your changes missing coverage. Please review.

Project coverage is 93.32%. Comparing base (5018505) to head (9460f86).
Report is 1 commits behind head on main.

Files
Patch %
Lines

flux_local/manifest.py
78.37%
8 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #709      +/-   ##
==========================================
- Coverage   93.42%   93.32%   -0.11%
==========================================
  Files          20       20
  Lines        2128     2156      +28
==========================================
+ Hits         1988     2012      +24
- Misses        140      144       +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I'll look at the failures in a few hours.

I'm not sure what the CI failure is. "Resource not accessible by integration"?

Thank you for this contribution!
I think the failure may be a permission issue trying to post the diff to the PR comment thread (likely something already broken before your change).
I'd love to see what it would take to get this feature fully working e.g. if its loading the chart from somewhere else, we may be able to make it work similar to how values from references work. Have you thought about what it would take?
Either way, will be happy to review in depth. (may take a few days)

Thank you for this contribution!
I think the failure may be a permission issue trying to post the diff to the PR comment thread (likely something already broken before your change).
I'd love to see what it would take to get this feature fully working e.g. if its loading the chart from somewhere else, we may be able to make it work similar to how values from references work. Have you thought about what it would take?
Either way, will be happy to review in depth. (may take a few days)

Yeah I hope to follow up with more code to handle dereferencing the chartRef. For a HelmChart reference, most of the existing code will be reusable. For OCIRepository references, I'm not yet sure what that will look like. Maybe nothing more than parsing the OCIRepository and dealing with the various ref forms.

I'd like to understand more about where this is headed. The reason is because of the change to make the chart option touched a lot of parts of the code. If we're just ignoring them, then I want to find sinpler ways to ignore them until I better understand the plan to add useful functionality for them.
So is the goal for now just to make flux local not fail?

I'd like to understand more about where this is headed. The reason is because of the change to make the chart option touched a lot of parts of the code. If we're just ignoring them, then I want to find sinpler ways to ignore them until I better understand the plan to add useful functionality for them.
So is the goal for now just to make flux local not fail?

This specific PR is only to allow flux-local to not throw an exception parsing HelmRelease objects with a chartRef instead of a chart field, indeed.

Given that this adds no other functionality, then i'd like to find a way to make minimal code changes to do the filtering only. (essentially do it without changing the types to make the chart optional which is the part that requires many other changes)
Adding actual support for this feature might be a heavy lift, and if you don't intend to add full support then i'd rather not have half way support and have the complexity in the code of making the chart optional. Happy to talk about how to design full support for this (which may be similar to substituteFrom, but i'm not sure).

I think I do want to implement this fully. I only did a rebase to avoid too much upstream drift. I will have some time later in the summer when I take some time off.

OK, i'm happy to start with filtering then adding full support when ready.

I would really like to see a solution here, because using one chartRef makes flux-local in the way it is used also for diff in the cluster-template completely unusable.☹
To better show you an example, what I‘m speaking of:
https://github.com/volschin/home-ops/commit/8433109d2b9b92e97353ff44552cc9ec5e8388a7

A solution I would accept is to filter them out with minimal code changes so the rest of the cluster still works, given we don't have a proposal to fully support.

Looking at this closer, I don't think we need to populate a HelmChart on the fly here rahther than dropping it and having a separate object for the chartRef. We can abstract this complexity HelmChart.

I think i was able to get full support working in https://github.com/allenporter/flux-local/pull/769

Thanks for working on it, I was just about to start looking into this again (finally on vacation!).


Show IPFS Actions on DNSLink Sites

cc #689

When we introduced option to opt-out from redirect per site (#687 ) it came with a side effect of removing IPFS context actions.
This PR:

Adds context actions on DNSLink sites (when redirect is disabled)

This adds context actions such as "Copy IPFS path", "Copy CID", "Pin" to DNSLink websites without redirect.
It includes refactoring of ipfsPathValidator to expose high level resolvers used in various places by the Companion:

resolveToPublicUrl: always return a meaningful, publicly accessible URL that can be accessed without the need of IPFS client.
resolveToIpfsPath: return a valid IPFS path that can be accessed with IPFS client.
resolveToImmutableIpfsPath: same as resolveToIpfsPath, but the path is always immutable /ipfs/
resolveToCid: returns direct CID without anything else

Adds a bunch of tests
Tweaks behavior of pin/unpin via browser action menu

In the past we closed menu after toggling pin status, but now we have a nice visual toggle switch and other uses of the switch do not close the menu, so we unified behavior and toggle action no longer closes the popup menu.

Works around missing dnslink resolver under js-ipfs

js-ipfs v0.34 does not support DNSLinks in ipfs*resolve methods yet: ipfs/js-ipfs#1918
This temporary workaround detects known errors and falls back to using path from DNSLink cache. It won't be as fresh as original, but at least resolv will work.

@ericronne  yes, it will have correct colors (when #692 is merged)


Ripple Not Disabled on NavItem

Describe the bug
Ripple=false should disable ripple effect
What is the expected behavior?
Ripple=false should disable ripple effect
What are the steps to reproduce?
Go play with the drawer nav item in the playground
Screenshots or links to minimum reproduction example
https://github.com/brightlayer-ui/angular-component-library/pull/467
Environment

Chrome Windows
Anything else to add?
No

Screen recording for reproducing bug is here recording


--target is set incorrectly if C++11 is in use on OSX

[I'm not sure where to file this bug, it's an awkward intersection of things]
In order to use c++11, you need to have a target triple targeting at least x86_64-apple-darwin11. If you specify x86_64-apple-darwin (as cargo does), and don't explicitly specify -stdlib=libc++, it will fail with very obscure error messages that individual symbols in c++11 aren't defined. If you do explicitly specify -stdlib=libc++, it will fail with the useful:
clang: error: invalid deployment target for -stdlib=libc++ (requires OS X 10.7 or later)

Either way, if the project being built targeting OSX uses c++11 features, it needs to specify --target=x86_64-apple-darwin11 not x86_64-apple-darwin. Not specifying a target at all also works fine.
It's unclear to me who should be specifying this:

Cargo doesn't know whether you're going to link against libc++, so shouldn't be making this decision. But maybe it should default to darwin11 unless you specify otherwise, just in case.
cmake-rs doesn't know whether you're going to link against libc++, so shouldn't be making this decision.
Individual library writers probably shouldn't have to explicitly write the code to conditionally change the darwin triple any time they use libc++.

Maybe the right answer is to just not set the target triple when compiling for the host target. Maybe cargo should target x86_64-apple-darwin11 by default. Maybe cmake-rs should have a "uses-c++11" boolean or something which you can set to have cmake-rs do the target-twiddling for you. Maybe everyone who uses c++11 and cmake should have to conditionally call .target(...) if the target platform is OSX and they use libc++.
I welcome any thoughts!
I suspect that this is also responsible for https://github.com/alexcrichton/cmake-rs/issues/37 - explicitly setting .target("x86_64-apple-darwin11") makes boringssl compile correctly.

Oh, this is probably actually unrelated to #37, as that is actually talking about build_target not target :)

Thanks for the report! I'm also not sure what the best way would be to solve this, but maybe there should be helper methods to enable c++11 or various standards for doing all this automatically?


Error information: "Error: SignerSign() failed." (-2147467259/0x80004005)

Hello there, I'm trying to sign an MSI file but it won't go past this point:
Azure Code Signing

Version: 1.0.52

"Metadata": {
  "Endpoint": "https://eus.codesigning.azure.net/",
  "CodeSigningAccountName": "zea-sync",
  "CertificateProfileName": "zea-sync-cert",
  "ExcludeCredentials": [
    "InteractiveBrowserCredential"
  ]
}

Submitting digest for signing...
Unhandled managed exception
Azure.RequestFailedException: Service request failed.
Status: 403 (Forbidden)

Headers:
Date: Mon, 06 May 2024 22:28:01 GMT
Connection: keep-alive
Strict-Transport-Security: REDACTED
x-azure-ref: REDACTED
X-Cache: REDACTED
Content-Length: 0
SignTool Error: An unexpected internal error has occurred.

   at Azure.CodeSigning.CertificateProfileRestClient.SignAsync(String codeSigningAccountName, String certificateProfileName, SignRequest body, String xCorrelationId, String clientVersion, CancellationToken cancellationToken)
   at Azure.CodeSigning.CertificateProfileClient.StartSignAsync(String codeSigningAccountName, String certificateProfileName, SignRequest body, String xCorrelationId, String clientVersion, CancellationToken cancellationToken)
   at Azure.CodeSigning.Dlib.Core.DigestSigner.SignAsync(UInt32 algorithm, Byte[] digest, SafeFileHandle safeFileHandle, CancellationToken cancellationToken)
   at Azure.CodeSigning.Dlib.Core.DigestSigner.Sign(UInt32 algorithm, Byte[] digest, SafeFileHandle safeFileHandle)
   at AuthenticodeDigestSignExWithFileHandleManaged(_CRYPTOAPI_BLOB* pMetadataBlob, UInt32 digestAlgId, Byte* pbToBeSignedDigest, UInt32 cbToBeSignedDigest, Void* hFile, _CRYPTOAPI_BLOB* pSignedDigest, _CERT_CONTEXT** ppSignerCert, Void* hCertChainStore)

Error information: "Error: SignerSign() failed." (-2147467259/0x80004005)

This is what my "Certificate profiles" page looks like:

Any help would be much appreciated. Thank you.

You are getting Status: 403 (Forbidden). This looks like an authentication issue? Did you assign the correct role here: https://dlemstra.github.io/github-stories/2023/imagemagick-now-uses-azure-code-signing/#create-and-authorize-app-registration?

Hi @mauroporras, what form of authentication are you attempting to use?

@dlemstra, @japarson, thank you very much guys. I got it working using the ImageMagick guide
I'm using azure-client-secret auth.


dispose of opened socket

seems to solve #28

I know about this bug in Mono it does not correctly implement:
_disconnectEvent.DisconnectReuseSocket = true;
On .Net it correctly reuse socket, so it saves some allocations and OS calls. I would accept it if you check for environment type and did it only on Mono.
I still hope that someone will implement correct and optimal Saea implementation for Linux.

Do you have any #if mono directives already setup?
On 12 March 2015 at 21:44, Boris Letocha<EMAIL_ADDRESS>wrote:

Closed #30 https://github.com/Bobris/Nowin/pull/30.
--
Reply to this email directly or view it on GitHub
https://github.com/Bobris/Nowin/pull/30#event-251320343.

Do you have any #if mono directives already setup?
If not would you be happy if I added them and would I set _disconnectEvent.DisconnectReuseSocket = false; as well as the socket.Dispose already in this PR?

I though about not doing it by #if directives but by checking "something" at runtime, so you don't need to recompile .dll you get from Nuget.
 public static bool IsRunningOnMono ()
 {
      return Type.GetType ("Mono.Runtime") != null;
 }

Sure I can add that.
Do I just need to set that boolean value to false and dispose of the socket
after that?
On 13 March 2015 at 07:22, Boris Letocha<EMAIL_ADDRESS>wrote:

I though about not doing it by #if directives but by checking "something"
at runtime, so you don't need to recompile .dll you get from Nuget.
public static bool IsRunningOnMono ()
{
return Type.GetType ("Mono.Runtime") != null;
}
--
Reply to this email directly or view it on GitHub
https://github.com/Bobris/Nowin/pull/30#issuecomment-78845966.

Set all 3:
        _receiveEvent.DisconnectReuseSocket = true;
        _sendEvent.DisconnectReuseSocket = true;
        _disconnectEvent.DisconnectReuseSocket = true;

To false already in void RecreateSaeas(), based on running on Mono. Maybe even name it "does not support DisconnectReuseSocket"

Thanks. I've updated the PR

Looks like as the PR is closed you cant see the changes in the diff

Reopened, but still don't see anything.

Should be there now :smile:

I will polish it a little bit. And build new version after that.

Thanks, appreciate it!

Done, please test.

Left an update on #28


Reimplement wac-parser on wac-graph.

This PR is a complete reimplementation of wac-parser's resolution to be based on wac-graph.
It also contains an overhaul of the wac-graph API to facilitate its use from wac-parser:

Reimplements CompositionGraph on top of petgraph::StableDiGraph, which eliminates needing to keep track of nodes internally to CompositionGraph.
Packages now populate their type information into the graph's types collection rather than having their own collections.
Splits the Error enum into discrete error types for each graph operation; this allows the AST resolver to properly attach span information to errors.
Add type dependency edges for defined types, ensuring topological ordering.
Fix encoding of defined type aliases so that each encoded alias refers to the export index of the previously encoded type.
Added a debug formatter for CompositionGraph that outputs a DOT representation (used in resolution tests as well).
Graph encoding now includes an optional producers custom section.
Fixed toposort to be in ascending node index order for independent nodes.

The subtype checker implementation was reverted to the previous implementation, where invert and revert are called at specific points to change the type of check being performed; the newer implementation did not give the correct "expected, found" error messages for AST resolution.
wac resolve now outputs a DOT representation of the resolved composition graph instead of JSON.
Most of the deleted code comes from code that has already moved into wac-types and wac-graph.
Closes #79.
Fixes #78.
Fixes #76.

When this and a fix for #81 lands, I plan on adding a publish script to CI and creating a 0.1.0 tag to publish to crates.


Change Survey Seeder file

maintain this sequence
Cost of Migration Survey
Distributive Trade Survey
Hotel and Restaurant survey
Household Income and Expenditure Survey
Improvement of GDP Compilation and Rebasing of Indices
Improving Labour Market Information through Labour Force Survey (LFS)
Measuring ICT Access and Use by Households and Individuals
Multiple Indicator Cluster Survey (MICS)
National Hygiene survey
National Survey on Persons with Disabilities
Pilot Survey on SADDD and CCA & DRR
Report on Sample Vital Registration System (SVRS)
Report on the Survey of Private Healthcare Institutions
Strengthening of Environment, Climate Change and Disaster Related Statistics
Survey of Manufacturing Industries (SMI)
Survey of Travel Agent, Tour Operator and Clearing & Forwarding Agent
Survey on Gross Marketed Surplus of Agricultural Commodities
Survey on Private Commercial Mechanized and Non-Mechanized and Water Transport
Survey on Private Health Service Establishment
Time Use Survey
Tourism Satellite Account
Urban Socioeconomic Assessment Survey
Urban Socioeconomic Assessment Survey Determination of Causes of Deaths by Verbal Autopsy (VA) Perception Survey on Livelihood (PSL)
User Satisfaction Survey
Wholesale and Retail Trade Survey

done

merged


Frederic/avoid multiple preflight reqs

Codecov Report

Merging #10 (2097f89) into main (ff665f6) will increase coverage by 0.16%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   79.88%   80.04%   +0.16%
==========================================
  Files          23       23
  Lines        1735     1729       -6
==========================================
- Hits         1386     1384       -2
+ Misses        349      345       -4

Impacted Files
Coverage Δ

tandem_http_client/src/lib.rs
0.00% <0.00%> (ø)

tandem_http_server/src/engine.rs
75.30% <100.00%> (+0.30%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Skip tests that depend on the BCI repo for the LTSS container

We have the LTSS container on a service pack still in general support, so a container-specific exclusion rather than merely a version based exclusion is needed.

LGTM in general, but I think handling this in a decorator would be more obvious as that's where we generally handle skipifs

it is a lot of repetition imho. anyhow, updated.

LGTM, but some CI checks fail

Well, we also need to fix the remaining test (metadata for instance is red as well)

did you check the error?

Command '['docker', 'pull', 'registry.suse.com/suse/ltss/sle15.5/bci-base:15.5']' returned non-zero exit status 1

That's because this PR is not being merged. metadata checks are green in openqa, the only thing blocking it from being erleased is this PR.

moving ahead, we need to get the container out..


Python Linter, Code Completion Problem with v2021.2.633441544

Environment data

VS Code version: 1.54.2
Python v2021.2.633441544
Jypter v2021.3.619093157
OS and version: Win10 x64 v10.0.19041
Python version (& distribution if applicable, e.g. Anaconda): 3.9.1 x64
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: XXX
Relevant/affected Python-related VS Code extensions and their versions: XXX
Value of the python.languageServer setting: XXX

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
Python Linter, code completion and source code outline working.
Actual behaviour
I have the following error notification showing when trying to goto declarations of variables (is the case for almost all variables/class definitions):
"unable to open 'undefined'. cannot open file \undefined"
Also, linter error checking, syntax highlighting and autocomplete seem to have stopped working.
Some variable declarations are found, but everything says "undefined". Most variable declarations are not found when hovering with the mouse over the variable while pressing CTRL.
Steps to reproduce:

Install Python v2021.2.633441544, Jupyter version does not seem to matter. Error appears.
Switch back to v2021.2.625869727 and everything works.

Logs
XXX

Duplicate of #15638


Tiling Problem 2XN

💥 Proposal
I would like to implement the code for  tiling problem 2XN in C.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


Gps103 ignoring "help me" Message without location

Hi,
testing an obd tracker inside my home, I have tried to use the sos button which send an "help me" message.
But the message is ignored by the gps103 decoder.
Here is one sample message (IMEI changed)
imei:359710099999999,help me,,,L,,,113b,,558f,,,,,0,0,,,;
I have tried to add this message in the Gps103ProtocolDecoderTest, and the error is "position is null"
Apparently the GPS signal is off where I'm testing, the tracker are sending the message and the decoder just ignoring because does not have a valid position.
Is possible to accept this kind of message and just use the last valid location for device and set an attribute indicating this ? Maybe setting "Valid" to false/no.
I have tried with current version and it does not recognize this specific message.
The current version recognize the message that is sent  when the GPS have a valid location, not when the help message come without valid lat/lon.

Those kind of messages will be supported starting from next release. The support is already implemented, so if you don't want to wait, you can compile a build from source code.

I have tried with the current github code, but does not work.
The simple alive messages update the status time (but are not passed thru the handlers).
But the messages like the one I posted before does not update status time, neither go thru the handlers.
Its important to register when someone pressed the help button on device, and also make the message pass thru the handlers to take care.
In my testing I'm trying to bridge the tracking information to an mqtt server. This will provide many possibilities for integration.
Here part of the log, without decoding.
2015-11-27 10:27:16 DEBUG: [43ADFDC8: 5001 < <IP_ADDRESS>] HEX: 23232c696d65693a3335393731303034393037353039372c413b
2015-11-27 10:27:16 DEBUG: [43ADFDC8: 5001 > <IP_ADDRESS>] HEX: 4c4f4144
2015-11-27 10:27:25 DEBUG: [43ADFDC8: 5001 < <IP_ADDRESS>] HEX: 696d65693a3335393731303034393037353039372c68656c70206d652c2c2c4c2c2c2c313133622c2c353538662c2c2c2c2c302c302c2c2c3b
2015-11-27 10:27:37 DEBUG: [43ADFDC8: 5001 < <IP_ADDRESS>] HEX: 696d65693a3335393731303034393037353039372c68656c70206d652c2c2c4c2c2c2c313133622c2c353538662c2c2c2c2c302c302c2c2c3b
2015-11-27 10:27:42 DEBUG: [43ADFDC8: 5001 < <IP_ADDRESS>] HEX: 696d65693a3335393731303034393037353039372c747261636b65722c2c2c4c2c2c2c313133622c2c353538662c2c2c2c2c302c302c2c2c3b
2015-11-27 10:28:13 DEBUG: [43ADFDC8: 5001 < <IP_ADDRESS>] HEX: 696d65693a3335393731303034393037353039372c747261636b65722c2c2c4c2c2c2c313133622c2c353538662c2c2c2c2c302c302c2c2c3b
2015-11-27 10:28:38 DEBUG: [43ADFDC8: 5001 < <IP_ADDRESS>] HEX:<PHONE_NUMBER>303034393037353039373b
2015-11-27 10:28:38 DEBUG: [43ADFDC8: 5001 > <IP_ADDRESS>] HEX: 4f4e
2015-11-27 10:29:13 DEBUG: [43ADFDC8: 5001 < <IP_ADDRESS>] HEX:<PHONE_NUMBER>303034393037353039373b
2015-11-27 10:29:13 DEBUG: [43ADFDC8: 5001 > <IP_ADDRESS>] HEX: 4f4e
2015-11-27 10:30:13 DEBUG: [43ADFDC8: 5001 < <IP_ADDRESS>] HEX:<PHONE_NUMBER>303034393037353039373b
2015-11-27 10:30:13 DEBUG: [43ADFDC8: 5001 > <IP_ADDRESS>] HEX: 4f4e

Fixed the problem. It should work now.


Error when running initialise.py

%run initialise.py

UnicodeDecodeError                        Traceback (most recent call last)
~\mygit\numpy-100\initialise.py in
1 import numpy as np
2
----> 3 import generators as ge
4
5
~\mygit\numpy-100\generators.py in
34
35 HEADERS = ktx_to_dict(os.path.join('source', 'headers.ktx'))
---> 36 QHA = ktx_to_dict(os.path.join('source', 'exercises100.ktx'))
37
38
~\mygit\numpy-100\generators.py in ktx_to_dict(input_file, keystarter)
9
10     with open(input_file, 'r+') as f:
---> 11         lines = f.readlines()
12
13     k, val = '', ''
~\Anaconda3\lib\encodings\cp1251.py in decode(self, input, final)
21 class IncrementalDecoder(codecs.IncrementalDecoder):
22     def decode(self, input, final=False):
---> 23         return codecs.charmap_decode(input,self.errors,decoding_table)[0]
24
25 class StreamWriter(Codec,codecs.StreamWriter):
UnicodeDecodeError: 'charmap' codec can't decode byte 0x98 in position 53: character maps to
I think this error is due to the default cp1251 encoding on my localized Windows 10.
I changed line 10 in generators.py to
with open(input_file, 'r+', encoding='utf-8') as f:
and problem was solved.
I'm noob in GitHub. Should I do something else? Pull request?

Might be related to #100. @SebastianoF Do you know what could be wrong ?

Read too fast, I did not see you found the fix ! If you can make a PR that would be perfect !


license and license header ?

will be required !

I agree. This issue already created #1.

Fixed by Maruyama-san at commit: 0fb9cc621ff9c9f6f205aee13e9be83f11253fd5


[AutoPR locationbasedservices/resource-manager] [LocationBasedServices] Resource manager client name should be more specific

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2596

(message created by the CI based on PR content)

Codecov Report

Merging #2106 into restapi_auto_locationbasedservices/resource-manager will not change coverage.
The diff coverage is n/a.

@@                                 Coverage Diff                                  @@
##           restapi_auto_locationbasedservices/resource-manager    #2106   +/-   ##
====================================================================================
  Coverage                                                53.06%   53.06%
====================================================================================
  Files                                                     4776     4776
  Lines                                                   119485   119485
====================================================================================
  Hits                                                     63408    63408
  Misses                                                   56077    56077

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 e46bebe...7e65738. Read the comment docs.