Skip to content

Compute v2: Clean up diskconfig package - #471

Merged
jrperritt merged 1 commit into
gophercloud:masterfrom
jtopjian:computev2-diskconfig-cleanup
Nov 4, 2017
Merged

jrperritt merged 1 commit into
gophercloud:masterfrom
jtopjian:computev2-diskconfig-cleanup

Conversation

@jtopjian

Copy link
Copy Markdown
Contributor

For #470
For #446

A unit test for the extension can be added after #466 is merged since it modifies the same area of code. I tested it locally and it works as expected.

@jtopjian
jtopjian requested a review from jrperritt August 14, 2017 01:28
@coveralls

coveralls commented Aug 14, 2017 •

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 71.518% when pulling 4ef61ad on jtopjian:computev2-diskconfig-cleanup into 8a1afad on gophercloud:master.

@jtopjian
jtopjian force-pushed the computev2-diskconfig-cleanup branch from 4ef61ad to 4e91496 Compare September 8, 2017 04:13
@coveralls

coveralls commented Sep 8, 2017 •

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 71.58% when pulling 4e91496 on jtopjian:computev2-diskconfig-cleanup into 2bf16b9 on gophercloud:master.

@jtopjian

jtopjian commented Sep 8, 2017

Copy link
Copy Markdown
Contributor Author

@jrperritt ping for review. Unit test has been added now that #466 was merged.

@jrperritt jrperritt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yep, looks good

@jrperritt
jrperritt merged commit 57a8169 into gophercloud:master Nov 4, 2017
@jtopjian
jtopjian deleted the computev2-diskconfig-cleanup branch April 24, 2018 03:17
cardoe pushed a commit to cardoe/gophercloud that referenced this pull request Aug 27, 2020
Fix OpenStack single volume attach example
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.

3 participants