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

[images] build based on ubuntu24.04 #530

Merged
merged 2 commits into from
Jul 17, 2024

Conversation

yaronkaikov
Copy link
Collaborator

@yaronkaikov yaronkaikov commented Jul 3, 2024

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

@yaronkaikov yaronkaikov force-pushed the move-to-ubuntu-24.04 branch 6 times, most recently from b208f89 to fbedcc5 Compare July 8, 2024 18:17
@yaronkaikov yaronkaikov force-pushed the move-to-ubuntu-24.04 branch 3 times, most recently from 5af4d18 to 5900488 Compare July 15, 2024 09:10
@yaronkaikov yaronkaikov marked this pull request as ready for review July 15, 2024 10:10
@yaronkaikov
Copy link
Collaborator Author

Copy link
Contributor

@syuu1228 syuu1228 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@Annamikhlin Annamikhlin left a 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 = '''
```
@yaronkaikov
Copy link
Collaborator Author

LGTM just don't forget to fixup the wip before the merge :)

Done :-)

@yaronkaikov
Copy link
Collaborator Author

@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

yaronkaikov added a commit to yaronkaikov/scylla that referenced this pull request Jul 16, 2024
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
@yaronkaikov
Copy link
Collaborator Author

Based on https://github.com/scylladb/qa-tasks/issues/1729#issuecomment-2231083515, it seems we are good to go

@mykaul

@yaronkaikov yaronkaikov merged commit 9f133de into scylladb:next Jul 17, 2024
1 check passed
@yaronkaikov yaronkaikov deleted the move-to-ubuntu-24.04 branch July 17, 2024 11:14
avikivity pushed a commit to scylladb/scylladb that referenced this pull request Jul 17, 2024
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
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.

[images] build images based on ubuntu 24.04
3 participants