-
Notifications
You must be signed in to change notification settings - Fork 76
updating keda package to v2.17.0 and updating go version to 1.23.8 #473
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
updating keda package to v2.17.0 and updating go version to 1.23.8 #473
Conversation
Signed-off-by: Brett Thompson <[email protected]>
WalkthroughThe changes update the Go toolset version in the container build process and refresh multiple Go module dependencies to newer versions. No functional code changes, public API modifications, or alterations to the module structure are present. Changes
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
⏰ Context from checks skipped due to timeout of 90000ms (2)
🔇 Additional comments (8)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
@brettmthompson: The following test failed, say
Full PR test history. Your PR dashboard. 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-sigs/prow repository. I understand the commands that are listed here. |
Need to re-run CI job failure once openshift/release#66265 is merged |
/cc @israel-hdez |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/aprove
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: brettmthompson, spolti The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
keda version bump no longer required. CVEs resolved in #464. Closing this PR |
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-sigs/prow repository. |
Description
In the latest sync from opendatahub-io to red-hat-data-services, the snyk scan failed: https://app.snyk.io/org/red-hat-openshift-data-science-rhods/pr-checks/46eb5b05-92b3-4f68-a859-21a3505d55eb
The failure was due to vulnerabilities identified in the
github.com/expr-lang/expr
v1.16.9 package which is a dependency ofgithub.com/kedacore/keda/v2
v2.16.1. Per snyk these issues are resolved ingithub.com/expr-lang/expr
v1.17.0 . This will require an update of thegithub.com/kedacore/keda/v2
package to a version >= v2.17.0.This PR updates the
github.com/kedacore/keda/v2
package to version v2.17.0. This updated package requires golang version 1.23.8 so the golang version is also bumped in this PR.How Has This Been Tested?
Merge criteria:
Summary by CodeRabbit