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

libvirt: update script to config libvirt on RHEL. #1822

Merged
merged 1 commit into from
Apr 28, 2024

Conversation

wyuany
Copy link
Contributor

@wyuany wyuany commented Apr 26, 2024

Update script to config libvirt not only work with Ubuntu but also RHEL

Fixes: #1821

Copy link
Member

@beraldoleal beraldoleal left a comment

Choose a reason for hiding this comment

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

Hi @wyuany this is welcome, thanks! LGTM, except it shows 3 commits.

Do you mind rebasing/squashing them?

Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

+1 to Beraldo's suggestion to squash the commits, otherwise it LGTM

Update script to config libvirt not only work with Ubuntu but also RHEL
Fixes: confidential-containers#1821

Signed-off-by: Yuan Yuan Wang <wyuany@cn.ibm.com>
Copy link

@huoqifeng huoqifeng left a comment

Choose a reason for hiding this comment

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

LGTM

@huoqifeng huoqifeng dismissed beraldoleal’s stale review April 28, 2024 07:23

Review comments addressed already, thanks! Beraldo, To merge the PR, I have to dismiss the rejection.

@huoqifeng huoqifeng merged commit a589f66 into confidential-containers:main Apr 28, 2024
18 checks passed
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.

Config libvirt on RHEL
4 participants