Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2022-02-22
16:17:40 elodilles (just to have it in the logs)
16:18:02 bauzas :)
16:19:44 bauzas thanks elodilles
16:19:50 elodilles np
16:19:58 bauzas #topic Open discussion
16:20:08 bauzas liorf95: (lior Friedman) https://blueprints.launchpad.net/nova/+spec/nvmeof-multipath
16:20:14 bauzas liorf95: your turn
16:20:21 liorf95 yes please
16:20:23 bauzas or I can explain
16:20:32 bauzas maybe some of them don't understand the contect
16:20:36 bauzas context
16:20:36 bauzas so,
16:20:47 liorf95 we need this bp to be approved for https://review.opendev.org/c/openstack/nova/+/823941
16:21:34 bauzas basically we had https://review.opendev.org/c/openstack/nova/+/823941 that was reviewed
16:21:48 bauzas and stephenfin and sean-k-mooney accepted the change
16:21:49 bauzas but,
16:22:09 bauzas as you can look at the comments, we saw it was actually a feature
16:22:51 bauzas that's why I accepted to have a discussion in this meeting to see whether folks accept to merge this one even if we had not accepted the blueprint before
16:23:08 bauzas for the moment, there is a procedural -2 from me
16:23:38 bauzas but if folks agree here to have an exception for https://blueprints.launchpad.net/nova/+spec/nvmeof-multipath as a specless bp, then I could remove it
16:24:07 bauzas now, the question : is https://blueprints.launchpad.net/nova/+spec/nvmeof-multipath a specless BP and are folks accepting it as a exception ?
16:24:54 gibi I've nothing against it. It is just a volume driver so no spec needed. And the change is really small
16:25:12 bauzas correct
16:26:02 bauzas change is small and only touching a very few places
16:26:27 sean-k-mooney i am supprotive fo doign it as a specless blueprint
16:26:42 bauzas ok, anyone having concern ?
16:26:45 sean-k-mooney its after teh cutoff but the scale of it is small enought that i think an excption can be made
16:26:55 bauzas yeah
16:27:08 opendevreview Merged openstack/placement master: Extend the RP db query to support any-traits https://review.opendev.org/c/openstack/placement/+/825848
16:27:10 opendevreview Merged openstack/placement master: DB layer should only depend on trait id not names https://review.opendev.org/c/openstack/placement/+/826490
16:27:12 bauzas basically we have deadlines for accepted blueprints because they change lots of things
16:27:13 opendevreview Merged openstack/placement master: Enhance doc of _get_trees_with_traits https://review.opendev.org/c/openstack/placement/+/825780
16:27:16 opendevreview Merged openstack/placement master: Extend the RP tree DB query to support any-traits https://review.opendev.org/c/openstack/placement/+/825849
16:27:18 opendevreview Merged openstack/placement master: Add any-traits support for listing resource providers https://review.opendev.org/c/openstack/placement/+/826491
16:27:45 bauzas so we're procedural not for the sake of being it, but for making sure we are sure how this works
16:27:57 bauzas here, I don't see a problem
16:28:04 gibi I agree
16:29:00 bauzas ok so,
16:30:25 bauzas #agreed specless BP for https://blueprints.launchpad.net/nova/+spec/nvmeof-multipath as an exception
16:30:35 bauzas liorf95: I'll then remove my -2
16:30:41 bauzas and +W it
16:30:45 bauzas that's it I had
16:31:00 liorf95 Thanks a lot for your efforts here
16:33:28 bauzas any other topic to discuss as a last minute ?
16:33:36 gibi -
16:34:16 bauzas looks not
16:34:18 bauzas thanks all
16:34:22 bauzas #endmeeting
16:34:22 opendevmeet Meeting ended Tue Feb 22 16:34:22 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
16:34:22 opendevmeet Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-02-22-16.02.html
16:34:22 opendevmeet Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-02-22-16.02.txt
16:34:22 opendevmeet Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-02-22-16.02.log.html
16:34:23 gibi thanks
16:34:28 elodilles thanks bauzas o/
16:34:30 liorf95 Thanks
16:34:45 gibi sean-k-mooney: just a reminder you only left +1 on https://review.opendev.org/c/openstack/placement/+/82649 yesterday :)
16:35:34 chateaulav thanks o/
16:35:47 gmann thanks
16:38:53 gibi kashyap: https://logserver.rdoproject.org/83/38983/2/check/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset035-victoria/67d64d4/logs/undercloud/var/log/tempest/stestr_results.html.gz the tempest logs does not contain either the server uuid or the request id for the snapshot command :/
16:39:01 gibi I cannot correlate the logs :/
16:39:01 kashyap gibi: Good news!
16:39:04 gibi oh
16:39:18 kashyap gibi: I just talked to Peter Krempa on #virt, he already identified it as a libvirt bug ...
16:39:22 kashyap gibi: The problem is this, in short:
16:39:55 kashyap [quote]
16:39:56 kashyap libvirt has a piece of code which ensures that thh backing image of the reused destination image is added only when finishing the job. On cancellation of the [copy] job, we want to unplug the image, but the backing image was not yet plugged in.
16:40:00 kashyap [/quote]
16:40:08 kashyap It assumes a bit of libvirt storage knowledge, but Peter is already working on a fix
16:41:13 kashyap gibi: I'm filing RHEL bugs to get it backported to CentOS 8 and 9
16:41:43 gibi awesome
16:41:48 gibi thanks for the info
16:44:53 kashyap gibi: But Peter is again asking what is the test "exactly doing"
16:44:57 kashyap Is this exactly it?
16:45:02 kashyap "boot servers, then snapshot the server and upload the image to Glance"
16:45:27 kashyap Did you mean boot a "server", as opposed to plural?
16:47:36 gibi kashyap: create a server, and then create a snapshot
16:47:45 gibi kashyap: the test case itself create multiple servers
16:48:04 kashyap I see
16:57:14 kashyap gibi: https://bugzilla.redhat.com/show_bug.cgi?id=2057067
16:57:28 kashyap gibi: But Peter also says there might be another issue too, because:
16:57:40 kashyap ... "this is on a code path that can't propagate errors to the caller"
16:57:48 kashyap gibi: Anyway, he posted a patch here: https://listman.redhat.com/archives/libvir-list/2022-February/msg00790.html
17:19:45 gibi kashyap: ack. I would need a tripleoo job result where I can correlate logs better to be able to look more into it
17:20:03 gibi kashyap: I serched for the QEMU error in the upstream nova job results and there is a lot of hit
17:20:16 gibi but I guess it is a generic error message
17:20:51 gibi as the QEMU error does not correlate with the tempest test failure
17:21:11 gibi I stop for today but happy to help tomorrow if there are things to look at
18:53:42 opendevreview Rajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368
18:53:58 opendevreview Merged openstack/nova master: Support use_multipath for NVME driver https://review.opendev.org/c/openstack/nova/+/823941
19:18:20 opendevreview Dmitrii Shcherbakov proposed openstack/nova master: Fix migration with remote-managed ports & add FT https://review.opendev.org/c/openstack/nova/+/829974
19:33:25 opendevreview Rajat Dhasmana proposed openstack/python-novaclient master: Add parameter to rebuild boot volume https://review.opendev.org/c/openstack/python-novaclient/+/827163
19:37:04 whoami-rajat dansmith, Hey, if you get some time, can you take a look at my feature patches?
19:37:06 whoami-rajat nova: https://review.opendev.org/c/openstack/nova/+/820368
19:37:11 whoami-rajat novaclient: https://review.opendev.org/c/openstack/python-novaclient/+/827163
19:37:53 dansmith whoami-rajat: ah I didn't see that you had updated them.. is it working now?
19:39:33 dansmith whoami-rajat: ah I didn't see that you had updated them.. is it working now?
19:39:45 dansmith whoami-rajat: I assume you've got a tempest patch somewhere (or planned) to poke it
19:39:46 dansmith ?
19:40:28 opendevreview Merged openstack/placement master: Add any-traits support for allocation candidates https://review.opendev.org/c/openstack/placement/+/826492
19:40:40 whoami-rajat dansmith, just updated everything and yes it is working now. thanks!

Earlier   Later