| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-14 | |||
| 13:58:56 | sean-k-mooney | your going to need an attach_share rpc call | |
| 13:59:09 | sean-k-mooney | however hard reboot should not really call any external services | |
| 13:59:26 | sean-k-mooney | like it should not udpate cinder attachment or neutron prot bindings | |
| 13:59:38 | sean-k-mooney | so i woudl find it odd for it to update manila grants | |
| 14:00:07 | Uggla | ok I can implement that in rpc. Can I just make a sync call to rpc get the result and do the rest in the API ? | |
| 14:00:19 | Uggla | result = get the ip | |
| 14:00:45 | Uggla | or is it better to call the rpc and do the required work in the rpc ? | |
| 14:04:30 | gibi | bauzas: I left feedback on the prelude | |
| 14:04:43 | bauzas | ack, looing | |
| 14:06:26 | stephenfin | sean-k-mooney: the release note will be moved to zed when the tag is created | |
| 14:06:43 | stephenfin | (sorry, my VPN disconnected and I never noticed) | |
| 14:07:03 | stephenfin | (my ZNC deployment is now behind VPN) | |
| 14:09:24 | sean-k-mooney | no worries | |
| 14:10:45 | opendevreview | Sylvain Bauza proposed openstack/nova master: Prelude section for Zed release https://review.opendev.org/c/openstack/nova/+/857467 | |
| 14:10:50 | bauzas | gibi: sean-k-mooney: updated ^ | |
| 14:11:04 | bauzas | stephenfin: well, not automatically | |
| 14:11:23 | stephenfin | wdym? | |
| 14:11:24 | bauzas | stephenfin: we need to tell reno to look at the new stable/zed branch | |
| 14:11:25 | gibi | bauzas: that was fast :) | |
| 14:12:06 | bauzas | stephenfin: I mean, by for example https://review.opendev.org/c/openstack/nova/+/833243 | |
| 14:12:30 | stephenfin | bauzas: oh yeah, but the bot will do that | |
| 14:12:31 | bauzas | stephenfin: if we don't merge this file, then none of the files look at the new stable branch | |
| 14:12:41 | sean-k-mooney | bauzas: that also has the pbr tag in the commit | |
| 14:12:41 | stephenfin | you're right though | |
| 14:12:50 | bauzas | so we need to merge it quickly after the new branch | |
| 14:13:18 | stephenfin | it'll appear under the 26.0.0 tag but the zed page won't exist until created | |
| 15:18:36 | opendevreview | Merged openstack/os-vif master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/os-vif/+/856783 | |
| 15:36:44 | opendevreview | Merged openstack/python-novaclient master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/python-novaclient/+/856791 | |
| 15:46:09 | opendevreview | Merged openstack/nova master: Bump min oslo.concurrencty to >= 5.0.1 https://review.opendev.org/c/openstack/nova/+/857491 | |
| 16:50:20 | opendevreview | Merged openstack/nova master: Update compute rpc version alias for zed https://review.opendev.org/c/openstack/nova/+/855706 | |
| 17:13:50 | opendevreview | Merged openstack/nova master: Prelude section for Zed release https://review.opendev.org/c/openstack/nova/+/857467 | |
| 17:29:22 | nicolasbock | The server's port will get a dns_assignment with the generic domain from neutron.conf in the fqdn instead of the dns domain from the network. This breaks reverse DNS lookups from the VM. | |
| 17:29:22 | nicolasbock | 3. Created server on network | |
| 17:29:22 | nicolasbock | 2. Created zone | |
| 17:29:22 | nicolasbock | 1. Created network with dns_domain | |
| 17:29:22 | nicolasbock | Hi! I am trying to get Neutron/Designate integration to work and am running into the following issue: | |
| 17:29:23 | nicolasbock | Am I missing something? | |
| #openstack-nova - 2022-09-15 | |||
| 09:13:48 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova-specs master: spec: allowing target state for evacuate https://review.opendev.org/c/openstack/nova-specs/+/857838 | |
| 10:13:57 | auniyal__ | Hello | |
| 10:14:08 | auniyal__ | I need to test this function https://opendev.org/openstack/nova/src/commit/b8c62f2055de1079657e13aacf34d691d954f305/nova/virt/libvirt/driver.py#L3238 | |
| 10:14:21 | auniyal__ | so I added below, in integrated helpers https://opendev.org/openstack/nova/src/branch/master/nova/tests/functional/integrated_helpers.py#L626 | |
| 10:14:32 | auniyal__ | server['id'], {'createImage': {'name': snapshot_name}}) | |
| 10:14:32 | auniyal__ | admin_api.post_server_action( | |
| 10:14:32 | auniyal__ | admin_api = getattr(self, 'admin_api', self.api) | |
| 10:14:32 | auniyal__ | def _create_snapshot(self, server, snapshot_name): | |
| 10:14:42 | auniyal__ | self._create_snapshot(server, "snapshot-1") | |
| 10:14:42 | auniyal__ | then called like this from my test | |
| 10:15:21 | auniyal__ | but control never seems go in the target function | |
| 10:58:10 | opendevreview | Rajesh Tailor proposed openstack/os-vif master: Move mtu update request into ovsdb transaction https://review.opendev.org/c/openstack/os-vif/+/857868 | |
| 13:14:46 | opendevreview | Bence Romsics proposed openstack/nova stable/train: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/857877 | |
| 13:52:48 | bauzas | elodilles: gibi: sean-k-mooney: seems everything is in good shape for RC1 | |
| 13:52:55 | bauzas | any concerns before we branch ? | |
| 13:53:14 | bauzas | context : https://etherpad.opendev.org/p/nova-zed-rc-potential | |
| 13:53:36 | bauzas | elodilles: we'll need to wait for https://review.opendev.org/c/openstack/nova/+/856895 to be merged tho | |
| 14:06:07 | gibi | bauzas: I have nothing else for Zed. as soon as ^^ merges we can branch | |
| 14:23:24 | sean-k-mooney | bauzas: i have been meing to get back to https://review.opendev.org/c/openstack/nova-specs/+/856173 | |
| 14:23:44 | sean-k-mooney | ill try and spot check that today to make sure everythin that is done is updated | |
| 14:23:50 | bauzas | cool | |
| 14:23:53 | sean-k-mooney | but i think we are good to go for zed too | |
| 14:23:59 | bauzas | we don't branch the specs repo | |
| 14:24:00 | elodilles | bauzas: \o/ | |
| 14:24:01 | sean-k-mooney | bar the min bump | |
| 14:24:08 | sean-k-mooney | correct we dont | |
| 14:24:19 | sean-k-mooney | its just in the needed for rc section | |
| 14:24:35 | elodilles | bauzas: one more question | |
| 14:24:45 | bauzas | elodilles: I'll ping you when the min version bump will be merged | |
| 14:25:07 | elodilles | bauzas: https://review.opendev.org/c/openstack/releases/+/857153 | |
| 14:25:17 | elodilles | bauzas: what about placement? ^^^ | |
| 14:25:32 | bauzas | elodilles: should be good too | |
| 14:25:40 | elodilles | bauzas: ack | |
| 14:25:43 | elodilles | thanks! | |
| 14:25:44 | bauzas | for 20 mins | |
| 14:27:02 | elodilles | sean-k-mooney: you can also PTL-approved+1 the placement patch as release liaison o:) | |
| 14:28:40 | sean-k-mooney | sure i can do it i bauzas absece ill review it now quickly | |
| 14:29:18 | sean-k-mooney | for placment in partiacl i dont htink we had anything pendign but for nova we need to waith for the min oslo concurence bump to merge | |
| 14:32:35 | elodilles | sean-k-mooney: yepp, for nova we still wait for https://review.opendev.org/c/openstack/nova/+/856895 | |
| 14:33:49 | elodilles | sean-k-mooney: but according to bauzas placement should be OK, so if you have time to double-check & review it would be aweseome :) | |
| 14:50:41 | bauzas | elodilles: gave PTL+1 | |
| 14:50:46 | bauzas | for placement | |
| 15:03:32 | elodilles | bauzas: thanks \o/ | |
| 15:19:47 | opendevreview | Christian Rohmann proposed openstack/nova master: db: Drop redundant indeces on instances and console_auth_tokens tables https://review.opendev.org/c/openstack/nova/+/856757 | |
| 15:20:07 | opendevreview | Christian Rohmann proposed openstack/nova master: db: Drop redundant indexes on instances and console_auth_tokens tables https://review.opendev.org/c/openstack/nova/+/856757 | |
| 15:49:00 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/placement/+/857898 | |
| 15:49:02 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/placement/+/857899 | |
| 15:49:03 | opendevreview | OpenStack Release Bot proposed openstack/placement master: Update master for stable/zed https://review.opendev.org/c/openstack/placement/+/857900 | |
| 15:49:05 | opendevreview | OpenStack Release Bot proposed openstack/placement master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/placement/+/857901 | |
| 17:42:48 | opendevreview | Christian Rohmann proposed openstack/nova master: db: Drop redundant indexes on instances and console_auth_tokens tables https://review.opendev.org/c/openstack/nova/+/856757 | |
| 21:21:38 | opendevreview | Merged openstack/nova master: Update min supported service version for Zed https://review.opendev.org/c/openstack/nova/+/856895 | |
| #openstack-nova - 2022-09-16 | |||
| 06:24:24 | opendevreview | Amit Uniyal proposed openstack/nova stable/victoria: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/854980 | |
| 06:38:50 | opendevreview | Dylan McCulloch proposed openstack/nova master: Limit vgpu inventory to specified device_addresses https://review.opendev.org/c/openstack/nova/+/858012 | |
| 08:06:58 | bauzas | good morning | |
| 08:07:18 | bauzas | elodilles: could you please respin https://review.opendev.org/c/openstack/releases/+/857149 based on the hash I mentioned ? | |
| 08:09:04 | elodilles | bauzas: yepp, will do in a minute | |
| 08:09:09 | bauzas | thanks | |
| 08:10:31 | gibi | o/ | |
| 08:10:39 | gibi | happy stable/zed branc day! | |
| 08:11:10 | elodilles | gibi: not there yet... :D | |
| 08:11:22 | gibi | we will have it today, wont we? | |
| 08:11:33 | elodilles | that's the plan :) | |
| 08:11:41 | elodilles | bauzas: there you go: https://review.opendev.org/c/openstack/releases/+/857149 | |