Skip to content

Compute v2: Flavor Access Remove#688

Merged
jrperritt merged 2 commits intogophercloud:masterfrom
jtopjian:computev2-flavor-access-remove
Jan 19, 2018
Merged

Compute v2: Flavor Access Remove#688
jrperritt merged 2 commits intogophercloud:masterfrom
jtopjian:computev2-flavor-access-remove

Conversation

@jtopjian
Copy link
Copy Markdown
Contributor

For #506

Code and reference is listed in #506

@jtopjian jtopjian changed the title [pending #687] Computev2 flavor access remove [pending #687] Compute v2: Flavor Access Remove Dec 21, 2017
@jtopjian jtopjian mentioned this pull request Dec 21, 2017
3 tasks
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 21, 2017

Coverage Status

Coverage increased (+0.03%) to 72.858% when pulling 418fb7b on jtopjian:computev2-flavor-access-remove into cf81d92 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Dec 21, 2017

@jtopjian jtopjian force-pushed the computev2-flavor-access-remove branch from 418fb7b to 12f9b93 Compare December 21, 2017 07:17
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 21, 2017

Coverage Status

Coverage increased (+0.03%) to 72.858% when pulling 12f9b93 on jtopjian:computev2-flavor-access-remove into cf81d92 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Dec 21, 2017

@jtopjian jtopjian force-pushed the computev2-flavor-access-remove branch from 12f9b93 to 676f4c6 Compare December 21, 2017 22:15
@jtopjian jtopjian changed the title [pending #687] Compute v2: Flavor Access Remove Compute v2: Flavor Access Remove Dec 21, 2017
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 21, 2017

Coverage Status

Coverage increased (+0.004%) to 72.858% when pulling 676f4c6 on jtopjian:computev2-flavor-access-remove into 7b1b877 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Dec 21, 2017

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 21, 2017

Coverage Status

Coverage increased (+0.004%) to 72.858% when pulling 2030088 on jtopjian:computev2-flavor-access-remove into 7b1b877 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Dec 21, 2017

@jtopjian jtopjian force-pushed the computev2-flavor-access-remove branch from 2030088 to 391f1a3 Compare December 22, 2017 03:11
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.004%) to 72.925% when pulling 391f1a3 on jtopjian:computev2-flavor-access-remove into 05116c7 on gophercloud:master.

1 similar comment
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.004%) to 72.925% when pulling 391f1a3 on jtopjian:computev2-flavor-access-remove into 05116c7 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Dec 22, 2017

@jtopjian
Copy link
Copy Markdown
Contributor Author

@dklyle @jrperritt this is ready for review for whenever someone has time.

Copy link
Copy Markdown
Contributor

@dklyle dklyle left a comment

Choose a reason for hiding this comment

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

Covers the API functionality, and the function renames are included. LGTM.

@jtopjian
Copy link
Copy Markdown
Contributor Author

@dklyle Thanks!

@jrperritt can you do a quick review when you have a moment? I don't want to jinx myself again by thinking it's good to go :)

@jrperritt
Copy link
Copy Markdown
Contributor

Yes. I plan to go through all the open PRs today

@jrperritt jrperritt merged commit 8a6dfa8 into gophercloud:master Jan 19, 2018
@jtopjian jtopjian deleted the computev2-flavor-access-remove branch April 24, 2018 03:15
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.

4 participants