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

Add volume attach tag with nova microversion 2.49 #1713

Merged

Conversation

jichenjc
Copy link
Contributor

@jichenjc jichenjc commented May 14, 2024

@jichenjc jichenjc force-pushed the add_volume_tag branch 4 times, most recently from 9de7635 to 5524317 Compare May 14, 2024 09:02
@jichenjc
Copy link
Contributor Author

@nikParasyr could you please help review this ? THanks

Copy link
Member

@nikParasyr nikParasyr left a comment

Choose a reason for hiding this comment

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

Thank you for this.
Some small nits on docs

docs/resources/compute_volume_attach_v2.md Outdated Show resolved Hide resolved
docs/resources/compute_volume_attach_v2.md Outdated Show resolved Hide resolved
@jichenjc
Copy link
Contributor Author

@nikParasyr thanks for the info, updated the doc accordingly

Copy link
Member

@nikParasyr nikParasyr left a comment

Choose a reason for hiding this comment

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

LGTM

@nikParasyr nikParasyr merged commit 9bb7462 into terraform-provider-openstack:main May 16, 2024
6 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