| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-01 | |||
| 16:43:12 | elodilles | https://releases.openstack.org/reference/release_models.html#independent | |
| 16:43:20 | elodilles | the definition ^^^ | |
| 16:43:27 | bauzas | damn, burned | |
| 16:43:34 | elodilles | :] | |
| 16:43:59 | sean-k-mooney | we currently use cycle-with-intermediary which works fine | |
| 16:44:20 | bauzas | well, then let's discuss this at the PTG | |
| 16:44:24 | sean-k-mooney | but most of the changes tend to be paper work | |
| 16:44:26 | sean-k-mooney | ack | |
| 16:44:39 | bauzas | enforcing a release per cycle is something a bit loud, but this could be necessary | |
| 16:45:04 | bauzas | releasing the enforcement doesn't sound bad at a first glance, but I wonder the implications | |
| 16:45:16 | elodilles | one significant thing is that os-vif won't have stable branches, | |
| 16:45:26 | elodilles | but i think not so much backports arrives there | |
| 16:45:34 | sean-k-mooney | elodilles: ya but we dont really do backports often | |
| 16:45:46 | elodilles | sean-k-mooney: ++ | |
| 16:45:53 | sean-k-mooney | we do bug i can proably count them on two hands for the last 2-3 releases | |
| 16:45:55 | bauzas | but then we loose the possibility of doing them :) | |
| 16:46:02 | sean-k-mooney | yep | |
| 16:46:07 | sean-k-mooney | lets talk about it later | |
| 16:46:10 | bauzas | yup | |
| 16:46:13 | sean-k-mooney | just wanted people to think about it | |
| 16:46:16 | bauzas | thanks | |
| 16:46:25 | bauzas | any other topic ? | |
| 16:46:54 | bauzas | sean-k-mooney: please put it somewhere in your brain until the ptg etherpad is created, I don't want to miss this conversation at the PTG | |
| 16:47:28 | bauzas | that said, if nobody has any items... | |
| 16:47:37 | bauzas | thanks all | |
| 16:47:42 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-03-01-16.00.log.html | |
| 16:47:42 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-03-01-16.00.txt | |
| 16:47:42 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-03-01-16.00.html | |
| 16:47:42 | opendevmeet | Meeting ended Tue Mar 1 16:47:42 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:47:42 | bauzas | #endmeeting | |
| 16:47:47 | elodilles | thanks bauzas o/ | |
| 16:47:51 | chateaulav | \o | |
| 16:48:04 | gibi | \o | |
| 16:48:44 | chateaulav | gibi: thanks as well, i appreciate all the help again! | |
| 16:48:59 | chateaulav | sean-k-mooney: you as well | |
| 16:50:21 | gibi | chateaulav: thanks for the work! | |
| 17:11:44 | sean-k-mooney | chateaulav: no worries. | |
| 17:16:19 | opendevreview | Dan Smith proposed openstack/nova master: Add conductor RPC interface for rebuild https://review.opendev.org/c/openstack/nova/+/831219 | |
| 17:16:19 | opendevreview | Dan Smith proposed openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368 | |
| 17:16:20 | opendevreview | Dan Smith proposed openstack/nova master: Add API support for rebuilding BFV instances https://review.opendev.org/c/openstack/nova/+/830883 | |
| 18:00:24 | dansmith | whoami-rajat: looks like there were more unit fails on the base patch.. I had only been going after the one that failed in the top patch | |
| 18:00:30 | dansmith | probably more of the same needed further down | |
| 18:04:59 | whoami-rajat | dansmith, those were the tests i was referring to, they should be fixed with a single line in the test conductor code | |
| 18:05:01 | whoami-rajat | https://review.opendev.org/c/openstack/nova/+/831219/10/nova/tests/unit/conductor/test_conductor.py | |
| 18:05:05 | whoami-rajat | L#389 | |
| 18:05:23 | dansmith | whoami-rajat: ah okay so you're on top of it? | |
| 18:06:01 | whoami-rajat | dansmith, i was skeptical about fixing conductor tests in the compute patch initially | |
| 18:06:08 | whoami-rajat | i can do that update if it sounds right | |
| 18:06:29 | dansmith | yeah, conductor calls compute rpc, so if compute rpc client changes, then those tests have to change to match | |
| 18:09:53 | whoami-rajat | dansmith, but we're fixing these tests because we added the conductor parameter (hack) which shouldn't have been there in the first place in the compute patch, also they're easier to fix since all rebuild args are placed at one place but we're fixing 18 tests because of a code change that shouldn't exist in this patch | |
| 18:10:00 | whoami-rajat | that's why i was skeptical about it ^ | |
| 18:10:29 | dansmith | I don't see it that way at all :) | |
| 18:10:57 | dansmith | the reason to break these things apart is to make sure each step works, because in reality, not all the services get upgraded at the same time in a real deployment, | |
| 18:11:36 | dansmith | so doing the steps like this help ensure that we're not just shifting the state of the world atomically, which is how we discovered that the behavior of rebuild had changed, when we separated the api patch from the implementation | |
| 18:12:13 | dansmith | it's not a "hack" to add a parameter that is the equivalent of what we have today as we step through those changes, it's just keeping things compatible | |
| 18:13:54 | whoami-rajat | I understand why we are doing it like this but we already have rpc versions to check and maintain that compatibility (although this provides more assurance) | |
| 18:14:24 | whoami-rajat | but again, in a real deployment as well, if the compute is new and conductor is old, we will see the same failure in the logs as these tests | |
| 18:14:30 | dansmith | but we had versions in the initial unified patch, which weren't properly honoring the old behavior :) | |
| 18:14:59 | dansmith | whoami-rajat: just FYI, conductor always has to be upgraded before compute, it's in our upgrade process | |
| 18:15:08 | dansmith | api and conductor together, compute after | |
| 18:18:10 | whoami-rajat | we weren't passing the reimage_boot_volume parameter down to the compute in the initial patch so there weren't the RPC bumps but i get what you're saying, we wouldn't have discovered the issue if it wasn't for the split | |
| 18:18:25 | whoami-rajat | maybe it's just new for me to split the different services code apart | |
| 18:18:32 | whoami-rajat | sorry for all the fuzz created :) | |
| 18:18:46 | dansmith | s'ok, let's just get it right :) | |
| 18:19:03 | whoami-rajat | ack, on it | |
| 18:19:21 | opendevreview | Merged openstack/nova master: Move file system freeze after end of mirroring https://review.opendev.org/c/openstack/nova/+/803713 | |
| 18:40:09 | chateaulav | gibi: im guessing for https://review.opendev.org/c/openstack/nova/+/828372 the -2 V is just an occasional hiccup with gate jobs | |
| 18:41:09 | elodilles | dansmith: when you have some time, can you please use your stable core power on this one-liner patch? :) https://review.opendev.org/c/openstack/nova/+/830656/ | |
| 18:41:21 | gibi | chateaulav: yepp, feel free to recheck | |
| 18:42:01 | chateaulav | cool | |
| 18:45:03 | dansmith | elodilles: done | |
| 18:47:58 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368 | |
| 18:47:59 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances https://review.opendev.org/c/openstack/nova/+/830883 | |
| 18:47:59 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild https://review.opendev.org/c/openstack/nova/+/831219 | |
| 18:48:26 | whoami-rajat | dansmith, instead of one, it was 3 line change but tests are passing now ^ | |
| 18:48:38 | dansmith | cool | |
| 18:58:14 | elodilles | dansmith: thanks! \o/ | |
| 20:28:45 | opendevreview | Merged openstack/nova stable/wallaby: skip test_tagged_attachment in nova-next https://review.opendev.org/c/openstack/nova/+/830656 | |
| 22:05:22 | opendevreview | Merged 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 | |
| #openstack-nova - 2022-03-02 | |||
| 00:34:10 | gmann | bauzas: gibi easy one to avoid grenade-skip job running on doc/test patches https://review.opendev.org/c/openstack/nova/+/831229 | |
| 00:34:29 | gmann | melwitt: sean-k-mooney ^^ if any one of you around | |
| 00:53:38 | melwitt | gmann: I'm curious why *nova-base-irrelevant-files and not *policies-irrelevant-files? shouldn't nova/policies/ changes be tested with the skip level upgrade job? | |
| 01:02:05 | melwitt | it also shows as merge conflict | |
| 06:48:21 | gibi | gmann, melwitt: I agree with melwitt, I think we need to trigger on policy changes too | |
| 07:47:46 | bauzas | gibi: gmann: melwitt: dansmith: me too | |
| 07:47:55 | bauzas | easy change FTW | |
| 09:28:10 | opendevreview | Merged openstack/nova stable/wallaby: Migrate RequestSpec.numa_topology to use pcpuset https://review.opendev.org/c/openstack/nova/+/827871 | |
| 10:32:18 | opendevreview | ribaudr proposed openstack/nova-specs master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova-specs/+/831506 | |
| 10:33:05 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | |
| 10:37:26 | Uggla | sean-k-mooney, I have just pushed the unshelve to host specs ans api changes. I'll push the clients part asap. | |
| 10:37:54 | Uggla | bauzas, fyi see msg just above. | |
| 10:45:27 | bauzas | Uggla: all good | |
| 10:47:06 | Uggla | bauzas, Should I remove the link to the bug ? As the bug in not a lunchpad one but a bz one. | |
| 10:47:44 | bauzas | Uggla: oh yes | |
| 10:48:02 | bauzas | Uggla: don't push any internal BZ upstream :p | |
| 10:48:12 | bauzas | not all of us are working on the same company :p | |
| 10:49:24 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | |
| 10:50:32 | Uggla | bauzas, updated | |
| 10:50:52 | Uggla | bauzas, should be clean now. | |
| 10:52:45 | bauzas | ++ | |