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

make the openstack vm image based and have a volume attached #131

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

bennyz
Copy link
Member

@bennyz bennyz commented Mar 13, 2024

No description provided.

@bennyz bennyz requested a review from ahadas March 14, 2024 08:21
@bennyz bennyz force-pushed the openstack-imagebased branch 2 times, most recently from 0af4985 to 4b01b46 Compare March 19, 2024 19:36
Copy link
Member

@ahadas ahadas left a comment

Choose a reason for hiding this comment

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

the changes look good but the change for the storage profile is good on its own, better be separated to a different PR

@@ -28,9 +28,9 @@ function k8s_grant_permissions {

# workaround to https://github.com/kubevirt/kubevirt/issues/7078
function k8s_patch_storage_profile {
kubectl patch --type merge -p '{"spec": {"claimPropertySets": [{"accessModes": ["ReadWriteOnce"]}]}}' StorageProfile standard
kubectl patch --type merge -p '{"spec": {"claimPropertySets": [{ "volumeMode": "Filesystem", "accessModes": ["ReadWriteOnce"]}]}}' StorageProfile standard
Copy link
Member

Choose a reason for hiding this comment

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

please post this change in a separate PR

@bennyz
Copy link
Member Author

bennyz commented Mar 20, 2024

the changes look good but the change for the storage profile is good on its own, better be separated to a dif

@bennyz bennyz closed this Mar 20, 2024
@bennyz bennyz reopened this Mar 20, 2024
@ahadas ahadas merged commit 3cb5df1 into kubev2v:main Mar 20, 2024
2 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.

2 participants