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

osbuilder: add pkg bash for alpine #8456

Merged
merged 1 commit into from Nov 30, 2023

Conversation

cheriL
Copy link
Contributor

@cheriL cheriL commented Nov 16, 2023

Fix: #8447

@katacontainersbot
Copy link
Contributor

Can one of the admins verify this patch?

Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @cheriL!

@fidencio
Copy link
Member

I've approved this, but you'll need to re-work the PR to properly add a commit message explaining why we're adding the bash component into the alpine rootfs / initrd.

Please, do that and our CI will then be happy.

Copy link
Contributor

@cmaf cmaf left a comment

Choose a reason for hiding this comment

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

@cheriL could you also make sure the Fixes line is in the commit (following the contributing guide)

@cheriL
Copy link
Contributor Author

cheriL commented Nov 17, 2023

Sure @fidencio @cmaf ! Done.

@amshinde
Copy link
Member

/test

@fidencio
Copy link
Member

@cheriL, can we have this one rebased again? Sorry, for inconvenience.

The bash component is required in the guest for debug console to work properly.

Fixes: kata-containers#8447

Signed-off-by: soup <lqh348659137@outlook.com>
@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label Nov 30, 2023
@cheriL
Copy link
Contributor Author

cheriL commented Nov 30, 2023

@fidencio Done

@fidencio
Copy link
Member

/test

@fidencio
Copy link
Member

/retest-dragonball

@gkurz gkurz merged commit 48e5596 into kata-containers:main Nov 30, 2023
175 of 239 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

can not enter guest os using kata-runtime exec
7 participants