| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-25 | |||
| 14:40:48 | whoami-rajat | ack, that will be helpful | |
| 15:01:11 | bauzas | ok, fwiw, starting to use the -2 hammer | |
| 15:08:05 | bauzas | gibi: I guess there is no procedural markup for the any-traits series ? | |
| 15:08:49 | gibi | bauzas: what do you mean by procedural markup? | |
| 15:08:51 | bauzas | oh shit, nevermind, found the story | |
| 15:09:06 | bauzas | gibi: I guess we can close https://storyboard.openstack.org/#!/story/2005345 ? | |
| 15:09:26 | gibi | yeeah | |
| 15:09:33 | bauzas | ok, doing it | |
| 15:09:42 | gibi | I thought my commit messages updated the story but it didn't | |
| 15:09:43 | gibi | :/ | |
| 15:09:53 | gibi | I will add a link to the story to point to the implementation topic | |
| 15:10:07 | gibi | hm | |
| 15:10:14 | gibi | the comments are there | |
| 15:10:20 | gibi | just no tasks were created | |
| 15:10:46 | gibi | ahh comments is only from the spec patch | |
| 15:10:55 | gibi | storyboard :/ | |
| 15:11:20 | bauzas | yeah :( | |
| 15:17:50 | dansmith | bauzas: are you starting the -2 hammer for features? | |
| 15:18:00 | bauzas | dansmith: I'm on it | |
| 15:18:07 | bauzas | just did the implemented scrubbing | |
| 15:18:17 | bauzas | now, next is the procedural hammer | |
| 15:18:25 | dansmith | okay | |
| 15:18:35 | dansmith | I guess that volume-backed-rebuild one is going to be off the table | |
| 15:18:47 | dansmith | I think it's missing osc support and tempest testing | |
| 15:18:51 | dansmith | bummer :( | |
| 15:19:17 | sean-k-mooney | osc support missed yes but is the api change merged? | |
| 15:19:30 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 15:19:31 | dansmith | bauzas: the splitting helped uncover that it was introducing a data loss bug for the old path, so I'm super glad we did that | |
| 15:19:35 | dansmith | sean-k-mooney: no | |
| 15:19:43 | bauzas | dansmith: hah, good to know | |
| 15:20:07 | sean-k-mooney | dansmith: ok then ya i guess that will miss | |
| 15:20:30 | sean-k-mooney | the cinder side is done? | |
| 15:23:29 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 15:24:49 | whoami-rajat | sean-k-mooney, yes, cinder side changes are merged | |
| 15:25:06 | whoami-rajat | bauzas, dansmith is there a chance we can get FFE for it? | |
| 15:25:30 | dansmith | whoami-rajat: up to bauzas but without osc and tempest yet it seems like a stretch :/ | |
| 15:26:01 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 15:26:12 | whoami-rajat | i think we already discussed the OSC thing above, tempest I will complete by monday for sure | |
| 15:26:33 | bauzas | whoami-rajat: I haven't planned any FFE yet | |
| 15:27:03 | dansmith | well, I'll review the latest set in a bit regardless | |
| 15:27:26 | bauzas | whoami-rajat: I can left dansmith to reviewing the series | |
| 15:27:40 | bauzas | whoami-rajat: but my concern is about the API microversion, it's a bit late | |
| 15:28:00 | dansmith | whoami-rajat: I really want to see tempest before I even +2, FWIW | |
| 15:28:08 | dansmith | for something like this that'd always be my plan | |
| 15:29:15 | whoami-rajat | ack, I'm still working on passing the reimage_boot_volume parameter from API->Compute, bumping up RPC versions for conductor, compute, tests for it etc that is more work than i anticipated | |
| 15:29:43 | whoami-rajat | almost done with it | |
| 15:30:11 | dansmith | whoami-rajat: ah, I thought that was done in the latest.. | |
| 15:30:26 | whoami-rajat | but again, I don't want to force any decisions, if the team doesn't want to include this as part of Yoga, we can continue work in Z | |
| 15:30:38 | whoami-rajat | dansmith, completed it in manager but the 3rd patch is getting larger and larger | |
| 15:30:45 | whoami-rajat | will update that in sometime | |
| 15:31:12 | whoami-rajat | need to deal with the breaking tests since there's a new parameter in like 10-15 methods | |
| 15:31:28 | whoami-rajat | so the 2nd patch is ready for review | |
| 15:31:32 | whoami-rajat | but 3rd needs some more work | |
| 15:32:57 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 15:34:14 | chateaulav | just running my one ci job, please dont think im trying to kill the pippeline. lol | |
| 15:36:07 | bauzas | whoami-rajat: we'll open the Zed release in two weeks | |
| 15:36:23 | bauzas | I mean, the master will be Zed in two weeks | |
| 15:36:47 | plibeau1 | gibi: Hello, I have reply on your question for this change: https://review.opendev.org/c/openstack/nova/+/803713 | |
| 15:36:55 | bauzas | whoami-rajat: so I propose you to provide again the spec for Zed as just a pre-approved spec | |
| 15:37:08 | bauzas | we could accept this spec quickly | |
| 15:37:24 | gibi | plibeau1: I've added your patch to my review queue | |
| 15:37:25 | bauzas | so we could just review your implementation changes again in less than one month | |
| 15:37:39 | plibeau1 | gibi: thx | |
| 15:37:40 | sean-k-mooney | chateaulav: no worries did you see my comment | |
| 15:37:54 | bauzas | gibi: do you know where I could find a document for the -2 comment ? :) | |
| 15:38:08 | bauzas | gibi: I mean, when I'll -2 the existing changes | |
| 15:38:13 | gibi | I don't think it is documented in the PTL guide | |
| 15:38:23 | chateaulav | sean-k-mooney: yeah it was acting weird the first couple times when i did just that, think i got it set right now | |
| 15:38:40 | gibi | I just used a sentece stating that we are in feature freeze | |
| 15:39:07 | sean-k-mooney | chateaulav: you still hae the templates mind if i quickly fix it for you | |
| 15:39:19 | bauzas | gibi: yeah, that's what I thought | |
| 15:39:24 | bauzas | gibi: I'm pretty sure I can find it | |
| 15:39:48 | kashyap | bauzas: Thx for the paperwork here - https://blueprints.launchpad.net/nova/+spec/virtio-as-default-display-device | |
| 15:39:51 | kashyap | ! | |
| 15:41:26 | opendevreview | sean mooney proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 15:41:47 | sean-k-mooney | chateaulav: ^ that should just run your job in check and check for merge conflicts | |
| 15:43:17 | sean-k-mooney | oh the alias | |
| 15:45:07 | opendevreview | sean mooney proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 15:47:23 | sean-k-mooney | chateaulav: if you want to keep an eye on it you can see it running here https://zuul.openstack.org/status#828372 | |
| 15:47:48 | whoami-rajat | bauzas, ack | |
| 15:57:16 | sean-k-mooney | im going to go to the store at the top of the hour then work form my laptop. when i get back so ill be off irc for a bit | |
| 16:14:11 | bauzas | gmann: melwitt: I trust you for rechecking again your already approved patches in case you have problems with the CI | |
| 16:14:42 | bauzas | I'll do a quick check on Monday morning my time to see if everything is merged | |
| 16:21:24 | gmann | bauzas: yeah. release time and CI will behave like that :) | |
| 16:26:34 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances https://review.opendev.org/c/openstack/nova/+/830883 | |
| 16:26:50 | whoami-rajat | dansmith, ^ done with final patch | |
| 16:27:01 | dansmith | ack | |
| 16:40:48 | opendevreview | Merged openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/c/openstack/nova/+/712141 | |
| 16:50:01 | bauzas | ok, I'm done for this (busy) week | |
| 16:50:05 | bauzas | see ya folks | |
| 16:56:52 | whoami-rajat | dansmith, hey, the OSC client rebuild call is calling novaclient (AFAICS) so we do need the change in novaclient as well right? https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server.py#L3541 | |
| 16:57:27 | whoami-rajat | or there will be a structural change proposal to remove dependency of novaclient somehow... | |
| 16:57:30 | dansmith | whoami-rajat: I'm not on top of the state of that stuff.. I expect there are a bunch of calls not converted to native that still use novaclient? not sure | |
| 16:57:51 | dansmith | but if rebuild is currently going through novaclient today, I doubt anyone is going to make you change that before you merge :) | |
| 16:59:07 | whoami-rajat | ack, I'm totally confused with the deprecating novaclient and proposing changes directly to OSC first then... | |
| 16:59:19 | whoami-rajat | sean-k-mooney, can you help with my query above? ^ | |
| 17:00:42 | dansmith | whoami-rajat: yeah just not sure, but you're right to be confused/frustrated.. I'll look for sean-k-mooney to answer aw well | |
| 17:00:43 | dansmith | *as | |
| 17:03:07 | whoami-rajat | good to know I'm not the only one :) | |
| 17:16:14 | whoami-rajat | so this is my patch for OSC but it won't work without the novaclient patch https://review.opendev.org/c/openstack/python-openstackclient/+/831014 | |