Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-29
19:03:28 cdent ouch. yeah. that is painful
19:05:32 sean-k-mooney cdent: and yes i am assiming a db backend. honestly i spend enough time getting our ci test to stop going to the db directly and use the api instead so ya curl would be better if its going to last long enough for db assumtions to expire
19:06:13 sean-k-mooney anyway time for food, see ye tomorow
19:07:27 mriedem cdent: i think we'd have to account for instances that aren't yet deleted
19:07:44 mriedem cdent: but, the post_test_hook stuff doesn't fail the job i don't think, so this would just be an early report to start
19:07:53 mriedem so we can kick ideas around
19:20:58 mriedem sdague: do you just want the commit message updated for this? https://review.openstack.org/#/c/457636/6/lib/placement
19:21:49 sdague mriedem: ok, this is long enough ago, I'm going to have to regain context
19:22:09 sdague I think the question is... do you always want that library installed?
19:22:34 sdague because, that won't install it in the normal case
19:22:41 sdague only when you specify it as LIBS_FROM_GIT
19:22:43 mriedem yes, we are going to want it in all cases i think
19:22:54 sdague which won't give you the release version, only the master version
19:22:57 mriedem right,
19:23:09 sdague ok, you you'll need an else
19:23:11 mriedem i think roman did that because he was working on getting an osc placement functional test job to run from a proposed change
19:23:21 sdague that does a pip_install osc-placement
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

Earlier   Later