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

dist/docker: revert dropping systemd package #18643

Closed
wants to merge 1 commit into from

Conversation

syuu1228
Copy link
Contributor

@syuu1228 syuu1228 commented May 13, 2024

On 7ce6962 we dropped openssh-server, it also dropped systemd package and caused an error on Scylla Operator (#17787).

This reverts dropping systemd package and fix the issue.

Fix #17787

  • ** Backport reason (please explain below if this patch should be backported or not) **

@mykaul
Copy link
Contributor

mykaul commented May 13, 2024

If reverting as a short-term solution works, fine. But let's make sure we have an item on our roadmap to fix this fully and properly.

@yaronkaikov
Copy link
Contributor

On 7ce6962 we dropped openssh-server, it also dropped systemd package and caused an error on Scylla Operator (#17787).

This reverts dropping systemd package and fix the issue.

Fix #17787

  • ** Backport reason (please explain below if this patch should be backported or not) **

Please mark this checkout and explain why should it backport (or why shouldn't it)

@scylladb-promoter
Copy link
Contributor

🟢 CI State: SUCCESS

✅ - Build
✅ - Container Test

Build Details:

  • Duration: 2 hr 24 min
  • Builder: i-08a2c8350fe85c365 (m5ad.8xlarge)

@yaronkaikov yaronkaikov added the backport/5.4 Issues that should be backported to 5.4 branch once they'll be fixed label May 20, 2024
@yaronkaikov
Copy link
Contributor

@syuu1228 please rebase, you have now conflicts

On 7ce6962 we dropped openssh-server,
it also dropped systemd package and caused an error on Scylla Operator
(scylladb#17787).

This reverts dropping systemd package and fix the issue.

Fix scylladb#17787
@syuu1228
Copy link
Contributor Author

Rebased.

@scylladb-promoter
Copy link
Contributor

🟢 CI State: SUCCESS

✅ - Build
✅ - Container Test

Build Details:

  • Duration: 2 hr 26 min
  • Builder: i-0f8da53b7c6dfba0f (m5d.8xlarge)

@yaronkaikov
Copy link
Contributor

@scylladb/scylla-maint can you please merge this one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/5.4 Issues that should be backported to 5.4 branch once they'll be fixed promoted-to-master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Container image lacks command used by Scylla scripts
4 participants