| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-09-19 | |||
| 16:08:00 | sean-k-mooney | im here but.... | |
| 16:08:04 | sean-k-mooney | i only had one topic | |
| 16:09:32 | sean-k-mooney | basically do we ant to skip the trunk migration test in the ovn jobs | |
| 16:09:48 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/895655 | |
| 16:10:00 | gibi | I'm OK to skip it now and re-enable it once it is fixed | |
| 16:10:15 | sean-k-mooney | once its merged i can propsoe the revert | |
| 16:10:22 | gibi | yeah that is good idea | |
| 16:10:32 | sean-k-mooney | and then we can hold it until neutron tells us to merge the revert | |
| 16:10:50 | sean-k-mooney | so thats all i had | |
| 16:11:20 | gibi | the RC1 patch is already approved by bauzas https://review.opendev.org/c/openstack/releases/+/894693 | |
| 16:11:34 | gibi | so I'm not sure we want to land that on master now | |
| 16:11:47 | gibi | maybe after RC1 released | |
| 16:11:57 | gibi | I don't want to mix up the RC1 rel process | |
| 16:12:33 | sean-k-mooney | ya we can wait | |
| 16:12:54 | sean-k-mooney | i can backport it to stable if need or we can include it in rc2 | |
| 16:13:09 | gibi | ack | |
| 16:13:21 | gibi | anybody has anything else? | |
| 16:15:20 | gibi | I guess we can wrap this up | |
| 16:15:40 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.log.html | |
| 16:15:40 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.txt | |
| 16:15:40 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.html | |
| 16:15:40 | opendevmeet | Meeting ended Tue Sep 19 16:15:40 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:15:40 | gibi | #endmeeting | |
| #openstack-nova - 2023-09-20 | |||
| 00:19:03 | opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class in NovaObjectRegistry https://review.opendev.org/c/openstack/nova/+/894892 | |
| 01:04:49 | opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class https://review.opendev.org/c/openstack/nova/+/894892 | |
| 01:17:11 | opendevreview | Jorhson Deng proposed openstack/nova master: Clear the UPDATING_PASSWORD when compute restarted https://review.opendev.org/c/openstack/nova/+/845242 | |
| 01:39:22 | opendevreview | Jorhson Deng proposed openstack/nova master: Allow user defined stop type https://review.opendev.org/c/openstack/nova/+/861333 | |
| 03:34:52 | opendevreview | Jorhson Deng proposed openstack/nova master: Allow user defined stop type https://review.opendev.org/c/openstack/nova/+/861333 | |
| 03:38:12 | opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class https://review.opendev.org/c/openstack/nova/+/894892 | |
| 03:42:17 | opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class https://review.opendev.org/c/openstack/nova/+/894892 | |
| 03:45:22 | opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class https://review.opendev.org/c/openstack/nova/+/894892 | |
| 11:38:35 | dvo-plv | Hello, Nova Folks. Maybe you will have a chance to review this patch https://review.opendev.org/c/openstack/nova/+/876075/25 | |
| 11:46:16 | sean-k-mooney | dvo-plv: you need to basically copy paste the spec and we need to approve that again before this can merge | |
| 11:46:26 | sean-k-mooney | but i can take a look later today | |
| 11:47:19 | dvo-plv | Should it be new commit? | |
| 11:48:15 | sean-k-mooney | the nova patch is fine and should remain the same | |
| 11:48:30 | sean-k-mooney | but you need to submit a new nova-spec patch | |
| 11:48:47 | sean-k-mooney | which copies the spec file to the new 2024.1 direcotry | |
| 11:49:01 | sean-k-mooney | and adds "reporpeosed" to the revision history | |
| 11:49:22 | sean-k-mooney | oh sec | |
| 11:49:45 | sean-k-mooney | dvo-plv: like this https://review.opendev.org/c/openstack/nova-specs/+/895648 | |
| 11:50:22 | sean-k-mooney | you just copy the file and update the history section https://review.opendev.org/c/openstack/nova-specs/+/895648/1/specs/2024.1/approved/assisted-volume-extend.rst#284 | |
| 11:51:09 | sean-k-mooney | then we just re review it but for reporposals with no change in scope we have a light weight process for that | |
| 11:51:41 | dvo-plv | okay, i see , thanks | |
| 12:00:18 | opendevreview | Danylo Vodopianov proposed openstack/nova-specs master: Re-propose using extend volume completion action for 2024.1 https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 12:12:24 | opendevreview | Danylo Vodopianov proposed openstack/nova-specs master: Re-propose using VirtIO PackedRing Configuration supportfor 2024.1 https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 12:15:02 | opendevreview | Danylo Vodopianov proposed openstack/nova-specs master: Re-propose using VirtIO PackedRing Configuration support for 2024.1 https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 12:34:58 | opendevreview | Danylo Vodopianov proposed openstack/nova-specs master: Re-propose using VirtIO PackedRing Configuration support for 2024.1 https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 13:08:02 | sahid | guys, quick question, we are considering to increase heal_instance_info_cache_interval from 60 to 600 that to decrease a bit the pressiur on neutron server | |
| 13:08:41 | sean-k-mooney | if netron has no bugs or spefcificly the neutron backend you can actully turn that off | |
| 13:08:59 | sahid | do you see any side effect that can affect or env doing that? | |
| 13:09:12 | sahid | sean-k-mooney: oh that is interesting | |
| 13:10:08 | sean-k-mooney | in general its still good to not trun it off but its there to wrokaround the fact that neutron can currpt the network info cache if ti returns partial results | |
| 13:10:43 | sahid | yes i think we will try to be consersavatif on keep it | |
| 13:10:44 | sean-k-mooney | cisco aci or other core plugins that use external databases | |
| 13:10:53 | sean-k-mooney | were partically prone to makeing port list falky | |
| 13:11:07 | sahid | i see | |
| 13:11:34 | sean-k-mooney | what neutorn backend are ye using | |
| 13:12:39 | sean-k-mooney | we have considerd defaullting this to off in our downstream instller going forwrad but we havent actully done that yet | |
| 13:14:48 | sean-k-mooney | it might we worth consdierign defaulting it to off upstream or exteneding the default timeout upstream | |
| 13:54:23 | sahid | sean-k-mooney: we are using ovs | |
| 14:03:28 | sean-k-mooney | ml2/ovs is not one of the backend that partcalarly need the heal perodic | |
| 14:03:57 | sean-k-mooney | in fact its proably the one that is most tollererent to a long timeout or disableing it | |
| 14:05:40 | opendevreview | Danylo Vodopianov proposed openstack/nova-specs master: Re-propose using VirtIO PackedRing Configuration support for 2024.1 https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 14:06:29 | sahid | sean-k-mooney: thank you we will set it from 60 to 600 and then see if we can completly disable it | |
| #openstack-nova - 2023-09-21 | |||
| 09:42:24 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP:Adds device tagging functional tests https://review.opendev.org/c/openstack/nova/+/895162 | |
| 10:04:52 | manuvakery1 | Hi .. I noticed that the Live migration is taking too long to complete and in the event show its showing the compute_check_can_live_migrate_destination is taking huge time | |
| 10:05:19 | manuvakery1 | "result": "Success", | |
| 10:05:19 | manuvakery1 | "finish_time": "2023-09-21T05:57:23.000000", | |
| 10:05:19 | manuvakery1 | "start_time": "2023-09-21T05:41:21.000000", | |
| 10:05:19 | manuvakery1 | "event": "compute_check_can_live_migrate_destination", | |
| 10:05:19 | manuvakery1 | { | |
| 10:05:20 | manuvakery1 | self.force_reraise()\n File \"/usr/lib/python3.6/site-packages/oslo_utils/excutils.py\", line 196, in force_reraise\n six.reraise(self.type_, self.value, self.tb)\n File \"/usr/lib/python3.6/site-packages/six.py\", line 693, in reraise\n raise value\n File \"/usr/lib/python3.6/site-packages/nova/compute/manager.py\", line 207, in decorated_function\n return function(self, context, *args, **kwargs)\n File | |
| 10:05:20 | manuvakery1 | "traceback": " File \"/usr/lib/python3.6/site-packages/nova/compute/utils.py\", line 1372, in decorated_function\n return function(self, context, *args, **kwargs)\n File \"/usr/lib/python3.6/site-packages/nova/compute/manager.py\", line 219, in decorated_function\n kwargs['instance'], e, sys.exc_info())\n File \"/usr/lib/python3.6/site-packages/oslo_utils/excutils.py\", line 220, in __exit__\n | |
| 10:05:22 | manuvakery1 | in _create_shared_storage_test_file\n fd, tmp_file = tempfile.mkstemp(dir=dirpath)\n File \"/usr/lib64/python3.6/tempfile.py\", line 344, in mkstemp\n return _mkstemp_inner(dir, prefix, suffix, flags, output_type)\n File \"/usr/lib64/python3.6/tempfile.py\", line 262, in _mkstemp_inner\n fd = _os.open(file, flags, 0o600)\n File \"/usr/lib/python3.6/site-packages/eventlet/green/os.py\", line 118, in open\n | |
| 10:05:22 | manuvakery1 | \"/usr/lib/python3.6/site-packages/nova/compute/manager.py\", line 6897, in check_can_live_migrate_destination\n block_migration, disk_over_commit)\n File \"/usr/lib/python3.6/site-packages/nova/virt/libvirt/driver.py\", line 8432, in check_can_live_migrate_destination\n filename = self._create_shared_storage_test_file(instance)\n File \"/usr/lib/python3.6/site-packages/nova/virt/libvirt/driver.py\", line 8749, | |
| 10:05:24 | manuvakery1 | } | |
| 10:05:24 | manuvakery1 | fd = __original_open__(file, flags, mode)\n" | |
| 10:05:58 | manuvakery1 | Any hint on this | |
| 10:36:55 | opendevreview | OpenStack Release Bot proposed openstack/nova stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/nova/+/896019 | |
| 10:36:59 | opendevreview | OpenStack Release Bot proposed openstack/nova stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/nova/+/896020 | |
| 10:37:03 | opendevreview | OpenStack Release Bot proposed openstack/nova master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/nova/+/896021 | |
| 13:08:00 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Do not store InstanceInvalidState as instance_fault https://review.opendev.org/c/openstack/nova/+/860702 | |
| 13:30:16 | opendevreview | Stephen Finucane proposed openstack/nova master: Implement add_consumer, remove_consumer KeyManager APIs https://review.opendev.org/c/openstack/nova/+/896100 | |
| 14:12:02 | bauzas | just in case people don't know | |
| 14:12:06 | bauzas | welcome in Caracal ! | |
| 17:52:30 | opendevreview | Merged openstack/placement stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/placement/+/895491 | |
| 17:52:32 | opendevreview | Merged openstack/placement stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/placement/+/895492 | |
| #openstack-nova - 2023-09-22 | |||
| 03:20:53 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/896175 | |
| 09:07:53 | bauzas | fyi, folks, did the launchpad usual janitory for creating a new series and milestones on both placement and nova : https://launchpad.net/placement/2024.1 and https://launchpad.net/nova/2024.1 | |
| 09:08:17 | bauzas | as you can see, I stopped using the release name to mock what we already have in our specs repo | |
| 09:36:05 | opendevreview | Merged openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/896175 | |
| 10:11:47 | stephenfin | bauzas: Can we merge the outstanding sqlalchemy 2.0 stuff now? | |
| 10:12:37 | stephenfin | actually, it's sean-k-mooney I should be asking. Sean, could you look at https://review.opendev.org/c/openstack/nova/+/860829/ again? | |
| 10:13:05 | sean-k-mooney | yes please | |
| 10:13:21 | stephenfin | though I am going to respin https://review.opendev.org/c/openstack/nova/+/886230 to address those regex warnings | |
| 10:15:19 | sean-k-mooney | what about https://review.opendev.org/c/openstack/nova/+/894635 | |
| 10:16:11 | opendevreview | Stephen Finucane proposed openstack/nova master: Add job to test with SQLAlchemy master (2.x) https://review.opendev.org/c/openstack/nova/+/886230 | |
| 10:16:43 | stephenfin | sean-k-mooney, bauzas: ^ fixed to address the RE2 warning from zuul. It's otherwise unchanged. | |
| 10:16:52 | stephenfin | sean-k-mooney: I'll clean that last patch up today | |