-
Notifications
You must be signed in to change notification settings - Fork 581
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
Update to 2.11.5-SNAPSHOT #5904
Conversation
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.
Thanks for the updates @corneil .
The changes for the snapshot update look good.
However, I don't think we want to update the stream apps version in 2.11.x
though as that would switch the default apps version in a patch release for the carvel packages. We are already updating to the latest apps in main-3
.
Please submit a PR w/ only the snapshot updates.
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.
Thanks for the updates @corneil . The stream apps are still not on the default version shipped w/ SCDF 2.11.x (3.2.1 -> 2021.*
).
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.
LGTM, for my requests. Once @onobc marks it good, then merge.
The docs specify that the registration scripts uses 2022.0.0 |
Updates to snapshot.
Updates local test deployment scripts to register 2024.0.0/5.0.0 stream apps.