| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-14 | |||
| 13:40:56 | sean-k-mooney | *those | |
| 13:46:48 | opendevreview | Merged openstack/os-vif master: Update master for stable/zed https://review.opendev.org/c/openstack/os-vif/+/856782 | |
| 13:50:07 | Uggla | question about my_block_storage_ip. This flag can be available on each compute nodes or the default is the compute ip. Am I right ? | |
| 13:50:37 | sean-k-mooney | yes | |
| 13:51:49 | Uggla | is there a way to get this parameter from the api or it can be only provided by the compute node itself ? | |
| 13:52:00 | sean-k-mooney | only on the compute | |
| 13:52:15 | sean-k-mooney | that is why you need to do the grant on the compute not the api | |
| 13:52:21 | sean-k-mooney | or conductor | |
| 13:52:49 | sean-k-mooney | it has to be done in the compute agent | |
| 13:53:28 | Uggla | so it needs a new "entry" in the compute API for this. Correct ? | |
| 13:54:19 | sean-k-mooney | for share attament yes you need that anyway | |
| 13:55:02 | sean-k-mooney | this is an internal detail or the attach_share rpc call | |
| 13:56:05 | Uggla | so far there is no rpc call for attach. As everything could be done in the API. | |
| 13:56:34 | sean-k-mooney | that would never work | |
| 13:56:49 | sean-k-mooney | we need a call to the compute to actully do the mounting of the share ectra | |
| 13:57:14 | sean-k-mooney | were you jsut goign to put that into spawn or something? | |
| 13:57:32 | Uggla | attach is just an entry within the db. And mount + real work is done at power_on | |
| 13:57:35 | sean-k-mooney | i have not look at this in a while so i dont remember the details | |
| 13:58:03 | sean-k-mooney | i guess that work today only because we require the vm to be off/shelved | |
| 13:58:23 | sean-k-mooney | but the rpc would be required if we supprot live attach in the future | |
| 13:58:24 | Uggla | yes off | |
| 13:58:45 | sean-k-mooney | ya so unless yo defer the grant to the power on stage | |
| 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 | |