| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-01 | |||
| 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: Robustify Compute Node Hostnames backlog spec https://review.opendev.org/c/openstack/nova-specs/+/853837 | |
| 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 | |
| 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 | |
| 16:06:46 | dansmith | darkhorse: well, here's an example: https://review.opendev.org/c/openstack/nova/+/862404/3/nova/tests/unit/compute/test_api.py#1271 | |
| 16:07:03 | dansmith | if it gets called more than once, mock.has_calls() | |
| 17:19:28 | gibi | sean-k-mooney: I'm not sure we have explicit tunelled network support but I 70% sure that the nova code does not make any assumption about the port.resource_request | |
| 17:19:57 | gibi | so what neutron try to do is probably work with the existing nova support out of the box | |
| 17:20:36 | sean-k-mooney | proably but i tought we benchmarked it with tunnels at one point | |
| 17:20:49 | sean-k-mooney | i think your are right that the nova side will work | |
| 17:21:12 | sean-k-mooney | what then need to be carful of is not assuming the tunnel bandwith is independnet of the physnet bandwith | |
| 17:21:30 | sean-k-mooney | it can be | |
| 17:22:03 | sean-k-mooney | but it would be the less common deployment model | |
| 17:23:43 | gibi | yeah, if the tunelled networks using the same phyiscal interface than a physnet uses then the currently proposed model doesn't work as the physnet and the tunelled bandwidth then cannot be represented separately | |
| 17:25:10 | sean-k-mooney | well that is what ooo should be doing today (sharing one interface for tunneled and notn tunneled traffic) by default | |
| 17:28:59 | opendevreview | Dan Smith proposed openstack/nova master: Test ceph-multistore with a real image https://review.opendev.org/c/openstack/nova/+/860864 | |
| 17:33:56 | gibi | sean-k-mooney: OK, I see you highlighted that in a comment. cool. the current proposal only works if the tunelled traffic has its own dedicated interface | |
| 17:38:44 | sean-k-mooney | that wont work for ovs-dpdk or hardware offloads really | |
| 17:39:07 | sean-k-mooney | you can confirue it so that they could use a dedicated interface but not without more work and cost | |
| 17:39:32 | sean-k-mooney | im actully not sure about hardwar offloaded ovs | |
| 17:40:19 | gibi | if they share interface then the deployer should statically split bw inventory between the RPs representing the physnet and the tunnel | |
| 17:41:36 | gibi | sean-k-mooney: could there be more than one interface for the tunneled traffic ? | |
| 17:43:38 | opendevreview | Alex Chan proposed openstack/nova master: record action log when deleting shelved instance Closes-Bug: #1993736 Change-Id: I9ce18cbba5083c55d15d9b7c2a89133d227754ea https://review.opendev.org/c/openstack/nova/+/863547 | |