| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-05 | |||
| 11:52:46 | lyarwood | so yeah default to volume if --image is provided | |
| 11:52:52 | lyarwood | and destination_type isn't | |
| 11:52:56 | stephenfin | they're all optional except UUID | |
| 11:53:12 | lyarwood | and reject the use of image as a destination_type if --image is already provided in the cli | |
| 11:53:20 | stephenfin | of local you mean | |
| 11:53:25 | lyarwood | yeah sorry | |
| 11:53:27 | lyarwood | local | |
| 11:53:35 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add device event constants to fakelibvirt https://review.opendev.org/c/openstack/nova/+/778887 | |
| 11:53:45 | stephenfin | --image + --block-device source_type=image,destination_type=local ==> FAIL | |
| 11:54:02 | lyarwood | I can add a note in the api docs about this | |
| 11:54:11 | stephenfin | actually, for OSC I might just leave this to the server | |
| 11:54:16 | stephenfin | as you suggested in the review | |
| 11:54:39 | lyarwood | ah damn past me | |
| 11:54:42 | stephenfin | simply using a sensible destination_type (volume) would be good | |
| 11:54:46 | lyarwood | but that makes sense | |
| 11:54:46 | stephenfin | enough | |
| 11:54:51 | lyarwood | yeah | |
| 12:26:50 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Replace grenade and nova-grenade-multinode with grenade-multinode https://review.opendev.org/c/openstack/nova/+/778885 | |
| 12:27:19 | lyarwood | ^ tosky / gmann ; I'm not sure what you both think about this approach but it could allow us to finish our move to zuulv3 before M3 | |
| 12:27:56 | lyarwood | tl;dr - move everything to grenade-multinode and add coverage for ceph later once that multinode base job is finished | |
| 12:55:07 | sean-k-mooney | lyarwood: that wont actully remove grenade unless we remove it form the integrated-gate-compute template | |
| 12:55:42 | sean-k-mooney | it wil still be added by integrated-gate-compute and it will no longer use the irrelevant-files list | |
| 12:56:02 | sean-k-mooney | so what you really need to do is also remove the integrated-gate-compute template | |
| 12:56:51 | sean-k-mooney | oh never mind | |
| 12:57:04 | sean-k-mooney | https://github.com/openstack/tempest/blob/master/zuul.d/integrated-gate.yaml#L442-L454 the patch to remvoe grenade form that has been merged | |
| 13:05:19 | gmann | yeah that is already done | |
| 13:05:32 | gmann | lyarwood: ack, thanks, I will check during weekend or on Monday | |
| 13:06:28 | lyarwood | sean-k-mooney: it's already removed from that template | |
| 13:06:35 | lyarwood | sean-k-mooney: yeah | |
| 13:07:16 | tosky | lyarwood: that's entirely up to what you (nova) think it's better | |
| 13:08:28 | lyarwood | I'd rather avoid duplication and move to zuulv3 now instead of waiting to get the ceph coverage in place | |
| 13:09:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Remove nova-dsvm-multinode-base https://review.opendev.org/c/openstack/nova/+/778908 | |
| 13:09:48 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Create and use nova-libvirt-irrelevant-files https://review.opendev.org/c/openstack/nova/+/778909 | |
| 13:11:53 | stephenfin | lyarwood: Low priority, but you may like 'Either a URI-style path (\'file:\\\\{path}\') to a JSON file ' | |
| 13:11:55 | stephenfin | whoops | |
| 13:12:01 | stephenfin | https://review.opendev.org/c/openstack/python-openstackclient/+/778910 | |
| 13:13:12 | lyarwood | stephenfin: nice idea | |
| 13:13:35 | stephenfin | I stole it https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/block-device-mapping-concepts.html | |
| 13:13:47 | stephenfin | (search for --block-device-mappings) | |
| 13:18:51 | sean-k-mooney | speaking of osc https://review.opendev.org/c/openstack/python-openstackclient/+/778571 | |
| 13:18:54 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Create and use nova-libvirt-irrelevant-files https://review.opendev.org/c/openstack/nova/+/778909 | |
| 13:18:56 | sean-k-mooney | that is for vdpa | |
| 13:19:27 | sean-k-mooney | stephenfin: care to take a look i dont know if i need a release note for that or anything else | |
| 13:19:40 | sean-k-mooney | stephenfin: i just did then minium i needed to not have to use curl | |
| 13:20:30 | stephenfin | sean-k-mooney: will do | |
| 13:20:45 | stephenfin | that's merged in neutron now? | |
| 13:20:53 | sean-k-mooney | yep | |
| 13:20:56 | stephenfin | cool | |
| 13:21:06 | stephenfin | will review fully after lunch but a small release note would be A+ | |
| 13:21:21 | stephenfin | and a link to the neutron change that added it (for context) | |
| 13:21:43 | sean-k-mooney | sure ill see if i can do that later today | |
| 13:30:11 | openstackgerrit | Merged openstack/nova master: tests: Stop mocking 'nova.virt.libvirt.utils.get_arch' https://review.opendev.org/c/openstack/nova/+/778879 | |
| 13:41:07 | tosky | lyarwood: in that patch, maybe you can also emove nova-dsvm-multinode-base | |
| 13:41:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move instance power state check to _detach_with_retry https://review.opendev.org/c/openstack/nova/+/778918 | |
| 13:42:28 | lyarwood | tosky: I've done that in the following change | |
| 13:55:29 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Placeholders for DB migration backports to Wallaby https://review.opendev.org/c/openstack/nova/+/778923 | |
| 13:55:39 | bauzas | gibi: stephenfin: sorry, was at some gym, here is the wallaby db placeholders change ^ | |
| 13:55:49 | bauzas | lemme look when we merged the other ones | |
| 13:55:53 | gibi | bauzas: thanks | |
| 13:56:30 | bauzas | gibi: IIRC, we wanted to have the placeholders *before* any other db migration for a cycle | |
| 13:56:37 | gibi | yes | |
| 13:56:48 | gibi | so if we merge it at the start of the cycle then we are goo | |
| 13:56:49 | gibi | d | |
| 13:56:57 | bauzas | like, you write a db migration for wallaby, you need to have the placeholders *before* | |
| 13:57:17 | bauzas | sure, but then we are lucky to not have any merged db migration during this cycle | |
| 13:57:37 | bauzas | my point is, we should backport this sooner than later and write another change for reserving the xena ones early in the cycle | |
| 13:57:53 | bauzas | don't we have this in the ptl guide or something like it ? | |
| 13:58:22 | bauzas | https://review.opendev.org/c/openstack/nova/+/686411 is the Train one | |
| 13:58:36 | gibi | bauzas: post release check list https://wiki.openstack.org/wiki/Nova/ReleaseChecklist | |
| 13:59:34 | gibi | linked in the https://docs.openstack.org/nova/latest/contributor/ptl-guide.html | |
| 13:59:38 | bauzas | the above change was merged on oct 4th | |
| 13:59:49 | bauzas | (2019) | |
| 13:59:56 | bauzas | and the train timeline was https://releases.openstack.org/train/schedule.html | |
| 14:00:08 | gibi | according to git it was included from ussuri | |
| 14:00:09 | sean-k-mooney | bauzas: so for xena i think stephenfin wants to move to alemnbic | |
| 14:00:22 | sean-k-mooney | so that might be different in the future | |
| 14:00:29 | bauzas | gibi: /me facepalms | |
| 14:00:38 | sean-k-mooney | not sure if we still need to do reservations the same way | |
| 14:00:40 | bauzas | so, yeah, this was merged early in the cycle, not late | |
| 14:00:56 | gibi | bauzas: hm, I think ti was merged after RC1 and therefore after we branched out stable/train | |
| 14:01:05 | bauzas | sean-k-mooney: we'll open the xena cycle before the PTG, we need the placeholders sooner than later | |
| 14:01:33 | bauzas | gibi: sure, but at this point in time, the ussuri placeholders change was merged when we did the master branch to ussuri | |
| 14:01:33 | sean-k-mooney | bauzas: sure just pointing out thye may not be used | |
| 14:01:41 | gibi | bauzas: correct | |
| 14:01:41 | sean-k-mooney | that said they dont hurt to have really | |
| 14:01:49 | gibi | bauzas: so it is only in ussuri but not in train | |
| 14:02:08 | bauzas | gibi: correct, that's a PEBKAC when I said Train | |
| 14:02:37 | bauzas | oh wai | |
| 14:02:40 | gibi | sean-k-mooney: it is like we don't know yet how the alembic migration will look like so I think we have to do the safe move and add reservation | |
| 14:02:43 | bauzas | man, this is confusing | |
| 14:02:47 | bauzas | https://review.opendev.org/c/openstack/nova/+/686411/2/nova/tests/functional/db/api/test_migrations.py | |
| 14:03:08 | bauzas | this was merged just after the branch but we name them train migrations | |
| 14:03:29 | bauzas | so, hence my confusion, placeholders go last | |
| 14:03:39 | bauzas | and just after we cut | |
| 14:03:40 | gibi | yeah it is like placeholder for migration backported from ussuri to train | |
| 14:03:47 | gibi | so both name can be correct :) | |
| 14:03:48 | bauzas | gibi: so you were right, we should hold until the branhc | |
| 14:03:58 | sean-k-mooney | gibi: yep we should for that reason and because whilte stephenfin want to do the move next cycle something might come up which means we cant finish it | |
| 14:04:10 | sean-k-mooney | so we need to prepare for backport in anycase | |
| 14:04:13 | gibi | sean-k-mooney: correct | |