| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-23 | |||
| 14:52:07 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: demo 2 https://review.opendev.org/c/openstack/nova/+/871481 | |
| 14:52:08 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: demo middle https://review.opendev.org/c/openstack/nova/+/871482 | |
| 15:58:59 | sahid | o/ - quick question I have to rebase on conflict a micro-version change, I have some tests failing but struggling to find the issue https://paste.ubuntu.com/p/RNw8yxQtj6/ | |
| 15:59:05 | sahid | any idea that can help me? | |
| 16:01:34 | sahid | Template: ^2.95$ | |
| 16:01:36 | sahid | Sample: 2.94 | |
| 16:01:49 | sahid | I don't see where this 2.94 is comming from | |
| 16:57:55 | bauzas | sahid: are you sure you updated all your API change files so it now uses 2.95 ? | |
| 17:04:37 | bauzas | sahid: commented your patch | |
| 17:34:03 | sahid | bauzas: well I want to say yes but I have probably missed something :-) | |
| 17:34:10 | sahid | I will double check | |
| 17:34:45 | bauzas | sahid: I tried to quick looked at the test to understand why it autogenerates 2.94 | |
| 17:34:53 | bauzas | to quickly look* | |
| 17:35:11 | bauzas | but I haven't found a lot, if you still have the problem, you could try to pdb it | |
| 17:35:28 | bauzas | to find how it generates this template | |
| 17:35:50 | bauzas | if you can't, I can offer my help tomorrow | |
| 17:37:36 | sahid | no worries thanks to have looked at it. I will double check that tomorrow as-well, don't spend time on it I will ping you when it's ready :-) | |
| 18:28:21 | opendevreview | Sofia Enriquez proposed openstack/nova master: WIP: Implement encryption on backingStore https://review.opendev.org/c/openstack/nova/+/870012 | |
| 18:53:25 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| #openstack-nova - 2023-01-24 | |||
| 08:50:59 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 10:28:53 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: At start-up rework compareCPU() usage with a workaround https://review.opendev.org/c/openstack/nova/+/870794 | |
| 10:28:54 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | |
| 10:29:24 | kashyap | sean-k-mooney: --^ Good catch in the workaround patch; addressed your remarks. When you can, lemme know if that looks fine :) | |
| 10:31:04 | sean-k-mooney[m] | sure ill review it now | |
| 10:31:45 | kashyap | sean-k-mooney: Also: the bug reporter from the downstream also tested it on a real machine and it works | |
| 10:33:21 | sean-k-mooney[m] | nice am if you respin add a release note but im +2 on it | |
| 10:43:00 | kashyap | sean-k-mooney: I'm not sure if this warrants a release note? I don't mind adding, though. Thank you! | |
| 10:43:47 | kashyap | gibi: --^ Please have a gander when you can (the WA patch) | |
| 10:44:04 | sean-k-mooney[m] | i like to have release notes for most of the changes we make that might be of interest to operators | |
| 10:44:08 | kashyap | sean-k-mooney: The API replacement patch lost your +2, can you also re-look at it when you can? - https://review.opendev.org/c/openstack/nova/+/869950/9 | |
| 10:44:19 | kashyap | (And also it lost +W due to rebase) | |
| 10:44:32 | sean-k-mooney[m] | sure i tought it still had it when i looked but ill look again | |
| 10:44:52 | kashyap | Thx! | |
| 10:45:34 | sean-k-mooney[m] | +2 was still there it lost +w | |
| 10:45:52 | sean-k-mooney[m] | i have added that but its obviously pendeing the first change having both | |
| 10:50:09 | sean-k-mooney[m] | ok i think its time for coffee brb | |
| 10:55:55 | kashyap | Yeah, it won't merge anyway until the predicated patch is merged | |
| 11:00:33 | ratailor | sean-k-mooney[m], gibi Could you please review https://review.opendev.org/c/openstack/nova/+/852737 and https://review.opendev.org/c/openstack/nova/+/861738 | |
| 11:09:55 | sean-k-mooney | ratailor: i started looking at that yesterday actully i just got pulled into other discussions | |
| 11:10:28 | ratailor | sean-k-mooney, ack. np. Thanks! | |
| 11:18:58 | opendevreview | Merged openstack/nova stable/zed: Correct config help message related options https://review.opendev.org/c/openstack/nova/+/871247 | |
| 11:45:42 | opendevreview | Merged openstack/nova stable/train: func: Introduce a server_expected_state kwarg to InstanceHelperMixin._live_migrate https://review.opendev.org/c/openstack/nova/+/865382 | |
| 12:14:24 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 13:04:44 | sean-k-mooney | bauzas: ^ can you look at shaid's seriese i think its ready to go | |
| 13:05:05 | sean-k-mooney | zuul is still running on the last revision but the func test fix was trivial so i expect it to pass | |
| 13:08:38 | sean-k-mooney | dansmith: can you send this on its way https://review.opendev.org/c/openstack/nova/+/865071 that the last patch for the new defaults | |
| 13:29:24 | bauzas | sean-k-mooney: was on my todolist | |
| 13:30:08 | bauzas | btw. thanks for the first thoughts on my series | |
| 13:30:51 | sean-k-mooney | more or less it made sense to me | |
| 13:31:03 | sean-k-mooney | i just needed to figure out how you had it split up | |
| 13:32:24 | bauzas | sean-k-mooney : the 3rd patch was too large for the CI | |
| 13:32:31 | bauzas | hence the split | |
| 13:33:12 | sean-k-mooney | so i can generate a smaller set of files for you if you like | |
| 13:33:16 | sean-k-mooney | many of the files in that are not used | |
| 13:33:47 | sean-k-mooney | if you look you are only using a small subset of sysfs currenly | |
| 13:34:02 | sean-k-mooney | and most of the fiels are symlinks | |
| 13:35:48 | bauzas | yep I know | |
| 13:36:15 | bauzas | like in general sysfs btw ;-) | |
| 13:36:47 | sean-k-mooney | yep | |
| 13:37:00 | sean-k-mooney | it took a while to figureout how to make a copy that worked the same | |
| 13:37:41 | sean-k-mooney | if i rememebr correctly my first attempt flatened the symlinks and resulted in things getting out of sync | |
| 14:04:41 | opendevreview | Jorge San Emeterio proposed openstack/nova master: [DNM] Testing effects on privsep on a build. https://review.opendev.org/c/openstack/nova/+/871607 | |
| 14:05:01 | opendevreview | Jorge San Emeterio proposed openstack/nova master: [DNM] Testing effects on privsep on a build. https://review.opendev.org/c/openstack/nova/+/871607 | |
| 14:56:18 | dansmith | sean-k-mooney: yep | |
| 15:01:10 | opendevreview | Dan Smith proposed openstack/nova master: Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871612 | |
| 15:01:45 | opendevreview | Dan Smith proposed openstack/nova master: Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871612 | |
| 15:02:19 | opendevreview | Dan Smith proposed openstack/nova stable/zed: Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871616 | |
| 15:03:09 | opendevreview | Dan Smith proposed openstack/nova stable/xena: Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871622 | |
| 15:03:39 | opendevreview | Dan Smith proposed openstack/nova stable/yoga: Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871624 | |
| 15:03:43 | dansmith | bauzas: your help on these quickly is appreciated ^ | |
| 15:05:16 | bauzas | dansmith: kk, I've seen it before | |
| 15:06:08 | bauzas | sean-k-mooney: dansmith: I have a bit thoughts on upgrades for sahid's new API microversion https://review.opendev.org/c/openstack/nova/+/858384 | |
| 15:06:26 | bauzas | dansmith: +2d the master one | |
| 15:12:33 | sean-k-mooney | evacuate will work but not with the latest microverion until you are fully upgraded | |
| 15:13:05 | sean-k-mooney | so the only way around that is a new paramater in the api | |
| 15:13:18 | bauzas | sean-k-mooney: I know, see my comments | |
| 15:13:34 | dansmith | wow, that patch | |
| 15:13:35 | bauzas | sean-k-mooney: my only comment is to say we should at least explain it more | |
| 15:13:39 | sean-k-mooney | but that kind of defeats the idea of always pwowering off unless the new parmater default to powering off if not set | |
| 15:14:49 | sean-k-mooney | bauzas: is the concern mainly for nova-client | |
| 15:14:52 | bauzas | sean-k-mooney: sure, and again, I'm not *against* the new behaviour | |
| 15:15:16 | sean-k-mooney | openstack client need explcit opt in | |
| 15:15:24 | sean-k-mooney | baiscally im wondiering are you ok with just docs | |
| 15:15:26 | bauzas | sean-k-mooney: my only concern is that IMHO we don't really explain it correctly, at least we need an upgrade section | |
| 15:15:28 | sean-k-mooney | or do you want a design change | |
| 15:15:51 | sean-k-mooney | ok so you would like better docs and beeter release note to call that out | |
| 15:16:04 | bauzas | sean-k-mooney: at least I can accept the new behaviour, but I want to make sure that people know it | |
| 15:16:37 | bauzas | as you said, by default OSC will return an exception if you evacuate an instance on Antelope without upgrading all computes | |
| 15:16:46 | sean-k-mooney | no | |
| 15:16:50 | sean-k-mooney | by default it will evacuate | |
| 15:16:57 | sean-k-mooney | because it does not use the latest microverions | |
| 15:17:12 | sean-k-mooney | its only an issue for nova client which woudl use the latest microversion automatically | |
| 15:17:15 | bauzas | I thought that now OSC defaults to the latest | |
| 15:17:18 | sean-k-mooney | nope | |
| 15:17:20 | sean-k-mooney | 2.1 | |
| 15:17:50 | bauzas | sean-k-mooney: this is an issue for any client asking the latest | |
| 15:17:53 | sean-k-mooney | if you want anything else you have to choose it by env var, cli flag or in your clouds.yaml | |
| 15:17:54 | bauzas | point. | |
| 15:18:13 | bauzas | hence me saying I'm OK (again, see my comments) | |
| 15:18:14 | sean-k-mooney | bauzas: right which we tell them not to do to avoid this exact type of issue | |