| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-30 | |||
| 11:01:00 | sean-k-mooney | kashyap: correct | |
| 11:01:18 | sean-k-mooney | we do not want exisating vms to stop using cirrus automatically | |
| 11:01:23 | kashyap | sean-k-mooney: Sorry, if it was already discussed - but why not? | |
| 11:01:33 | kashyap | What does it break? | |
| 11:02:00 | sean-k-mooney | other then rhel6 we are not sure if it will break anything | |
| 11:02:45 | sean-k-mooney | you could try and test that. grab a windows image boot with cirrus use virt manager to change it and reboot | |
| 11:02:54 | kashyap | (I'm not sure even if RHEL6 actually breaks here. Needs to be tested.) | |
| 11:02:55 | sean-k-mooney | it proably will be fine | |
| 11:03:11 | kashyap | Yeah; needs testing. | |
| 11:03:22 | sean-k-mooney | kashyap: have you confirm if virtio works on all architectures | |
| 11:03:32 | sean-k-mooney | its the default on arm but powerpc defaults to vga | |
| 11:03:56 | sean-k-mooney | do we need to check the arch of the image | |
| 11:04:06 | sean-k-mooney | or just use virtio always | |
| 11:04:07 | kashyap | sean-k-mooney: "all" archs, I don't know. What do we care about? | |
| 11:04:28 | kashyap | For x86 and Arm, 'virtio' disply works fine | |
| 11:04:34 | sean-k-mooney | power8/9 aarch64 and x86_64 really | |
| 11:05:12 | kashyap | I can check for power8/9 - although I doubt how many non-topy deployments actually are there for it | |
| 11:52:07 | sean-k-mooney | kashyap: the power kvm ci should test that config for us | |
| 12:02:19 | kashyap | sean-k-mooney: I see; is the Power KVM CI is maintianed? | |
| 12:03:07 | sean-k-mooney | its a requirement for in tree dirver to remain so i hope so | |
| 12:03:11 | sean-k-mooney | i think it is | |
| 12:20:11 | kashyap | Ack | |
| 13:10:03 | ganso | bauzas, lyarwood: hi! when you have a minute could you please take a look at the train backport now (it is the same one as yesterday): https://review.opendev.org/c/openstack/nova/+/798717 ? Thanks in advance | |
| 13:30:25 | dansmith | sean-k-mooney: kashyap I thought you were already talking about annotating instances to avoid changing the default on existing ones, but with inverse logic and a waiting cycle | |
| 13:30:51 | dansmith | I definitely think that when we can avoid switching the ABI on an instance suddenly without a lot of other work, we should do that | |
| 13:31:21 | dansmith | vmware maintains a hardware reference version and while it prompts you after every upgrade to upgrade the guest ABI, it doesn't force you (until it's very old) | |
| 13:31:51 | dansmith | given how hard it might be to rescue a cloud instance with broken video, and how easy it is to avoid it and not even require a waiting cycle, I think it's worth doing | |
| 13:32:05 | kashyap | dansmith: Hi, reading back | |
| 13:32:21 | kashyap | dansmith: I'm a bit foggy here, what's the "inverse logic" referred to here? | |
| 13:32:42 | kashyap | dansmith: Nothing is "settled" here, BTW. I'm going to do a bunch of tests to come back with concrete results | |
| 13:33:02 | dansmith | IIUC the original proposal was to annotate all instances as "old video" in this release, so that next release we could switch anything not annotated to the new video | |
| 13:33:11 | dansmith | that's the opposite of what sean-k-mooney was describing from our conversation above | |
| 13:33:35 | kashyap | I see | |
| 13:33:37 | dansmith | kashyap: sure, but you won't be able to test every type of guest with every config, so just not changing the ABI is the safest thing, IMHO | |
| 13:34:58 | kashyap | dansmith: Yeah; I agree. I just wanted to get a sense of what breaks, if anything. So if I got you right: you're indeed saying to have a deprecation cycle - yes? | |
| 13:35:07 | dansmith | no | |
| 13:35:21 | dansmith | I'm saying if we invert the logic where we would need one, we can get away without one | |
| 13:35:22 | sean-k-mooney | dansmith: yes i was suggesting annotating the instance so they dont change | |
| 13:35:39 | dansmith | ack | |
| 13:35:54 | sean-k-mooney | so if the image meta is not set when we do a hard reboot use old logic then recored it so next time we just use teh db value | |
| 13:36:15 | dansmith | and record the new model on ambiguous new boots, so they take the new default | |
| 13:36:19 | sean-k-mooney | whihc means the curernt logic in the vidoe add funciton remains the same | |
| 13:36:25 | sean-k-mooney | yep | |
| 13:36:37 | kashyap | sean-k-mooney: dansmith: Okay; can you please summarize your thinking here on this DNM draft? - https://review.opendev.org/c/openstack/nova/+/798680 | |
| 13:37:20 | sean-k-mooney | am sure although ill do it in a few minutes want to finish looking at downstream case first | |
| 13:37:27 | kashyap | No rush. | |
| 13:37:40 | kashyap | Thanks, both. | |
| 13:37:47 | dansmith | cool, sean-k-mooney let me know if you want me to chime in there, else I'll leave it to you | |
| 13:38:59 | sean-k-mooney | kashyap: fyi the current patch would not have actully chaged the default the way you were hoping it would | |
| 13:39:29 | kashyap | sean-k-mooney: Yeah, it's a naive draft, to make sure I don't forget this thing :) | |
| 13:40:12 | sean-k-mooney | kashyap: at lest not consitently https://github.com/openstack/nova/blob/25e218484990b41485973fab86adf5afc21dd476/nova/virt/libvirt/driver.py#L5912-L5951 would have altered it in many cases. | |
| 13:40:15 | sean-k-mooney | ack | |
| 13:41:36 | kashyap | sean-k-mooney: Right; if we wanted to blanket-force for "arches we care about" I had to add an if conditional for x86 in _add_video_driver() | |
| 13:43:26 | kashyap | (AArch64 already defaults to 'virtio' in Nova) | |
| 16:37:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | |
| 16:37:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 16:37:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 16:40:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 16:40:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 19:41:51 | noonedeadpunk | \o/ | |
| 19:42:14 | noonedeadpunk | folks, any chance to tag osc-placementclient with 3.0.1 on https://opendev.org/openstack/osc-placement/commit/281ace20df8bd01f4cb8ae29c39d80e754a9d0a3 ? | |
| 19:42:27 | noonedeadpunk | * osc-placement | |
| 19:45:02 | opendevreview | Merged openstack/nova master: Make explicit the network backend used in the CI jobs https://review.opendev.org/c/openstack/nova/+/797641 | |
| 19:52:25 | noonedeadpunk | bauzas: ok, I just pushed https://review.opendev.org/c/openstack/releases/+/798951 to tag osc-placement (I guess you're release liason?) | |
| #openstack-nova - 2021-07-01 | |||
| 00:30:02 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 01:31:46 | opendevreview | norman shen proposed openstack/nova master: Fix volume_id in connection info https://review.opendev.org/c/openstack/nova/+/798968 | |
| 07:00:39 | gibi | noonedeadpunk: I provided my +1 on the osc-placement release patch thanks for proposing it | |
| 07:05:30 | noonedeadpunk | gibi: oh, thanks! | |
| 07:55:36 | bauzas | noonedeadpunk: gibi: +1 for me as well | |
| 08:00:12 | gibi | #startmeeting nova_extra | |
| 08:00:12 | opendevmeet | Meeting started Thu Jul 1 08:00:12 2021 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 08:00:12 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 08:00:12 | opendevmeet | The meeting name has been set to 'nova_extra' | |
| 08:00:27 | gibi | new month new nova extra meeting | |
| 08:00:29 | gibi | :) | |
| 08:00:40 | gibi | who do we have here for today meeting? | |
| 08:01:05 | bauzas | \o | |
| 08:01:18 | bauzas | heh for the name | |
| 08:01:30 | gibi | yeah, lets wait a bit | |
| 08:01:33 | bauzas | nothing to tell for me | |
| 08:01:39 | gibi | I imagine :) | |
| 08:02:27 | gibi | finally we get some rain and cool air during the night so I was able to go for a run this morning. so I'm pretty chill at the moment. I can wait :) | |
| 08:04:11 | bauzas | I can send some rain for you if you wish | |
| 08:04:54 | gibi | interesting. we had a serious heatwave in the last ~2 weeks. | |
| 08:05:12 | gibi | but you live at a mountain isn't it? | |
| 08:06:07 | gibi | btw, if there is anybody here for the meeting please say so | |
| 08:06:24 | gibi | otherwise I will close the meeting logging in 4 mins | |
| 08:07:19 | bauzas | gibi: not exactly *within* the mountains, rather at the bottom in a valley :) | |
| 08:07:57 | bauzas | alt. of 230m :) | |
| 08:07:59 | gibi | ohh | |
| 08:08:13 | gibi | that is not that high :) | |
| 08:10:29 | gibi | I close this meeting due to lack of participation | |
| 08:10:36 | gibi | #endmeeting | |
| 08:10:36 | opendevmeet | Meeting ended Thu Jul 1 08:10:36 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 08:10:36 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova_extra/2021/nova_extra.2021-07-01-08.00.html | |
| 08:10:36 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova_extra/2021/nova_extra.2021-07-01-08.00.txt | |
| 08:10:36 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova_extra/2021/nova_extra.2021-07-01-08.00.log.html | |
| 08:13:01 | bauzas | gibi: remember which city you're living in Hungary ? Budapest ? | |
| 08:13:05 | gibi | yepp in the capital | |
| 08:13:22 | gibi | that is around 150m alt :) | |