-
Notifications
You must be signed in to change notification settings - Fork 30
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
[images] build based on ubuntu24.04 #530
Conversation
b208f89
to
fbedcc5
Compare
5af4d18
to
5900488
Compare
9a642d3
to
3126d9f
Compare
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
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
just don't forget to fixup the wip
before the merge :)
As part of our work in scylladb/scylla-enterprise#4111 Updating our Images to the latest LTS Ubuntu Fixes: scylladb#527
Seen during my debug sessions ``` /opt/scylladb/scylla-machine-image/libexec/scylla_login:13: SyntaxWarning: invalid escape sequence '\|' MSG_HEADER = ''' ```
3126d9f
to
98b199f
Compare
Done :-) |
@fruch this is ready for merge, please advise about the performance rests you did as part of https://github.com/scylladb/qa-tasks/issues/1729, if the results are better/the same let's merge it /cc @mykaul |
Now that we added support for Ubuntu24.04 and also migrating our images to be based on that (scylladb/scylla-machine-image#530), we should also modify our docker image Fixes: scylladb#19738
Based on https://github.com/scylladb/qa-tasks/issues/1729#issuecomment-2231083515, it seems we are good to go |
Now that we added support for Ubuntu24.04 and also migrating our images to be based on that (scylladb/scylla-machine-image#530), we should also modify our docker image Fixes: #19738 Closes #19764
As part of our work in scylladb/scylla-enterprise#4111
Updating our Images to the latest LTS Ubuntu
Also include a minor syntax fix - 9a642d3
Fixes: #527