Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-02
12:24:32 kashyap stephenfin: Hi there, damned if I could find, mind pointing to the Git repo of this page: https://docs.openstack.org/glance/latest/admin/useful-image-properties.html
12:25:14 stephenfin kashyap: https://github.com/openstack/glance/blob/master/doc/source/admin/useful-image-properties.rst
12:26:25 kashyap stephenfin: Most excellent
12:34:29 openstackgerrit Merged openstack/nova master: Resource tracker: improve resource tracker periodic task https://review.openstack.org/576052
12:34:37 openstackgerrit Merged openstack/nova master: Fix regression when listing build_requests with marker and ip filter https://review.openstack.org/576161
12:34:49 openstackgerrit Merged openstack/nova master: Update admin/flavors document https://review.openstack.org/573063
12:51:10 alex_xu Kevin_Zheng: pong
12:55:05 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: Update admin/flavors document https://review.openstack.org/579542
13:01:01 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Resolve Sphinx errors https://review.openstack.org/579545
13:18:05 openstackgerrit Merged openstack/nova master: Simplify instance name generation https://review.openstack.org/516573
13:27:18 openstackgerrit Deepak Mourya proposed openstack/nova master: Resource_provider API handler does not return specific error codes https://review.openstack.org/579423
13:31:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
13:32:45 gibi stephenfin: lost a +W in a trivial rebase on https://review.openstack.org/#/c/410297/ could you put it back?
13:32:53 stephenfin gibi: sure
13:32:55 efried naichuans_: We're nowhere close on reshaping yet, sorry.
13:33:01 efried But nrp is merged!
13:33:06 gibi stephenfin: thanks
13:39:31 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Fix regression when listing build_requests with marker and ip filter https://review.openstack.org/579561
13:49:46 naichuans_ efried: got it, thanks, Eric. Anyway, we can't apply n-rp to a already existed rp until reshape merged, right?
13:49:58 openstackgerrit Surya Seetharaman proposed openstack/python-novaclient master: [WIP/POC] Client changes for down cell https://review.openstack.org/579563
13:52:12 openstackgerrit Balazs Gibizer proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
13:52:13 openstackgerrit Balazs Gibizer proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
13:52:14 openstackgerrit Balazs Gibizer proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
14:03:33 efried naichuans_: Well, it depends. If you've previously exposed the inventory in question as part of the compute node provider, then yes, you need to handle reshaping. But if it's new resources you've never shown before, you're fine.
14:05:31 stephenfin gibi: What was the bug with sets in o.vo you were talking about?
14:05:51 stephenfin gibi: I'm seeing intermittent issues e.g. http://logs.openstack.org/40/564440/13/check/openstack-tox-py27/6df2c7b/testr_results.html.gz as think it's that same issue
14:06:47 efried stephenfin: https://review.openstack.org/#/c/569417/ / https://bugs.launchpad.net/oslo.versionedobjects/+bug/1771804 ?
14:06:47 openstack Launchpad bug 1771804 in oslo.versionedobjects "ObjectVersionChecker.get_hash generates unstable hash for set() default value between py27 and py35" [Undecided,Fix released] - Assigned to Balazs Gibizer (balazs-gibizer)
14:07:03 gibi efried: you were faster
14:07:15 stephenfin efried: That's what I was referring to, yup. Now to see if it is the same thing. Cheers!
14:13:07 stephenfin Hmm, so if sets in Python are unordered, how do I make sure a test that relies on particular ordering doesn't fail intermittently? https://review.openstack.org/#/c/575486/6/nova/tests/unit/objects/test_selection.py
14:14:13 efried sort
14:14:36 stephenfin that's going to change the object type though
14:14:46 stephenfin I guess that doesn't matter so long as we can convert back
14:14:48 efried If you're comparing sets, it should just work
14:16:22 gibi stephenfin: if you break up the big assert to a list of smaller asserts then you can eventually compare sets as efried suggests
14:16:49 stephenfin Ooh, much better than what I was about to do
14:18:35 gibi gibi catches
14:18:41 gibi I mean
14:18:44 gibi s
14:30:16 efried mriedem: O bugmeister supreme, do we have a bug open for this: http://logs.openstack.org/62/576462/3/check/openstack-tox-lower-constraints/77e13fb/testr_results.html.gz
14:30:48 efried We're comparing json serialization of a dict, which is happening in unpredictable order (predictably).
14:31:00 efried This is popping up all over the place intermittently.
14:31:23 mriedem i personally haven't opened a bug nor seen that yet no
14:31:33 efried ight.
14:32:15 mriedem sahid: you should triage this probably https://bugs.launchpad.net/nova/+bug/1779626
14:32:15 openstack Launchpad bug 1779626 in OpenStack Compute (nova) "MTU setting feature conflicts with minimum Qemu version " [Undecided,New]
14:36:25 sahid mriedem: we should probably mark it as duplicate of 1747496
14:36:28 sahid sounds good for you?
14:36:54 sahid oh wait... i think i missunderstood
14:37:41 mriedem the new bug is saying the change is a regression
14:38:29 sahid yes sorry
14:39:38 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: fix min_version for parent_provider_uuid in responses https://review.openstack.org/579577
14:39:38 sahid i'm supprised to see QEMU in relation to this, since it's libvirt who is creating the tap and configuring the mtu
14:39:44 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Always retrieve network information if available https://review.openstack.org/564444
14:39:45 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Retrieve tunneled status in '_get_physnet_info' https://review.openstack.org/564445
14:39:46 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add NUMACell.network_metadata https://review.openstack.org/564439
14:39:47 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add NUMATopologyLimits.networks https://review.openstack.org/575486
14:39:48 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Start accounting for networks in NUMA placement https://review.openstack.org/564448
14:39:49 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add RequestSpec.network_metadata https://review.openstack.org/564442
14:39:50 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452
14:39:51 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440
14:39:52 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441
14:57:15 mriedem sahid: btw, i updated https://review.openstack.org/#/c/551370/ so re-review on that would be appreciated
14:57:33 mriedem that's the thing where we can change vif type during live migration,
14:57:45 mriedem so i'm overwriting the source <interface> elements with the dest vif config
15:01:21 sahid mriedem: sure i will do, i'm curently investigating that issue with MTU, there is that host_mtu parameter added for virtio-net interface which i'm supprised to see libvirt returning an error when not supported by QEMU we (libvirt) should probably just pass
15:05:37 mriedem nova will have to workaround that
15:05:49 mriedem i.e. nova can't wait for libvirt to change
15:07:07 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
15:07:08 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
15:07:09 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
15:09:44 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
15:09:45 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
15:09:46 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
15:10:14 gibi an easy doc generation bug needs a second core https://review.openstack.org/#/c/579436
15:13:46 mriedem gibi: what's missing? i don't see a difference between http://logs.openstack.org/36/579436/1/check/build-openstack-sphinx-docs/7558c85/html/reference/notifications.html#versioned-notification-samples and https://docs.openstack.org/nova/latest/reference/notifications.html#versioned-notification-samples
15:13:54 mriedem unless the latter is still built using py3
15:13:56 mriedem *py2
15:15:52 gibi mriedem: let me try to reproduce it locally
15:20:26 mriedem gibi: locally i see the aggregate ones are missing at least
15:20:45 gibi mriedem: confirm
15:20:54 gibi mriedem: I mean I can confirm
15:21:19 mriedem +W
15:21:30 gibi mriedem: thanks
15:21:33 mriedem efried: does anyone have a patch up for those busted tests?
15:21:39 mriedem and/or bug
15:21:40 dansmith stephenfin: sounds like "the chat" never happened.. I have another call at the top of the hour and then I assume you'll be gone before I'm done
15:21:58 dansmith stephenfin: shall we plan for tomorrow morning? maybe get something on calendars so we don't let it go?
15:22:31 stephenfin Yeah, that sounds good to me. I don't think I've any conflicts tomorrow
15:22:37 stephenfin I'll just stick something in my calendar now
15:23:32 dansmith stephenfin: can you send invites to make sure we're all good?
15:24:51 stephenfin dansmith: Done
15:24:57 dansmith thanks
15:27:47 efried mriedem: Not that I am aware of, swhy I asked you.
15:27:55 mriedem efried: ok, you can do that also right?
15:28:23 efried mriedem: I took a look at it last week, and I'm not sure how to fix it without sorting it on both sides (test and prod).
15:28:41 efried mriedem: Otherwise I would have to, like, dissect the payload and deserialize the json.
15:28:52 mriedem if it's a json compare we have a jsonEquals helper or something
15:29:05 mriedem yeah we likely have to do that

Earlier   Later