Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-01
21:57:51 tssurya mriedem : https://bugs.launchpad.net/nova/+bug/1746509
21:57:53 openstack Launchpad bug 1722404 in OpenStack Compute (nova) ocata "duplicate for #1746509 Database transactions can fail with "TypeError: Can't upgrade a READER transaction to a WRITER mid-transaction" because of scatter_gather_cells" [Undecided,In progress] - Assigned to Matt Riedemann (mriedem)
21:57:59 tssurya the one you mentioned in the meeting
21:58:48 mriedem ah that's not scatter/gather
21:59:05 mriedem and the ocata backport isn't ready yet, unit tests are failing, i haven't dug into how to fix those yet
21:59:35 mgagne mriedem: I don't use IsolatedHostsFilter because you would need to redeploy your config to add new hosts/images, that's suboptimal.
22:00:18 mgagne but good point about image_ref not being available with BFV, I'm not sure how we didn't catch that one in a similar filter we wrote
22:02:37 mriedem the request spec is also not good at telling anything if it's boot from volume or not
22:03:10 mriedem it doesn't even have the original bdms requested in it
22:03:21 mgagne I would have to dig into it a bit
22:03:26 mriedem so you'd basically have to assume that if the request spec doesn't have an image, it's volume-backed
22:04:00 mriedem maybe bauzas has thoughts on that one
22:07:48 mriedem rgerganov: can you take a look at this? (or someone from vmware?) https://bugs.launchpad.net/nova/+bug/1744182
22:07:49 openstack Launchpad bug 1744182 in OpenStack Compute (nova) "can not create instance when using vmware nova driver" [Undecided,New]
22:09:40 melwitt mriedem: commented on the review, I think yielding the copy of the context won't help because the callers aren't using the yielded context
22:10:09 melwitt I'm not sure if there's any clever thing we can do instead of having to fix all the callers in ocata
22:11:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional tests for traits-based scheduling https://review.openstack.org/539310
22:12:02 mriedem oh right...
22:12:07 mriedem guh
22:12:14 melwitt yeah :(
22:14:44 melwitt mriedem: looks like you've already documented the lack of neutron event in certain situations https://github.com/openstack/nova/blob/stable/ocata/nova/virt/libvirt/driver.py#L7508-L7512
22:15:51 mriedem that's the 2nd time i've looked at that today for different reasons
22:16:24 mriedem i don't remember writing that though
22:16:28 mriedem might have been blacked out
22:16:51 melwitt yeah. ugh, it's my fault I even told lyarwood not to pass vifs_already_plugged=True :(
22:17:17 melwitt suck
22:19:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional tests for traits-based scheduling https://review.openstack.org/539310
22:19:06 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: placement upgrade notes for queens https://review.openstack.org/539658
22:19:22 mriedem alex_xu: gibi: had to rebase that approved functional test ^
22:20:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
22:20:27 mriedem time to leave, i've got single parenting duty tonight
22:30:10 openstackgerrit melanie witt proposed openstack/nova master: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540168
22:33:53 openstackgerrit Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648
22:33:54 openstackgerrit Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246
22:33:54 openstackgerrit Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821
22:33:55 openstackgerrit Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517
22:33:55 openstackgerrit Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
22:33:56 openstackgerrit Eric Fried proposed openstack/nova master: Make generation optional in ProviderTree https://review.openstack.org/539324
22:34:18 efried jaypipes: ^ rebased and now tagged under bp update-provider-tree
22:42:47 jaypipes efried: cool, thanks. will review shortly.
22:42:50 jaypipes after dinner...
22:42:59 efried thx
22:47:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
22:48:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
22:48:29 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Move body examples to an isolated directory https://review.openstack.org/529998
22:49:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
22:49:42 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
22:50:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
22:50:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
22:51:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
23:22:40 openstackgerrit Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax (Rocky) https://review.openstack.org/540179
23:22:52 efried mriedem_parent, jaypipes: Re-proposed ^
23:23:09 efried dansmith too - this was your favorite spec
23:32:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
23:55:56 openstackgerrit Merged openstack/nova master: conf: Validate '[api] vendordata_providers' options https://review.openstack.org/513318
#openstack-nova - 2018-02-02
00:11:52 Spazmotic Morning novers
00:39:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
01:16:09 openstackgerrit Merged openstack/nova master: Fix invalid UUIDs in remaining tests https://review.openstack.org/539254
01:37:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
01:57:57 openstackgerrit Artom Lifshitz proposed openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266
02:02:12 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/539841
02:02:43 openstackgerrit Merged openstack/nova master: trivial: Fix few policy doc https://review.openstack.org/539780
02:02:56 openstackgerrit Merged openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042
02:22:49 Spazmotic Hey jianghuaw if you get a chance today could you take a look at Bob's comment on the Live Migration commit? If he's correct here we can just remove almost everythign except the swallow on this
02:23:05 Spazmotic Hopefully in that case with such a simple fix we can get it pushed through the gate quickly and move on with our lives for right now :D
02:27:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
02:46:25 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Improve performance when list instances with IP filter https://review.openstack.org/539469
02:50:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
02:54:54 jianghuaw Spazmotic, for OpenStack live-migration, the VDI_NOT_IN_MAP exception was found in the XS7 and forwarding releases.
02:55:14 Spazmotic So no way for it to get thrown previous to that?
02:55:18 jianghuaw Spazmotic, but the error type was added long ago: https://github.com/xapi-project/xen-api/commit/6eaad3b6745c0392ac3b3ef91c33451b1f110aed
02:55:35 Spazmotic That was my concern.. can we assume it is possible for it to trigger legitimately still outside of that path?
02:56:04 jianghuaw So I wondering there is case to trigger this error somehow; but I don't know how to trigger it in OpenSTack.
02:56:11 Spazmotic hehe same here..
02:56:37 Spazmotic I have access to some 6.5 test boxes, I could run some fake asserts directly through Python i guess
02:57:23 jianghuaw yes, it's worthy for the test.
02:57:49 Spazmotic I'll give it a try tonight, try with some empty sets and such and see if I can emulate it. I think it may be worthwhile to keep the logic in either way though considering this fault did exist since 6.0
02:58:03 Spazmotic Evne if we don't know how to cause it lol
02:58:32 jianghuaw agree.
02:59:06 Spazmotic Thanks for your time sir :)
02:59:16 jianghuaw np:-)
03:18:29 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
03:40:36 openstackgerrit Merged openstack/nova master: Migrate "launch instance" user guide docs https://review.openstack.org/540048
03:40:48 openstackgerrit Merged openstack/nova master: Cleanup launch instance and manage IPs docs https://review.openstack.org/540121
03:41:10 openstackgerrit Merged openstack/nova stable/pike: Use UEFI as the default boot for AArch64 https://review.openstack.org/531481
04:04:35 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161
04:14:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299
04:44:53 Spazmotic jianghuaw - One more question sir, sorry. It looks like this change is for the actual migrate_send method, does the assert try that when it runs assert_can_migrate, or does it have its own method for that test?
04:53:35 openstackgerrit Takashi NATSUME proposed openstack/nova stable/ocata: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513721
04:59:21 Spazmotic Interesting in my test environment running into relaxed SR errors.. thought that was default on now..
05:01:16 Spazmotic Ahh.. yeah Huan did a change to this that i bet we don't have
05:03:09 Spazmotic eugh. .yeah
05:03:26 openstackgerrit Merged openstack/nova master: Add functional tests for traits-based scheduling https://review.openstack.org/539310
05:30:05 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/540256
05:31:22 Spazmotic Well that's literally the only result for VDI_NOT_IN_MAP in the git log for xenapi so i'm guessing that's the one hehe
05:47:20 openstackgerrit melanie witt proposed openstack/nova master: Make InstanceGroup.get_hosts query all cells https://review.openstack.org/540258
05:51:27 openstackgerrit Merged openstack/nova master: Reduce policy deprecation warnings in test runs https://review.openstack.org/537603
05:51:36 openstackgerrit Merged openstack/nova master: Remove deprecated policy items from fake_policy https://review.openstack.org/537600
06:02:09 openstackgerrit Merged openstack/nova master: Log the events we timed out waiting for while plugging vifs https://review.openstack.org/538103
06:23:47 openstackgerrit chenxiangui proposed openstack/nova master: Fix the wrong description https://review.openstack.org/540265

Earlier   Later