| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-12-08 | |||
| 16:06:25 | elendrys | I tried to force scsci bus scan but it doesn't change | |
| 16:06:52 | elendrys | but when I extend an extra time, the previous size is seen instantaneously by the guest | |
| 16:07:10 | elendrys | it's just 1 extend late | |
| 16:17:20 | sean-k-mooney | well nova as far as i am aware will not need to notify qemu but perhaps we do | |
| 16:20:15 | clarkb | and those are sizes reported by lsblk? | |
| 16:20:35 | sean-k-mooney | unfortunetly i dont currently have the low level details loaded in my brain | |
| 16:20:39 | sean-k-mooney | clarkb: ya | |
| 16:20:55 | sean-k-mooney | the size of the block device repoted to the guest | |
| 16:21:12 | sean-k-mooney | which should eventualy be show in lsblk on the host and guest | |
| 16:21:44 | clarkb | ya I'm wondering if the guest does an explicit check via lsblk if it sees correct info and the only problem is in propogating the not on demand info | |
| 16:25:09 | elendrys | If it is better I can open the discussion on the mailing list | |
| 16:26:29 | kgube_ | the size seems to come from os-brick LinuxSCSI which calls "blockdev --getsize64" after a rescan | |
| 16:31:39 | kgube_ | so maybe the rescan in LinuxSCSI.extend_volume is not working | |
| 16:32:28 | kgube_ | elendrys, have you tried it with debug logging? | |
| 16:33:16 | elendrys | nope, but I can do that quickly | |
| 16:35:29 | kgube_ | there should be "Starting size: " and "volume size after scsi device rescan " in the log | |
| 16:36:37 | elendrys | Starting size: 76235669504 extend_volume /usr/lib/python3/dist-packages/os_brick/initiator/linuxscsi.py:555 | |
| 16:36:56 | elendrys | volume size after scsi device rescan 80530636800 extend_volume | |
| 16:39:03 | elendrys | hem | |
| 16:39:12 | elendrys | - /dev/disk/by-id/dm-uuid-mpath-3624a9370cebbf98ad4534b81000386de has shown up. wait_for_path | |
| 16:39:21 | elendrys | mpath(/dev/disk/by-id/dm-uuid-mpath-3624a9370cebbf98ad4534b81000386de) current size 76235669504 | |
| 16:39:29 | elendrys | mpath(/dev/disk/by-id/dm-uuid-mpath-3624a9370cebbf98ad4534b81000386de) new size 76235669504 | |
| 16:40:04 | elendrys | Extend iSCSI Volume /dev/dm-28; new_size=76235669504 extend_volume | |
| 16:40:11 | elendrys | Resizing target device /dev/dm-28 to 76235669504 _resize_attached_volume | |
| 16:40:35 | elendrys | I think that it rely on multipath but it is not refreshed quickly enough | |
| 16:43:01 | kgube_ | yeah that seems to come from the multipath code | |
| 16:43:03 | kgube_ | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/linuxscsi.py#L639 | |
| 16:45:00 | elendrys | it explains why I see the previous size in the guest | |
| 16:47:18 | kgube_ | I don't know enough about scsi to have any idea what to do about this, though | |
| 16:47:47 | elendrys | Should I fill an issue on the tracker or just ask the mailing list ? | |
| 16:48:05 | elendrys | Thank you for your time :) | |
| 16:53:14 | kgube_ | hm, I'm not sure. I'm very new to Nova dev | |
| 16:55:14 | elodilles | sean-k-mooney: sorry for pinging, but if you have some free minutes to review this old stable backport, that would be awesome: https://review.opendev.org/c/openstack/nova/+/764504 | |
| 16:55:27 | elodilles | (it has a lot of +1 on it) | |
| 16:57:49 | kgube_ | elendrys, maybe try the mailing list first there is also a lot of openstack users reading it | |
| 16:58:12 | kgube_ | elendrys, maybe someone has seen this before and found a workaround | |
| 17:08:58 | sean-k-mooney | oh ya that sure happy to see that merge | |
| 17:09:41 | sean-k-mooney | elodilles: give me 5 mins to finsih somehting and ill review it then | |
| 17:12:20 | elodilles | thanks in advance o/ | |
| 17:27:35 | sean-k-mooney | elodilles: once that merges im happy to continue moving this back to train so ill keep an eye out for that over then few days | |
| 17:28:12 | sean-k-mooney | elodilles: how are the stabel gates at the moment have they been affected by the tempest tox issue on master or the tox 4.0 issues | |
| 17:32:33 | elodilles | sean-k-mooney: thanks, i'll try to keep an eye on them, too :) good question about the gate, I haven't checked that yet :S but i guess we will see :/ | |
| #openstack-nova - 2022-12-09 | |||
| 06:03:04 | opendevreview | Hiroki Narukawa proposed openstack/nova master: libvirt: retry libvirt connection on live_migration_monitor https://review.opendev.org/c/openstack/nova/+/867077 | |
| 08:16:31 | opendevreview | Bence Romsics proposed openstack/nova master: doc: soft delete and shadow tables https://review.opendev.org/c/openstack/nova/+/867001 | |
| 08:52:03 | kgube | sean-k-mooney, sorry for pinging, but have you had a chance to look at this yet? https://review.opendev.org/c/openstack/nova-specs/+/855490 | |
| 08:52:16 | kgube | sean-k-mooney, whoami-rajat said he would like to avoid investing time in reviewing the cinder spec if it isn't clear that that the new direction of the change is acceptable for Nova | |
| 08:54:50 | kgube | sean-k-mooney, but cinder has spec freeze in a week so there is not alot of time left | |
| 09:32:19 | whoami-rajat | kgube, sean-k-mooney yeah, we already had reviewed the original spec several times and it was close to approval until the direction changed from nova side, it would be good to know if the new spec is the correct way forward | |
| 09:43:39 | elendrys | kgube: After several tests the enhanced code is present only in Zed (Ussuri here), and backporting it does not quite work either because the map/resize functions awaits either for the multipath name or the device mapper location, and the wwn is passed as argument | |
| 09:44:49 | elendrys | my conclusion may be that I should use an older version of multipathd, i'll see if someone reply on the list | |
| 10:29:30 | kgube | elendrys, it looks like multipathd is called with the wrong arguments in the os-brick code | |
| 10:30:07 | kgube | it's called like this: `self._execute('multipathd', 'resize', 'map', mpath_id, ...)` | |
| 10:30:54 | elendrys | yep | |
| 10:31:05 | kgube | but it seems that the command should be one argument: multipathd -k"resize map multipath_device" | |
| 10:31:31 | kgube | according to this: https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/6/html/storage_administration_guide/ch37s03 | |
| 10:31:48 | kgube | and the man page | |
| 10:33:21 | elendrys | this doc is quite old (RHEL 6), the wwid is not supported on my os | |
| 10:33:29 | elendrys | "fail" | |
| 10:33:43 | elendrys | multipathd[1661]: 3624a9370cebbf98ad4534b81000386de: invalid map name. cannot resize | |
| 10:34:29 | kgube | ok, yeah that information seems to be outdated | |
| 10:34:45 | kgube | sorry | |
| 10:35:25 | elendrys | https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/7/html/dm_multipath/online_device_resize | |
| 10:41:48 | zigo | Hi there! I've started a new project, using oslo.config. It works super well, however, I haven't find out (even reading the oslo.config doc...) how to select the default path for the config file. Does anyone know? | |
| 10:44:44 | zigo | Also, could anyone have a look into https://bugs.launchpad.net/python-novaclient/+bug/1998856 please? | |
| 11:09:09 | sean-k-mooney | whoami-rajat: well we had also reviewd it several times we will try and review it again we have a spec review day on wednesday | |
| 11:11:00 | whoami-rajat | sean-k-mooney, yes and we can save review bandwidth (both on nova and cinder side) with proper discussion about the path forward, that is my intent | |
| 14:55:58 | gibi | bauzas: we should probably start thinking about vancouver forum sessions as it has the same CFP deadline as the summit presentations (10th of Jan) | |
| 15:00:00 | bauzas | gibi: yup, I started to create three sessions for the summit | |
| 15:00:14 | bauzas | one is for the nova new features | |
| 15:04:33 | gibi | bauzas: cool | |
| 15:10:53 | opendevreview | Merged openstack/nova stable/victoria: Fix unplugging VIF when migrate/resize VM https://review.opendev.org/c/openstack/nova/+/764504 | |
| 17:17:47 | stephenfin | gibi: I got 5 patches into the series. I'll pick it up again Monday. They're a tough review :) | |
| 17:18:26 | gibi | stephenfin: thanks you very much for the effort | |
| 17:30:34 | sean-k-mooney | stephenfin: yep they are detailed | |
| 17:30:46 | sean-k-mooney | im hoping to finish reviewing the serise next week too | |
| 19:03:19 | opendevreview | Ghanshyam proposed openstack/nova master: Enable new defaults and scope checks by default https://review.opendev.org/c/openstack/nova/+/866218 | |
| #openstack-nova - 2022-12-12 | |||
| 08:36:30 | Uggla | Good morning Nova | |
| 08:45:33 | zigo | Hi there! FYI, python-cinderclient has the fix for the double parser.add_subparser() py 3.11 unit test crash: https://review.opendev.org/c/openstack/python-cinderclient/+/851467 | |
| 08:45:33 | zigo | Similar patch should be added to novaclient as well (it's the same code, as much as I can see...) | |
| 11:27:46 | gibi | bauzas: it seems that https://review.opendev.org/c/openstack/devstack/+/867065 resolved the tempest issue in nova-ceph-multistore and nova-grenade-multinode | |
| 12:21:59 | opendevreview | Jan Hartkopf proposed openstack/nova-specs master: Repropose spec for updating user data https://review.opendev.org/c/openstack/nova-specs/+/863884 | |
| 12:50:51 | opendevreview | Jan Hartkopf proposed openstack/nova-specs master: Repropose spec for updating user data https://review.opendev.org/c/openstack/nova-specs/+/863884 | |
| 13:05:23 | opendevreview | Jan Hartkopf proposed openstack/nova-specs master: Repropose spec for updating user data https://review.opendev.org/c/openstack/nova-specs/+/863884 | |
| 15:16:15 | opendevreview | Merged openstack/nova master: Support multiple config file with mod_wsgi https://review.opendev.org/c/openstack/nova/+/864014 | |
| 15:35:51 | opendevreview | Balazs Gibizer proposed openstack/nova stable/zed: Support multiple config file with mod_wsgi https://review.opendev.org/c/openstack/nova/+/867162 | |
| 15:36:21 | gibi | elodilles, bauzas: if you could look at this backport that would be awesome ^^ | |
| 15:37:06 | elodilles | gibi: looking | |
| 15:43:49 | opendevreview | Jan Hartkopf proposed openstack/nova-specs master: Repropose spec for updating user data https://review.opendev.org/c/openstack/nova-specs/+/863884 | |
| 17:34:02 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: testS: Fix Python 3.11 compatibility https://review.opendev.org/c/openstack/python-novaclient/+/867270 | |
| 17:48:50 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: trivial: Remove use of kwargs https://review.opendev.org/c/openstack/python-novaclient/+/806918 | |
| 17:49:27 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: tests: Fix Python 3.11 compatibility https://review.opendev.org/c/openstack/python-novaclient/+/867270 | |
| 19:53:39 | opendevreview | Merged openstack/python-novaclient master: Update python classifier for python 3.10 https://review.opendev.org/c/openstack/python-novaclient/+/861469 | |
| 21:51:47 | opendevreview | Merged openstack/nova stable/zed: Support multiple config file with mod_wsgi https://review.opendev.org/c/openstack/nova/+/867162 | |
| #openstack-nova - 2022-12-13 | |||
| 05:37:33 | opendevreview | Ghanshyam proposed openstack/nova master: Enable new defaults and scope checks by default https://review.opendev.org/c/openstack/nova/+/866218 | |
| 08:42:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce asym NUMA mixed CPU policy bug https://review.opendev.org/c/openstack/nova/+/862686 | |
| 08:42:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Handle zero pinned CPU in a cell with mixed policy https://review.opendev.org/c/openstack/nova/+/862687 | |
| 08:43:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce asym NUMA mixed CPU policy bug https://review.opendev.org/c/openstack/nova/+/862686 | |
| 08:43:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Handle zero pinned CPU in a cell with mixed policy https://review.opendev.org/c/openstack/nova/+/862687 | |
| 09:58:31 | opendevreview | Aaron S proposed openstack/nova master: Add further workaround features for qemu_monitor_announce_self https://review.opendev.org/c/openstack/nova/+/867324 | |
| 13:49:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP for the scheduler part of PCI in placement https://review.opendev.org/c/openstack/nova/+/862876 | |