Skip to content

switch bootfromvolume from os-volumes_boot to servers#1973

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
jichenjc:remove_os_volumes_boot
Jun 7, 2020
Merged

switch bootfromvolume from os-volumes_boot to servers#1973
jtopjian merged 1 commit intogophercloud:masterfrom
jichenjc:remove_os_volumes_boot

Conversation

@jichenjc
Copy link
Copy Markdown
Contributor

@jichenjc jichenjc commented Jun 1, 2020

according to
https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/routes.py#L726
os-volumes_boot is identical to servers routes and it will be deprecated later

https://docs.openstack.org/api-ref/compute/
os-volumes_boot is not listed at all

so switch from os-volumes_boot to servers

For #1974

@coveralls
Copy link
Copy Markdown

coveralls commented Jun 1, 2020

Coverage Status

Coverage remained the same at 79.379% when pulling c693a9c on jichenjc:remove_os_volumes_boot into d139d60 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 1, 2020

Build succeeded.

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 2, 2020

recheck stable/newton

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 2, 2020

Build failed.

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 2, 2020

recheck stable/ocata

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 2, 2020

Build failed.

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 2, 2020

recheck stable/pike

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 2, 2020

@jichenjc Thanks for submitting this. I'd like to make sure this change doesn't break with older releases of openstack, but it appears there's an issue with running these tests. I'll have to open an issue about this.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 2, 2020

Build failed.

@jichenjc
Copy link
Copy Markdown
Contributor Author

jichenjc commented Jun 2, 2020

thanks a lot for the help, yes, I saw the test is not running well
let's see the result first about older openstack release, thanks

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 2, 2020

recheck stable/queens

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 2, 2020

recheck stable/rocky

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 2, 2020

recheck stable/stein

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 2, 2020

Build failed.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 2, 2020

Build failed.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 2, 2020

Build failed.

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 2, 2020

recheck stable/stein

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 3, 2020

Build failed.

@wangxiyuan
Copy link
Copy Markdown

recheck

@wangxiyuan
Copy link
Copy Markdown

recheck stable/stein

@wangxiyuan
Copy link
Copy Markdown

recheck stable/rocky

@wangxiyuan
Copy link
Copy Markdown

recheck stable/queens

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 3, 2020

Build failed.

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 3, 2020

@wangxiyuan It looks like tests for "stein" are correctly running. The test failure seems legitimate and something I can look into later. Everything before stein seems to not even start, though.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 3, 2020

Build failed.

@wangxiyuan
Copy link
Copy Markdown

recheck stable/pike

@wangxiyuan
Copy link
Copy Markdown

recheck stable/ocata

@wangxiyuan
Copy link
Copy Markdown

recheck stable/newton

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 3, 2020

Build failed.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 3, 2020

Build failed.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 3, 2020

Build failed.

@wangxiyuan
Copy link
Copy Markdown

@jtopjian devstack failed setup before rocky release. It needs more time to debug. I guess it related to somthing with python3 or ubuntu version adoption.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 3, 2020

Build failed.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 3, 2020

Build failed.

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 3, 2020

@wangxiyuan

devstack failed setup before rocky release. It needs more time to debug. I guess it related to somthing with python3 or ubuntu version adoption.

Thank you so much for looking into this. Please take all the time you need :)

@jichenjc
Copy link
Copy Markdown
Contributor Author

jichenjc commented Jun 4, 2020

@wangxiyuan thanks, yes, I think py2->3 and ubuntu upgrade might be a challange

as some older releases has been unsupported by community itself https://releases.openstack.org/
(from Ocata ) ,do we really need to run such test? @jtopjian

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 4, 2020

It looks like all jobs are being run on Ubuntu Bionic? If so, is it possible to have certain older jobs run on Ubuntu Xenial?

As for older jobs, I try to test with as many older releases as possible as that's what reflects real-world deployments. That's key for changes such as what this PR is doing. But if we have to drop support for some releases, then we can drop Ocata and Pike.

@wangxiyuan
Copy link
Copy Markdown

openlab has Ubuntu 16.04 node. It's called ubuntu-xenial. You should update the nodeset in .zuul.yaml to use it.

https://github.com/gophercloud/gophercloud/blob/master/.zuul.yaml#L23

Just override it in the job you want.

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 4, 2020

@wangxiyuan Thanks! I'm testing this in #1975

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 4, 2020

@jichenjc I've merged #1975 which should help fix the issue with the immediate test failures.

Can you rebase this PR off of master to pull in the new .zuul.yaml?

It's OK if the entire test suite fails. I just want to make sure that changing this URL isn't going to break support in older releases. From looking through the git history, this URL alias has been in place for a number of years, so we should be safe. I just want to double check.

Thank you for your patience with this.

according to
https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/routes.py#L726
os-volumes_boot is identical to servers routes and it will be deprecated later

https://docs.openstack.org/api-ref/compute/
os-volumes_boot is not listed at all

so switch from os-volumes_boot to servers
@jichenjc jichenjc force-pushed the remove_os_volumes_boot branch from c68c41e to c693a9c Compare June 5, 2020 03:29
@jichenjc
Copy link
Copy Markdown
Contributor Author

jichenjc commented Jun 5, 2020

@jtopjian thanks for the info and the fix for older releases

I rebased just now and hopefully it helps

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 5, 2020

Build failed.

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 5, 2020

recheck

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Jun 5, 2020

recheck stable/ocata

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 5, 2020

Build failed.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 5, 2020

Build succeeded.

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.

Looks like the relevant Boot From Volume tests are passing as far back as Ocata, so I think this should be safe to merge.

Thank you for your patience with this.

@jtopjian jtopjian merged commit 91a6271 into gophercloud:master Jun 7, 2020
@jichenjc
Copy link
Copy Markdown
Contributor Author

jichenjc commented Jun 8, 2020

thanks a lot for the help :)

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