Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wsrpc version upgrade #15692

Merged
merged 7 commits into from
Dec 18, 2024
Merged

Conversation

JoshC2k
Copy link

@JoshC2k JoshC2k commented Dec 13, 2024

Upgrades wsrpc package version to fix a panic condition on the nodes.

@JoshC2k JoshC2k requested review from a team as code owners December 13, 2024 20:16
@JoshC2k JoshC2k requested review from EasterTheBunny and removed request for a team December 13, 2024 20:16
akuzni2
akuzni2 previously approved these changes Dec 13, 2024
Copy link
Contributor

github-actions bot commented Dec 13, 2024

AER Report: CI Core

aer_workflow , commit , Clean Go Tidy & Generate , Detect Changes , Core Tests (go_core_tests) , lint , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_race_tests) , Core Tests (go_core_fuzz) , Flakey Test Detection , SonarQube Scan

1. Missing grafana_org_id parameter:[Re-run tests]

Source of Error:
Re-run tests	2024-12-17T19:10:01.9099926Z 2024/12/17 19:10:01 Error re-running flakey tests: `grafana_org_id` is required
Re-run tests	2024-12-17T19:10:01.9115169Z ##[error]Process completed with exit code 1.

Why: The error occurred because the grafana_org_id parameter is required but was not provided in the command execution context.

Suggested fix: Ensure that the grafana_org_id parameter is correctly set and passed to the command. This might involve checking environment variables or configuration files to ensure the value is present.

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@JoshC2k JoshC2k force-pushed the wsrpcVersionUpgrade branch from c0a6a7a to 5ef1762 Compare December 17, 2024 13:12
Signed-off-by: Josh Brown <[email protected]>
Signed-off-by: Alex Kuznicki <[email protected]>
Signed-off-by: Josh Brown <[email protected]>
Signed-off-by: Alex Kuznicki <[email protected]>
Signed-off-by: Josh Brown <[email protected]>
Signed-off-by: Alex Kuznicki <[email protected]>
Signed-off-by: Josh Brown <[email protected]>
Signed-off-by: Alex Kuznicki <[email protected]>
@akuzni2 akuzni2 force-pushed the wsrpcVersionUpgrade branch from 5ef1762 to 0e5e2b2 Compare December 17, 2024 16:06
akuzni2
akuzni2 previously approved these changes Dec 17, 2024
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@Bwest981 Bwest981 merged commit ea86aaa into release/2.17.0-mercury-20241004 Dec 18, 2024
276 of 280 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants