Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-28
19:41:00 edleafe mriedem: will do
19:42:06 sdague mriedem: https://review.openstack.org/#/c/498477 and https://review.openstack.org/#/c/498478 are redone with the git sha
19:42:40 mriedem sdague: ok, but i'm going to hold off on stable/pike stuff until we release
19:42:51 sdague mriedem: that's fine
19:43:02 sdague it will still have real test results on it
19:43:19 openstackgerrit Steve Noyes proposed openstack/nova master: update live migration to use v3 cinder api https://review.openstack.org/463987
19:43:24 sdague I was less concerned for the release, but just more for "these things should be in there sometime"
19:46:08 openstackgerrit Ed Leafe proposed openstack/nova master: Clarify the field usage guidelines https://review.openstack.org/498581
19:46:10 edleafe mriedem: quick note clarification ^^
20:24:47 openstackgerrit Dan Smith proposed openstack/nova master: Add placeholder migrations for Pike backports https://review.openstack.org/496932
20:24:47 openstackgerrit Dan Smith proposed openstack/nova master: Add uuid to migration object and migrate-on-load https://review.openstack.org/496934
20:24:48 openstackgerrit Dan Smith proposed openstack/nova master: Add uuid online migration for migrations https://review.openstack.org/498589
20:24:48 openstackgerrit Dan Smith proposed openstack/nova master: Add uuid to migration table https://review.openstack.org/496933
20:25:02 dansmith mriedem: that placeholder one failed out of the gate if you want to hit it again ^
20:25:40 mnaser (sorry, trying to get some help from another project that might have ran into this: i've discovered a performance issue in sahara and identified it to sqlalchemy lazy='joined' in the relationships which results in a giant set of joins)
20:25:52 mnaser i noticed the nova code had none of that. was that a decision that was taken or just.. no one ever set that to anything
20:26:13 mnaser i figure nova is probably one of the longer codebases out here that might have ran into these perofmance regressions so i'd ask around
20:26:40 dansmith mnaser: we generally join things explicitly when we want them
20:27:06 dansmith intentionally.
20:28:14 mnaser dansmith makes perfet sense. looking at the git blame.. looks like it was just magically changed with no explanation. ill do my checks
20:28:16 mnaser thank you for that info
20:49:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor ServerMovingTests for non-move tests https://review.openstack.org/498596
20:50:29 mriedem dansmith: ^ as you asked for
20:50:38 dansmith ah thanks
20:52:12 openstackgerrit Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516
20:52:13 openstackgerrit Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537
20:52:13 openstackgerrit Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538
20:52:14 openstackgerrit Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542
20:52:14 openstackgerrit Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423
20:52:15 openstackgerrit Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664
20:52:16 openstackgerrit Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
20:52:16 openstackgerrit Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
20:52:17 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
20:52:17 openstackgerrit Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541
20:52:18 openstackgerrit Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486
20:52:21 openstackgerrit Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
20:52:22 openstackgerrit Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
20:52:23 openstackgerrit Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190
20:52:24 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
20:52:24 openstackgerrit Michael Still proposed openstack/nova master: Replace dd with shred for zeroing lvm volumes. https://review.openstack.org/495532
20:52:26 openstackgerrit Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438
20:53:10 mikal sdague: that's me rebasing that series, thanks for the review on the first one
20:53:53 mikal sdague: https://review.openstack.org/#/c/495532/ is a similarly trivial head-of-series refactor if you're bored
20:54:23 mikal sdague: well, "refactor" might be strong. "removal of historical silliness" might be more accurate.
20:55:19 sdague sure, it's about end of day here, so I can look tomorrow
20:55:27 edleafe mikal: I prefer "hysterical silliness", myself
21:00:37 mriedem mikal: your test removal in the bottom change cuts a bit too deep
21:03:48 mriedem wow
21:03:49 mriedem "Took 269.99 seconds to spawn the instance on the hypervisor."
21:04:08 mriedem clarkb: ^
21:04:12 cfriesen_ is that running on a raspberry pi?
21:04:18 mriedem ubuntu-xenial-vexxhost-ca-ymq-1-10660012
21:05:46 mriedem that was on http://logs.openstack.org/94/498094/3/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/8e0a0d0/
21:06:51 mikal mriedem: as in on that shred change?
21:07:06 mriedem mikal: yeah
21:07:18 mikal mriedem: hmmm, I'll take a look today
21:07:20 mriedem mikal: the slow node thing above is not that change
21:07:26 mriedem just the test removal
21:09:34 clarkb mriedem: what image is being booted? that looks normal for ubuntu
21:10:19 mriedem cirros
21:11:28 mriedem cf0d1557-c4b3-4304-b4ec-5e6eaf4a7123
21:11:43 mriedem http://logs.openstack.org/94/498094/3/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/8e0a0d0/logs/devstacklog.txt.gz#_2017-08-28_14_49_41_357
21:11:43 mriedem sorry
21:11:50 mriedem cirros-0.3.5-x86_64-disk
21:14:01 mriedem looking at most of these "seconds to spawn the instance on the hyperviso" messages, most are around 2 seconds
21:14:06 mriedem but some just spike to like 1-2+ minutes
21:19:53 openstackgerrit Merged openstack/nova master: rename binary to source in versioned notifications https://review.openstack.org/487126
21:23:42 openstackgerrit Merged openstack/nova master: Use enum value instead of string service name https://review.openstack.org/476538
21:27:20 openstackgerrit Merged openstack/nova master: Cleanup unnecessary logic in os-volume_attachments controller code https://review.openstack.org/485823
21:53:41 openstackgerrit Merged openstack/nova-specs master: Return alternate allocation requests to scheduler https://review.openstack.org/471927
22:02:25 mriedem stephenfin: efried: was the claims part of scheduling going to get built into this at some point? https://review.openstack.org/#/c/475810/
22:05:32 efried mriedem I have to defer to stephenfin / edleafe / whoever. I just made the words fit in the boxes.
22:06:34 mriedem http://docs-draft.openstack.org/10/475810/11/check/gate-nova-docs-ubuntu-xenial/5b58371//doc/build/html/reference/scheduling.html
22:06:37 mriedem they don't fit for me
22:08:08 efried mriedem Two changes built on top of that one make 'em fit.
22:08:25 efried And I think they merged. Where's the official link for that doc?
22:11:04 edleafe efried: mriedem: yeah, that will have to be updated in the future. It's been a moving target.
22:11:09 mriedem efried: https://docs.openstack.org/nova/latest/reference/index.html
22:11:11 mriedem it's not posted yet
22:11:17 mriedem the content is also wrong
22:11:22 mriedem so i'm not sure why this was merged as is
22:12:19 efried mriedem Well, for what it's worth, the wrong content is formatted properly via http://docs-draft.openstack.org/04/476204/5/check/gate-nova-docs-ubuntu-xenial/b74b92c//doc/build/html/reference/scheduling.html
22:12:56 mriedem heh, ok, well at least there is that
22:13:06 mriedem now i can read what the wrong words are
22:14:19 mriedem the diagram looks like ocata, the text looks like pike
22:14:25 mriedem and the alternatives stuff is queens
22:16:37 cfriesen my VPN is going nuts...grr
22:19:16 mriedem i'll see if i can adjust this a bit
22:35:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Updates to scheduling workflow doc https://review.openstack.org/498613
22:35:38 mriedem here we are ^
22:50:17 openstackgerrit Jamie Lennox proposed openstack/nova master: Remove unneeded attributes from context https://review.openstack.org/498615
22:52:02 cfriesen is it just me or is the response to a "GET /flavors/{flavor_id}/os-flavor-access" kind of odd? It looks like the flavor id will be returned once per access rule, which seems somewhat redundant.
23:21:10 beagles are the devs working on os-vif in this channel, neutron, or somewhere else?
23:21:51 beagles sean-k-mooney, ^
23:27:26 gmann mriedem, i agree on bug fix but for POST aggregate API its changing 200 to 400
23:28:06 mriedem beagles: here mostly i think
23:28:29 mriedem gmann: yeah but it was a wrong 200 which probably led to a later NoValidHost anyway
23:28:36 beagles mriedem, k thanks
23:29:44 gmann mriedem, humm
#openstack-nova - 2017-08-29
00:09:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional recreate test for live migration pre-check fails https://review.openstack.org/498627

Earlier   Later