Earlier  
Posted Nick Remark
#openstack-nova - 2017-08-29
19:23:41 mriedem ok. shouldn't the LIBS_FROM_GIT stuff be handled generically?
19:23:50 mriedem like, the job in project-config defines that i thought
19:24:02 mriedem and you had me do something like this for os-traits
19:24:07 mriedem to follow how the oslo libs are done
19:24:40 mriedem yeah, devstack/lib/libraries
19:25:33 mriedem https://github.com/openstack-dev/devstack/commit/aefc926cd45b2dc74d98f89e3a3b4cc92f2090ff
19:25:41 mriedem so i guess i can push something to do that for osc-placement
19:33:34 mriedem sdague: i should use pip_install_gr right?
19:33:46 sdague mriedem: yeh, though I think it all goes through that
19:33:59 sdague mriedem: most of the time the library is dragged in from requirements somewhere
19:34:03 mriedem pip_install_gr wraps pip_install
19:34:05 sdague but in the edge cases, like this
19:34:13 sdague where it's not in anyone's requirements list
19:34:18 sdague you need the fallback in code
19:34:36 mriedem hmm, i guess i can't use pip_install_gr yet since osc-placement isn't in global-requirements
19:34:50 sdague mriedem: we should fix that
19:34:57 sdague why isn't it in there yet?
19:35:04 mriedem nothing is using it?
19:35:13 sdague ah, ok, yeh chicken and egg
19:35:34 sdague is there a release yet?
20:19:13 mriedem osc-placement release is proposed
20:19:16 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Handle ComputeHostNotFound on startup https://review.openstack.org/489683
20:19:19 mriedem https://review.openstack.org/498924
20:31:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix nova assisted volume snapshots https://review.openstack.org/498845
20:35:15 mriedem ^ needs a +W to fix a multi-cell issue, which will go back to pike
20:38:28 efried jaypipes Is there any reason update_device_info() [names suck. feel free to suggest a better one.] needs to be separate from just get_inventory()?
20:57:56 dansmith mriedem: we hit that path because we literally have nothing more than the volume id to go on?
20:59:33 mriedem dansmith: yup, there is no instance in that API
20:59:48 mriedem this is the fun one that cinder calls back for FS-style backends
20:59:48 dansmith that sucks
20:59:51 mriedem like glusterfs and nfs
21:00:30 mriedem https://review.openstack.org/#/c/498510/
21:00:47 mriedem oops
21:00:50 mriedem https://developer.openstack.org/api-ref/compute/#create-assisted-volume-snapshots
21:01:23 mriedem maybe that create_info dict has the instance id in it
21:01:34 mriedem the api ref says that's optional and arbitrary
21:02:16 mriedem looking at the cinder code, they don't send it either
21:02:28 dansmith awesome
21:02:40 mriedem hey, thank your glusterfs red hat brethren
21:02:46 dansmith heh
21:02:48 mriedem from like, 4 years ago
21:03:35 mriedem so this only really impacts like SAP which uses NFS for everything
21:03:42 mriedem :)
21:07:41 openstackgerrit Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516
21:07:42 openstackgerrit Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537
21:07:42 openstackgerrit Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538
21:07:43 openstackgerrit Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542
21:07:43 openstackgerrit Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423
21:07:44 openstackgerrit Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664
21:07:47 openstackgerrit Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
21:07:48 openstackgerrit Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
21:07:49 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
21:07:49 openstackgerrit Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541
21:07:51 openstackgerrit Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486
21:08:00 openstackgerrit Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
21:08:01 openstackgerrit Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
21:08:03 openstackgerrit Michael Still proposed openstack/nova master: Replace dd with shred for zeroing lvm volumes. https://review.openstack.org/495532
21:08:03 openstackgerrit Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190
21:08:04 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
21:08:13 openstackgerrit Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438
21:08:20 mikal mriedem: that's the fix to those unit tests you wanted in the dd --> shred patch
21:08:34 mriedem aye
21:13:48 esberglu mriedem: For the PowerVM Driver work, would you prefer that we create a fresh blueprint or just retarget the existing one?
21:14:26 mriedem you can't retarget a completed blueprint
21:14:30 mriedem so it has to be new
21:14:44 esberglu okay tx. Will do
21:16:39 mriedem omg it's live https://developer.openstack.org/api-ref/placement/
21:16:40 mriedem finally
21:22:55 openstackgerrit Dan Smith proposed openstack/nova master: Move allocation manipulation out of drop_move_claim() https://review.openstack.org/498947
21:22:56 openstackgerrit Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949
21:22:56 openstackgerrit Dan Smith proposed openstack/nova master: Make allocation cleanup honor new by-migration rules https://review.openstack.org/498948
21:22:57 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Pre-create migration object https://review.openstack.org/498950
21:23:15 mriedem dansmith: i'm going through the spec for that right now btw
21:23:56 dansmith mriedem: okay I'm about to head out for a bit so take your time
21:27:42 mriedem ok posted
22:49:53 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: link to placement api-ref and history docs from main index https://review.openstack.org/498977
23:00:28 openstackgerrit melanie witt proposed openstack/nova master: Save updated libvirt domain XML after swapping volume https://review.openstack.org/498983
23:04:14 mriedem is swap volume ever going to be complete?
23:04:51 melwitt what you mean
23:06:18 mriedem what do YOU mean
23:07:25 melwitt I mean what do you mean by if it's going to be complete?
23:07:50 mriedem i'm joking that it's a never ending source of bugs
23:07:57 mriedem after all these years
23:08:04 melwitt oh yes
23:12:27 melwitt to your question, I tested it with a guest with persistent domain and it worked. delete_configuration is undefining the domain, which seems like a persistent config only thing
23:13:14 melwitt I changed it to read the live config, which AFAIK is completely separate from the persistent config. but I'm going to sanity-check with mdbooth or lyarwood to be sure
23:15:56 melwitt ergh, but this is about uefi support. which I didn't have in my repro env. so yeah I need to look into that
23:17:40 melwitt or wait, the uefi is just a flag for undefining the domain. so that call undefines the domain (removes the persistent config IIUC) whether there's uefi support or not
23:23:51 openstackgerrit Sean McCully proposed openstack/nova master: iso8601.is8601.Utc No Longer Exists https://review.openstack.org/498287
23:30:12 openstackgerrit Sean McCully proposed openstack/nova master: iso8601.is8601.Utc No Longer Exists https://review.openstack.org/498287
23:46:12 mikal mriedem: where did you want on nova-net? Is it going to heaven?
23:47:30 mikal s/want/land/
#openstack-nova - 2017-08-30
00:36:24 mriedem mikal: can't drop nova-net until we drop cells v1
00:36:40 mriedem and we can't drop cells v1 until we have some cells v2 limitations fixed, and probably actual multi-cell adoption
00:36:47 mriedem from large cells v1 users like CERN
00:56:59 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor ServerMovingTests for non-move tests https://review.openstack.org/498596
01:07:24 mikal Bummer, I was hoping to avoid privsepping nova-net
01:07:26 mikal Oh well
01:08:03 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379
01:08:03 openstackgerrit Alex Xu proposed openstack/nova master: placement: functional tests for multiple shared resource providers https://review.openstack.org/498737

Earlier   Later