| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-10-28 | |||
| 22:11:58 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/862964 | |
| #openstack-nova - 2022-10-29 | |||
| 00:06:37 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/862967 | |
| 00:06:37 | opendevreview | Artom Lifshitz proposed openstack/nova master: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/862964 | |
| 02:01:03 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/862967 | |
| 02:01:04 | opendevreview | Artom Lifshitz proposed openstack/nova master: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/862964 | |
| #openstack-nova - 2022-10-31 | |||
| 04:23:00 | opendevreview | shiyawei proposed openstack/nova master: fix bug: 1994018, the volume multiattach and in-use after retype another backend, then can not detach it Closes-Bug#1994018 https://review.opendev.org/c/openstack/nova/+/862355 | |
| 10:16:52 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/862603 | |
| 10:16:53 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/862604 | |
| 10:59:31 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova-specs master: spec: allowing target state for evacuate https://review.opendev.org/c/openstack/nova-specs/+/857838 | |
| 11:06:57 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova-specs master: spec: allowing target state for evacuate https://review.opendev.org/c/openstack/nova-specs/+/857838 | |
| 13:17:10 | opendevreview | Kirill proposed openstack/nova master: vnc support https://review.opendev.org/c/openstack/nova/+/863052 | |
| 13:22:18 | opendevreview | alecorps proposed openstack/nova master: Add mksproxy files https://review.opendev.org/c/openstack/nova/+/863055 | |
| 13:23:06 | opendevreview | alecorps proposed openstack/nova master: Add mksproxy files https://review.opendev.org/c/openstack/nova/+/863055 | |
| 14:27:46 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 14:27:46 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 14:44:07 | opendevreview | Andre Aranha proposed openstack/nova stable/zed: Remove the periodic Centos 8 job https://review.opendev.org/c/openstack/nova/+/862993 | |
| 16:52:57 | opendevreview | alecorps proposed openstack/nova master: Add mksproxy files https://review.opendev.org/c/openstack/nova/+/863090 | |
| #openstack-nova - 2022-11-01 | |||
| 00:23:01 | eandersson | Is CPU Pinning tied into numa_topology? | |
| 00:23:32 | eandersson | > hw_cpu_policy=dedicated | |
| 00:24:21 | eandersson | Trying to figure out if nova should be using the NUMA code even though we only have one CPU with only one NUMA zone visible to the OS. | |
| 00:35:35 | eandersson | Nvm found the documentation for this | |
| 17:09:02 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add stable-compute-uuid spec https://review.opendev.org/c/openstack/nova-specs/+/863152 | |
| 17:09:02 | opendevreview | Dan Smith proposed openstack/nova-specs master: Robustify Compute Node Hostnames backlog spec https://review.opendev.org/c/openstack/nova-specs/+/853837 | |
| 18:07:11 | opendevreview | Kirill proposed openstack/nova master: vnc support https://review.opendev.org/c/openstack/nova/+/863052 | |
| 18:07:36 | opendevreview | Dan Smith proposed openstack/nova-specs master: Robustify Compute Node Hostnames backlog spec https://review.opendev.org/c/openstack/nova-specs/+/853837 | |
| 18:07:37 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add stable-compute-uuid spec https://review.opendev.org/c/openstack/nova-specs/+/863152 | |
| 18:30:10 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add stable-compute-uuid spec https://review.opendev.org/c/openstack/nova-specs/+/863152 | |
| 18:48:15 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add stable-compute-uuid spec https://review.opendev.org/c/openstack/nova-specs/+/863152 | |
| 18:57:28 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add stable-compute-uuid spec https://review.opendev.org/c/openstack/nova-specs/+/863152 | |
| 18:57:35 | dansmith | man, I suck | |
| 19:14:05 | opendevreview | Kirill proposed openstack/nova master: vnc support https://review.opendev.org/c/openstack/nova/+/863052 | |
| 20:15:59 | opendevreview | Kirill proposed openstack/nova master: add vnc support for ironic https://review.opendev.org/c/openstack/nova/+/863177 | |
| 21:05:10 | opendevreview | Kirill proposed openstack/nova master: add vnc support for ironic https://review.opendev.org/c/openstack/nova/+/863177 | |
| 21:05:52 | opendevreview | Kirill proposed openstack/nova master: add vnc support for ironic https://review.opendev.org/c/openstack/nova/+/863177 | |
| 21:08:10 | opendevreview | Kirill proposed openstack/nova master: add vnc support for ironic https://review.opendev.org/c/openstack/nova/+/863177 | |
| #openstack-nova - 2022-11-02 | |||
| 07:32:42 | opendevreview | alecorps proposed openstack/nova master: Add mksproxy files https://review.opendev.org/c/openstack/nova/+/863090 | |
| 10:04:38 | opendevreview | alecorps proposed openstack/nova master: Add mksproxy files https://review.opendev.org/c/openstack/nova/+/863090 | |
| 10:05:42 | 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:48:42 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds check if resized to swap zero https://review.opendev.org/c/openstack/nova/+/857339 | |
| 12:22:50 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 12:46:57 | sean-k-mooney | bauzas: trivial doc update for you if your around https://review.opendev.org/c/openstack/nova/+/860936 | |
| 12:47:22 | bauzas | sean-k-mooney: can do, after a morning meeting :) | |
| 12:47:38 | sean-k-mooney | :) | |
| 12:57:31 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2023.1 Antelope https://review.opendev.org/c/openstack/nova/+/858238 | |
| 13:25:32 | opendevreview | Merged openstack/nova master: Correct wrong reference in docs https://review.opendev.org/c/openstack/nova/+/860936 | |
| 13:35:26 | opendevreview | Merged openstack/os-vif master: Move mtu update request into ovsdb transaction https://review.opendev.org/c/openstack/os-vif/+/857868 | |
| 16:47:18 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Add functional tests to reproduce bug #1994983 https://review.opendev.org/c/openstack/nova/+/863416 | |
| 20:26:14 | opendevreview | Dan Smith proposed openstack/nova master: Enable glance location tests on ceph-multistore https://review.opendev.org/c/openstack/nova/+/860863 | |
| 20:26:42 | opendevreview | Dan Smith proposed openstack/nova master: Enable glance location tests on ceph-multistore https://review.opendev.org/c/openstack/nova/+/860863 | |
| #openstack-nova - 2022-11-03 | |||
| 02:55:27 | opendevreview | Takashi Natsume proposed openstack/nova master: Add a hacking rule for the setDaemon method https://review.opendev.org/c/openstack/nova/+/854653 | |
| 08:40:01 | Uggla | Good morning Nova. | |
| 09:00:47 | gibi | Uggla: o/ | |
| 09:59:06 | elodilles | bauzas sean-k-mooney[m] : last meeting i was mostly focusing on the stein-queens EOL'ing that i forgot about nova's Wallaby-EM transition :/ | |
| 10:00:01 | elodilles | bauzas sean-k-mooney[m] : the transition patch was proposed 2 weeks ago, but now i realized that we have some merged patch to release as well: https://zuul.opendev.org/t/openstack/build/ff461f8d94b640c2a055935f3ca620bb/log/tox/list-changes-results.log#707-762 | |
| 10:02:10 | elodilles | bauzas sean-k-mooney[m] : so when you have time could you please decide whether we need this final-wallaby-release? it is a bit risky as the deadline was yesterday for the transition, so what we release will be the 'final', so we should be sure not to introduce regression in the final release | |
| 10:03:07 | elodilles | this is the release patch by the way: https://review.opendev.org/c/openstack/releases/+/863490 | |
| 11:08:08 | sean-k-mooney[m] | ack ill take a look | |
| 11:09:36 | sean-k-mooney[m] | elodilles: i am going to propose a release for os-vif on stable wallaby soon by the way. there isnt really any impoarnt patche except the ones that updated the upper constraits but it would be nice to use the correct one in the os-vif package. | |
| 11:12:35 | sean-k-mooney[m] | elodilles: what is that zuul link is that what we last released or is that the pending changes | |
| 11:13:02 | bauzas | sorry, I was on meetings this morning | |
| 11:13:04 | sean-k-mooney[m] | looking at the review its what is pending right | |
| 11:13:19 | bauzas | I'll try to look at that yes | |
| 11:14:37 | sean-k-mooney[m] | looking at the commits in the review i think its a good idea to do a final release yes | |
| 11:16:17 | sean-k-mooney[m] | the bdm patch is important if you host mount ceph volumes the evaucate path is also useful for operators, the ironic plug_vifs change is an important performance enhancement when restartign the nova compute agaent | |
| 11:16:51 | sean-k-mooney[m] | all of these are operational enhancments that operators will like to recive | |
| 12:28:17 | elodilles | sean-k-mooney[m]: OK, thanks, then let's release this and then do the EM transition \o/ | |
| 12:29:17 | elodilles | sean-k-mooney[m]: about the os-vif: upper-constraints won't be part of the package so it will be the very same as the previous release, thus i'd say it is not worth to release | |
| 12:29:24 | sean-k-mooney[m] | no worries my office is currently being used for an interview so im using the mornign to do reviews anyway | |
| 12:45:52 | darkhorse | Hi team - I have published a patchset https://review.opendev.org/c/openstack/nova/+/862404. It has passed CI build test and is still pending. | |
| 12:46:35 | darkhorse | Can anyone guide me what I should do to get it approved/merged? | |
| 13:48:20 | dansmith | darkhorse: can you add the assertions I just commented? | |
| 13:48:35 | dansmith | darkhorse: do that and add the bug link and I'll approve | |
| 13:55:36 | dansmith | bauzas: can you +W this so we can enable some more tests? https://review.opendev.org/c/openstack/nova/+/860863/2 | |
| 13:56:53 | dansmith | (or gibi or sean-k-mooney[m] ) | |
| 14:22:09 | bauzas | dansmith: done | |
| 14:22:25 | dansmith | bauzas: thanks | |
| 14:22:50 | dansmith | bauzas: I'm going to be back here asking for this one later, once a devstack patch merges: https://review.opendev.org/c/openstack/nova/+/860864/13 | |
| 14:23:18 | dansmith | but that ^ gets us a little more normalish behavior because the image can be more like a normal sized one.. meaning image operations don't take zero time like they do with cirros | |
| 14:25:22 | ralonsoh | gibi, hi, whenever you can, please check https://review.opendev.org/c/openstack/neutron-specs/+/860859/4/specs/2023.1/strict-minimum-bandwidth-tunnelled-networks.rst#95 | |
| 14:25:27 | ralonsoh | (no rush) | |
| 14:49:41 | sean-k-mooney[m] | oh right i rebooted i need to open irc on my main laptop https://review.opendev.org/c/openstack/nova/+/860863/2 looks like it shoudl be on its way to merging | |
| 14:49:52 | dansmith | yep | |
| 15:02:08 | opendevreview | Alex Chan proposed openstack/nova master: record action log when deleting shelved instance https://review.opendev.org/c/openstack/nova/+/862404 | |
| 15:11:23 | darkhorse | dansmith: I added the bug link but I did not get the other comments? Will you give me some examples? | |
| 15:11:29 | gibi | ralonsoh: done | |
| 15:15:54 | dansmith | darkhorse: https://review.opendev.org/c/openstack/nova/+/862404/2/nova/tests/unit/compute/test_api.py#1260 | |
| 15:16:10 | dansmith | darkhorse: also, you need to add the bug reference in the exact format that was suggested | |
| 15:16:50 | ralonsoh | gibi, thanks! | |
| 15:19:03 | sean-k-mooney | ralonsoh: you know tunneled supprot was in scope fo the orginal bandwith in placment spec right | |
| 15:19:38 | ralonsoh | sean-k-mooney, I don't remember that (and I created the spec) | |
| 15:19:41 | ralonsoh | maybe for Nova | |
| 15:20:01 | sean-k-mooney | it definetly was | |
| 15:20:44 | sean-k-mooney | i have left some comments on teh new spec but i dont have time to do a full review right now | |
| 15:24:14 | sean-k-mooney | am other thign gibi you support tunnteled networks for pps too right | |
| 15:24:44 | sean-k-mooney | we we dont we shoudl fix both at the same time | |
| 15:26:01 | sean-k-mooney | darkhorse: this might be worth reading over when you have time https://wiki.openstack.org/wiki/GitCommitMessages | |
| 15:26:27 | sean-k-mooney | also https://docs.opendev.org/opendev/infra-manual/latest/developers.html#working-on-bugs | |
| 15:31:24 | darkhorse | sean-k-mooney: thank you for the links about commit messaging. | |
| 15:32:40 | darkhorse | dansmith: Excuse me and thank you for your patience. I am still not understanding how I can add assert. Is there an example code that does something similar? | |
| 15:56:31 | opendevreview | Merged openstack/nova master: Enable glance location tests on ceph-multistore https://review.opendev.org/c/openstack/nova/+/860863 | |