| opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | 01:32 |
|---|---|---|
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | 01:32 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - create arqs https://review.opendev.org/c/openstack/nova/+/758944 | 01:32 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - build instance with smartnic arqs https://review.opendev.org/c/openstack/nova/+/798249 | 01:32 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | 01:32 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | 01:32 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | 01:32 |
| ralonsoh | sean-k-mooney, hi good morning! Just a heads-up: the idea for OVN (and OVS because is the same VIF type) has been approved. Well, at least the RFE. Now I need to push a spec. | 07:37 |
| ralonsoh | but the idea was understood and accepted | 07:37 |
| ralonsoh | context (for anyone else): https://bugs.launchpad.net/neutron/+bug/1933517 | 07:38 |
| opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | 07:54 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | 07:54 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - create arqs https://review.opendev.org/c/openstack/nova/+/758944 | 07:54 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - build instance with smartnic arqs https://review.opendev.org/c/openstack/nova/+/798249 | 07:54 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | 07:54 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | 07:54 |
| opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | 07:54 |
| yonglihe | Hi gibi, alex had finished the core logic review and +2 already. I split original 3rd patch to 3 one, that make review easier. Seems lots of review work, thanks. | 08:43 |
| kashyap | sean-k-mooney: I'll read back the discussion and respond. | 10:56 |
| sean-k-mooney | kashyap: tl;dr dansmith proposed a way that shoudl allow you to do the change of default this cycle by not changing the default in the code that currently sets it and instead populating the image metadata copy we have with the new default if its an intial boot | 10:58 |
| kashyap | sean-k-mooney: Also, I was wondering why not also set the default for x86 in _add_video_driver()? | 10:59 |
| kashyap | sean-k-mooney: The image metadat copy we have on initial boot: got a code pointer here? | 11:00 |
| sean-k-mooney | because we dont want to change existing image | 11:00 |
| kashyap | sean-k-mooney: So you're saying: we don't want to change the default for existing images on cold-boot? | 11:00 |
| sean-k-mooney | what we can do outside of _add_video_driver is update the exisitng vms to store what they have | 11:00 |
| sean-k-mooney | kashyap: correct | 11:01 |
| sean-k-mooney | we do not want exisating vms to stop using cirrus automatically | 11:01 |
| kashyap | sean-k-mooney: Sorry, if it was already discussed - but why not? | 11:01 |
| kashyap | What does it break? | 11:01 |
| sean-k-mooney | other then rhel6 we are not sure if it will break anything | 11:02 |
| 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 |
| kashyap | (I'm not sure even if RHEL6 actually breaks here. Needs to be tested.) | 11:02 |
| sean-k-mooney | it proably will be fine | 11:02 |
| kashyap | Yeah; needs testing. | 11:03 |
| sean-k-mooney | kashyap: have you confirm if virtio works on all architectures | 11:03 |
| sean-k-mooney | its the default on arm but powerpc defaults to vga | 11:03 |
| sean-k-mooney | do we need to check the arch of the image | 11:03 |
| sean-k-mooney | or just use virtio always | 11:04 |
| kashyap | sean-k-mooney: "all" archs, I don't know. What do we care about? | 11:04 |
| kashyap | For x86 and Arm, 'virtio' disply works fine | 11:04 |
| sean-k-mooney | power8/9 aarch64 and x86_64 really | 11:04 |
| kashyap | I can check for power8/9 - although I doubt how many non-topy deployments actually are there for it | 11:05 |
| sean-k-mooney | kashyap: the power kvm ci should test that config for us | 11:52 |
| kashyap | sean-k-mooney: I see; is the Power KVM CI is maintianed? | 12:02 |
| sean-k-mooney | its a requirement for in tree dirver to remain so i hope so | 12:03 |
| sean-k-mooney | i think it is | 12:03 |
| kashyap | Ack | 12:20 |
| 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:10 |
| *** ministry is now known as __ministry | 13:10 | |
| 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 |
| 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:30 |
| 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 |
| 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:31 |
| kashyap | dansmith: Hi, reading back | 13:32 |
| kashyap | dansmith: I'm a bit foggy here, what's the "inverse logic" referred to here? | 13:32 |
| kashyap | dansmith: Nothing is "settled" here, BTW. I'm going to do a bunch of tests to come back with concrete results | 13:32 |
| 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 |
| dansmith | that's the opposite of what sean-k-mooney was describing from our conversation above | 13:33 |
| kashyap | I see | 13:33 |
| 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:33 |
| 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:34 |
| dansmith | no | 13:35 |
| dansmith | I'm saying if we invert the logic where we would need one, we can get away without one | 13:35 |
| sean-k-mooney | dansmith: yes i was suggesting annotating the instance so they dont change | 13:35 |
| dansmith | ack | 13:35 |
| 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:35 |
| dansmith | and record the new model on ambiguous new boots, so they take the new default | 13:36 |
| sean-k-mooney | whihc means the curernt logic in the vidoe add funciton remains the same | 13:36 |
| sean-k-mooney | yep | 13:36 |
| 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:36 |
| sean-k-mooney | am sure although ill do it in a few minutes want to finish looking at downstream case first | 13:37 |
| kashyap | No rush. | 13:37 |
| kashyap | Thanks, both. | 13:37 |
| dansmith | cool, sean-k-mooney let me know if you want me to chime in there, else I'll leave it to you | 13:37 |
| sean-k-mooney | kashyap: fyi the current patch would not have actully chaged the default the way you were hoping it would | 13:38 |
| kashyap | sean-k-mooney: Yeah, it's a naive draft, to make sure I don't forget this thing :) | 13:39 |
| 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 |
| sean-k-mooney | ack | 13:40 |
| 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:41 |
| kashyap | (AArch64 already defaults to 'virtio' in Nova) | 13:43 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | 16:37 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | 16:37 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | 16:37 |
| 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 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | 16:40 |
| noonedeadpunk | \o/ | 19:41 |
| noonedeadpunk | folks, any chance to tag osc-placementclient with 3.0.1 on https://opendev.org/openstack/osc-placement/commit/281ace20df8bd01f4cb8ae29c39d80e754a9d0a3 ? | 19:42 |
| noonedeadpunk | * osc-placement | 19:42 |
| opendevreview | Merged openstack/nova master: Make explicit the network backend used in the CI jobs https://review.opendev.org/c/openstack/nova/+/797641 | 19:45 |
| noonedeadpunk | bauzas: ok, I just pushed https://review.opendev.org/c/openstack/releases/+/798951 to tag osc-placement (I guess you're release liason?) | 19:52 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!