Skip to content

Cinder V3: Update a volume’s bootable status#1891

Merged
jtopjian merged 2 commits intogophercloud:masterfrom
kayrus:volume-bootable-action
Mar 13, 2020
Merged

Cinder V3: Update a volume’s bootable status#1891
jtopjian merged 2 commits intogophercloud:masterfrom
kayrus:volume-bootable-action

Conversation

@kayrus
Copy link
Copy Markdown
Contributor

@kayrus kayrus commented Mar 11, 2020

Resolves #1890

@coveralls
Copy link
Copy Markdown

coveralls commented Mar 11, 2020

Coverage Status

Coverage decreased (-0.0001%) to 77.116% when pulling a73b00f on kayrus:volume-bootable-action into 965816e on gophercloud:master.

@kayrus kayrus force-pushed the volume-bootable-action branch from dc7939a to a73b00f Compare March 11, 2020 14:46
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Mar 11, 2020

Build succeeded.

@kayrus
Copy link
Copy Markdown
Contributor Author

kayrus commented Mar 11, 2020

@jtopjian ready for review

@kayrus kayrus force-pushed the volume-bootable-action branch from a73b00f to d810494 Compare March 12, 2020 11:05
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Mar 12, 2020

Build succeeded.

@jtopjian
Copy link
Copy Markdown
Contributor

This looks good to me. Is there any way to implement a small/quick acceptance test to validate that the field has changed from true/false (or vice versa)?

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Mar 12, 2020

Build succeeded.

@kayrus
Copy link
Copy Markdown
Contributor Author

kayrus commented Mar 12, 2020

@jtopjian acceptance tests done

Copy link
Copy Markdown
Contributor

@jtopjian jtopjian left a comment

Choose a reason for hiding this comment

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

LGTM - thank you!

@jtopjian jtopjian merged commit 330a03d into gophercloud:master Mar 13, 2020
@kayrus kayrus deleted the volume-bootable-action branch March 13, 2020 06:01
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.

Cinder V3: Update a volume’s bootable status

3 participants