| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-29 | |||
| 18:58:45 | mriedem | nova/tools/hooks/post_test_hook.sh | |
| 18:59:23 | sean-k-mooney | soory got confued ignore the collectd comment but what i was goning to say is im not sure if post_test_hook.sh runs after the logs are upladed or not | |
| 18:59:43 | mriedem | oh it's only in the nova-next job | |
| 19:00:05 | mriedem | kablam! http://logs.openstack.org/61/496861/1/check/gate-tempest-dsvm-neutron-nova-next-full-ubuntu-xenial-nv/572ceed/logs/devstack-gate-post_test_hook.txt.gz | |
| 19:00:13 | cdent | sean-k-mooney: I was thinking curl because of something quick and dirty that wouldn’t necessarily be a permanently running test, rather a diagnostic tool for nowish | |
| 19:00:32 | mriedem | cdent: so it would be cool to write something that queries for allocations in the post test hook at the end of a job to see if they are 0 or not | |
| 19:00:33 | cdent | also, what’s hard about getting a token? | |
| 19:00:59 | mriedem | anyway, i'll post the idea to the ML and people can think about it | |
| 19:01:04 | sean-k-mooney | cdent: more you just need to do that first befor just calling curl and add it to the correct header | |
| 19:01:25 | cdent | mriedem: would we’d need to track instance ids throughout the test or can we simply get all instances, including deleted ones? | |
| 19:01:32 | sean-k-mooney | cdent: if you want something quick just run a mysql command directly against the db | |
| 19:02:04 | sean-k-mooney | if there are still allocations in the output you know something is messed up | |
| 19:02:09 | cdent | sean-k-mooney: I tend do to a lot of export TOKEN=$(openstack token issue …); curl blah | |
| 19:02:40 | sean-k-mooney | cdent: huh i didnt know you could do openstack token issue ... | |
| 19:02:42 | cdent | but yeah, a db query would be fine too, but…boring :) | |
| 19:02:49 | sean-k-mooney | that makes things alost eaisier | |
| 19:03:10 | cdent | also it would mean that we are assuming that placement will always have a db backend ;) | |
| 19:03:19 | sean-k-mooney | any time i have used curl like that i have manually used curl to get the token by calling keysone myself which is a pain | |
| 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 | |