| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-30 | |||
| 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: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 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: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 | opendevmeet | The meeting name has been set to 'nova_extra' | |
| 08:00:12 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 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 | gibi | #startmeeting 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 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova_extra/2021/nova_extra.2021-07-01-08.00.log.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 | Minutes: https://meetings.opendev.org/meetings/nova_extra/2021/nova_extra.2021-07-01-08.00.html | |
| 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 | gibi | #endmeeting | |
| 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 :) | |
| 08:13:55 | bauzas | I knew about the heat wave, some influencers were traveling to your location during the last 2 weeks but suddently went back last Monday | |
| 08:14:15 | gibi | yeah there was some sport events | |
| 08:14:23 | bauzas | minor ones | |
| 08:14:46 | bauzas | apart from Le Tour de France, I don't think others are more important | |
| 08:17:34 | gibi | I'm not a sports fan so I only checked the results | |
| 08:18:17 | gibi | during the germany - hungary meeting I was sitting in a garden of one of my friends chilling and we just listened the noises of the suburb to judge the result of that meeting:) | |
| 08:18:26 | kashyap | bauzas: What do you mean "influencers"? | |
| 08:18:29 | kashyap | Morning, BTW | |
| 08:18:42 | bauzas | kashyap: this was a joke | |
| 08:19:02 | kashyap | Ah, I don't get it, then :) | |
| 08:19:23 | bauzas | kashyap: for explaining that sometimes I think some of our footballers are better with working for Instagram rather than playing with a ball and acting as a team | |
| 08:19:37 | gibi | :) | |
| 08:19:54 | kashyap | bauzas: Ah, "influencers" seems to be an Instagram keyword [I'm not on any social media for 8+ years now] | |
| 08:20:24 | bauzas | kashyap: not only Instagram but social medias in general | |
| 08:20:47 | bauzas | the fact that you have a community watching you and following your advices | |
| 09:51:14 | sean-k-mooney | stephenfin: so i have the os-vif patch mostly working now https://review.opendev.org/c/openstack/os-vif/+/798055 i need to rebase it on my patch to fix os-vif ci and add a release note and flip the default to false but can you give it a look and see what you think | |
| 09:51:23 | stephenfin | sure | |
| 09:51:34 | stephenfin | can do | |
| 09:57:08 | opendevreview | Stephen Finucane proposed openstack/nova master: Add --sleep option for archive_deleted_rows --until-complete https://review.opendev.org/c/openstack/nova/+/771749 | |
| 10:29:41 | opendevreview | Merged openstack/nova-specs master: Add attachmentId to responses of the os-volume_attachments API https://review.opendev.org/c/openstack/nova-specs/+/794799 | |
| 10:43:49 | lyarwood | sean-k-mooney: thanks for that | |
| 10:44:34 | sean-k-mooney | :) no worries did you see my top level comment to artom re changing id to internal_id in a followup if we feel that is needed | |
| 10:45:23 | sean-k-mooney | i didnt really think it was needed but i dont really object to it either if we want to disambiguate | |
| 10:46:52 | sean-k-mooney | if we did that i would not change the name in the db by the way i would jsut traslate in the api | |