| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-10-28 | |||
| 08:10:30 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Do not store InstanceInvalidState as instance_fault https://review.opendev.org/c/openstack/nova/+/860702 | |
| 09:51:24 | gibi | /home/fedora/rtox/nova/functional-py310/.tox/functional-py310/lib/python3.10/site-packages/oslo_db/sqlalchemy/enginefacade.py:537: DeprecationWarning: Using the 'mysql_enable_ndb' argument is deprecated in version '12.1.0': Support for the MySQL NDB Cluster storage engine has been deprecated and will be removed in a future release. | |
| 09:51:24 | gibi | stephenfin: do you have a hint why nova gets | |
| 09:51:36 | gibi | I see the deprecation decorator in oslo_db | |
| 09:51:49 | gibi | but I grepped for that param in nova and I don't see any usage of it | |
| 09:52:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP for the scheduler part of PCI in placement https://review.opendev.org/c/openstack/nova/+/862876 | |
| 10:02:48 | gibi | sean-k-mooney[m]: thanks for the reviews on the PCI series I opened a FUP ^^ to collect the minor things | |
| 10:05:53 | sean-k-mooney[m] | ill make my way true the rest on tuesday | |
| 10:06:16 | gibi | OK, thanks | |
| 10:06:43 | gibi | I've fixed the test instability in the last patch too | |
| 10:07:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Strictly follow placement allocation during PCI claim https://review.opendev.org/c/openstack/nova/+/855650 | |
| 10:07:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP for the scheduler part of PCI in placement https://review.opendev.org/c/openstack/nova/+/862876 | |
| 10:12:19 | gibi | bauzas: if you are around today, this is a simple spec repropose for A https://review.opendev.org/c/openstack/nova-specs/+/855661 | |
| 10:19:24 | bauzas | gibi: was on meeting but will look | |
| 10:20:20 | gibi | thanks | |
| 10:20:47 | bauzas | gibi: sent to the gate, bravo for the first approved spec afaik | |
| 10:21:41 | gibi | thanks. I was well prepared since Zed FF to land the re-propose early along with the code | |
| 10:32:52 | opendevreview | Merged openstack/nova-specs master: Re-propose PCI Device Tracking In Placement for A https://review.opendev.org/c/openstack/nova-specs/+/855661 | |
| 16:06:34 | darkhorse | hi team - how can i run functional test only for one function that i have added? tox command runs too many tests and takes too long. | |
| 16:08:54 | clarkb | darkhorse: generally projects respect some form of `tox -epy310 -- test_name_here` to filter tests by that name | |
| 16:09:07 | clarkb | you can look at the tox.ini to see how posargs are handled to confirm this | |
| 16:10:50 | darkhorse | clarkb: thank you for answering my question quickly. can you give me an example of running functional tests only (skipping all other tests like pep8)? | |
| 16:13:28 | gibi | darkhorse: this will run all functional tests with python 3.10: tox -e functional-py310 | |
| 16:13:41 | clarkb | darkhorse: if you look in the tox.ini file it lists the various targets. You specify a target using `tox -e` so in this case `tox -e functional-py310` Then if you want to filter for lists you'll see the functional targets pass in posargs to the stestr command which allows you to do something like `tox -e functional-py310 -- your_test_name_here` | |
| 16:13:56 | gibi | darkhorse: this will only run test from the SimpleRCAndTraitBasedPCIAliasTests class: tox -e functional-py310 SimpleRCAndTraitBasedPCIAliasTests | |
| 16:13:58 | clarkb | *filter for specific tests | |
| 16:15:08 | darkhorse | that makes sense. thank you all! | |
| 17:17:40 | opendevreview | Alex Chan proposed openstack/nova master: record action log when deleting shelved instance https://review.opendev.org/c/openstack/nova/+/862404 | |
| 18:09:29 | eandersson | What is the solution for race conditions in the scheduler with high density VCPU servers with core pinning enabled? cstone mentioned that setting max_concurrent_builds to 1 fixes it | |
| 18:14:12 | eandersson | > Failed to build and run instance: nova.exception.CPUPinningInvalid: CPU set to pin [81, 17] must be a subset of free CPU set [4, ....., 127] | |
| 18:22:56 | opendevreview | melanie witt proposed openstack/nova master: Add mock to avoid loading guestfs in unit test https://review.opendev.org/c/openstack/nova/+/862769 | |
| 21:55:43 | opendevreview | Patrick Hampson proposed openstack/nova master: Doc: Fix list servers detailed response param https://review.opendev.org/c/openstack/nova/+/862963 | |
| 22:10:27 | 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 | |
| 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 | |