| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-10 | |||
| 14:08:05 | openstackgerrit | sahid proposed openstack/nova master: libvirt: set emulator threads on isolated host cpus https://review.openstack.org/510897 | |
| 14:10:18 | jaypipes | dansmith, mriedem: in other news, I'm almost done with the new nested resource providers series. just fixing up a couple last unit testies. | |
| 14:10:32 | dansmith | I am literally on the edge of my seat | |
| 14:10:50 | jaypipes | dansmith: I can tell. | |
| 14:10:55 | mriedem | my ass is all the way to the back | |
| 14:11:01 | jaypipes | heh | |
| 14:16:54 | ericyoung | mriedem, sorry, was out a few days. is the emc scaleio ci running on all patches? if so, we should be able to | |
| 14:17:13 | ericyoung | mriedem, change it so it only runs ont he scaleio image backend | |
| 14:17:26 | mriedem | ericyoung: yeah it's been running on all changes i think | |
| 14:17:43 | jaypipes | mriedem: "running" might be an overstatement. | |
| 14:18:20 | mriedem | it has been responding to events on all changes and reporting failures on 99.99% of them | |
| 14:18:57 | ericyoung | Ack! ... ok, I | |
| 14:19:03 | ericyoung | will follow up. | |
| 14:21:11 | mriedem | ericyoung: i just noticed that when it does pass, it takes 3 hours? | |
| 14:21:12 | mriedem | EMC_ScaleIO_NOVASUCCESS in 2h 53m 01s | |
| 14:21:15 | mriedem | https://review.openstack.org/#/c/407440/ | |
| 14:21:19 | mriedem | that's nuts | |
| 14:21:33 | mriedem | are you guys literally building the server when it runs? :) | |
| 14:23:26 | ericyoung | I have to dig into it. When I run the tests manually on my development system (hardly very powerful)... it takes ~1 hour | |
| 14:25:50 | openstackgerrit | melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968 | |
| 14:25:50 | openstackgerrit | melanie witt proposed openstack/nova master: Make expected notifications output easier to read in tests https://review.openstack.org/510823 | |
| 14:25:51 | openstackgerrit | melanie witt proposed openstack/nova master: Improve the CellDatabases test fixture and usage https://review.openstack.org/508432 | |
| 14:43:32 | mriedem | johnthetubaguy: https://review.openstack.org/#/c/463987/ is waiting for your warm embrace | |
| 14:47:29 | openstackgerrit | Merged openstack/os-vif master: Move 'ips' field from Subnet object to VIF object https://review.openstack.org/508498 | |
| 14:50:16 | openstackgerrit | Merged openstack/nova master: Only query BDMs once in API during rebuild https://review.openstack.org/509574 | |
| 15:05:53 | abhi89 | Hi all.. can any stable core please review https://review.openstack.org/#/c/505546/.. waiting for one more +2 | |
| 15:34:11 | openstackgerrit | Merged openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | |
| 15:38:32 | dansmith | jaypipes: when you're done posting your nrp set, maybe you could vet all the db fu in here: https://review.openstack.org/#/c/500410/12 | |
| 15:38:40 | dansmith | it'd make it easier for me to just stamp it if you did | |
| 15:38:53 | jaypipes | ack | |
| 15:39:03 | jaypipes | I'll do it on the next test run | |
| 15:39:20 | dansmith | cool | |
| 15:40:34 | thingee | mriedem: re forum topics I'll send an update. | |
| 15:42:46 | dansmith | melwitt: are you respinning the bottom change for that notification target fix? | |
| 15:43:17 | melwitt | dansmith: yeah, figured it's best to have the comment explain what's going on since gibi figured it out | |
| 15:43:26 | dansmith | yep, just wanted to know if I should +W or not | |
| 15:43:31 | melwitt | instead of the "duh I dunno why this doesn't work" | |
| 15:43:47 | melwitt | yeah, k. one sec | |
| 15:44:22 | openstackgerrit | melanie witt proposed openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967 | |
| 15:44:23 | openstackgerrit | melanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells https://review.openstack.org/510691 | |
| 15:44:23 | openstackgerrit | melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968 | |
| 15:44:24 | openstackgerrit | melanie witt proposed openstack/nova master: Make expected notifications output easier to read in tests https://review.openstack.org/510823 | |
| 15:44:24 | openstackgerrit | melanie witt proposed openstack/nova master: Improve the CellDatabases test fixture and usage https://review.openstack.org/508432 | |
| 15:44:52 | openstackgerrit | sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580 | |
| 15:47:42 | mriedem | johnthetubaguy: i also went through ^ https://review.openstack.org/#/c/375580/ yesterday, and i think when you originally wrote that it was before we had a plan for how to do the new style volume attachment stuff, but i think we'll want to do similar with the port bindings API | |
| 15:48:11 | mriedem | i.e. turn it on once everything is new enough to use it and then we can key move operations off whether or not the port was attached using the port binding API | |
| 15:52:21 | jaypipes | dansmith, cdent: remind me again why this sentence is true: "have an AllocationList from multiple project ids, something we want to be able to do with the new use of migration uuids." | |
| 15:52:51 | dansmith | jaypipes: it's not because of the migration change, | |
| 15:52:53 | jaypipes | dansmith, cdent: how does using a migration UUID as a consumer mean we need allocations with >1 user or project? | |
| 15:53:11 | jaypipes | oh, ok | |
| 15:53:13 | dansmith | jaypipes: we just said that if we're making this change, we might as well let you swap allocations atomically, not just within one user/project | |
| 15:53:19 | jaypipes | got it. | |
| 15:53:23 | jaypipes | ok, thanks | |
| 15:53:49 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | |
| 15:54:18 | dansmith | this needs +W, which is just minor feedback rollup from the earlier set of patches: https://review.openstack.org/#/c/506416 | |
| 15:54:27 | dansmith | in case anyone needs stats padding | |
| 15:55:40 | dansmith | melwitt: in case you haven't seen, these three patches further improve listing: https://review.openstack.org/#/c/508595/7 | |
| 15:56:02 | dansmith | including BRs in the merge, so that they sort normally instead of the weird "always first" behavior | |
| 15:56:03 | dansmith | and | |
| 15:56:18 | dansmith | we only scatter to cells that you have instances in | |
| 15:57:31 | jaypipes | dansmith: https://review.openstack.org/#/c/506416/ +W'd | |
| 15:57:52 | dansmith | jaypipes: thanks | |
| 15:58:26 | melwitt | dansmith: a-ha, cool. thanks for the heads up, will review | |
| 16:08:19 | openstackgerrit | sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580 | |
| 16:14:54 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 16:18:25 | mriedem | oh yay we regressed volume detach API semantics in pike | |
| 16:18:54 | sean-k-mooney | mriedem: oh how so? | |
| 16:18:59 | mriedem | we introduced a race | |
| 16:19:12 | mriedem | by detaching the volume in cinder before deleting the bdm in nova | |
| 16:19:22 | mriedem | it changes the behavior of the volume attachments API | |
| 16:19:31 | mriedem | and tempest relies on that behavior | |
| 16:19:42 | mriedem | i.e. tempest handles a 404 differently from a 400 when detaching a volume | |
| 16:20:03 | sean-k-mooney | so is that why my instaces that i booted from volume form image with delete volume on delete instance nolonger deletes the volume | |
| 16:20:05 | mriedem | arguably it could be a tempest bug or bad client expectations | |
| 16:20:24 | mriedem | that sounds different | |
| 16:20:39 | mriedem | that's a delete_on_termination thing | |
| 16:20:52 | sean-k-mooney | im not sure if its intended or not as i only noticed it happening when the instance goes to error state and i delete it | |
| 16:22:21 | mriedem | error state during scheduling? | |
| 16:22:24 | mriedem | or during building on the host? | |
| 16:22:24 | sean-k-mooney | mriedem: well tempest is ment to work across releases so if this changes across release it an api change | |
| 16:22:48 | mriedem | sean-k-mooney: well, in this case, tempest is requesting to detach a volume that is already detached | |
| 16:22:52 | mriedem | it's part of a cleanup routine | |
| 16:22:55 | sean-k-mooney | mriedem: durign building basically when the instance fails to boot due to port binding failed | |
| 16:23:02 | mriedem | arguably the client shouldn't be making that request | |
| 16:23:39 | sean-k-mooney | mriedem: ah ok ya that sound like a client bug as the precondition is not ment. e.g. it must be attached to detach it | |
| 16:23:41 | mriedem | sean-k-mooney: are you sure you're setting delete_on_termination=True for the bdm? | |
| 16:24:41 | openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | |
| 16:24:48 | efried | mriedem Should removal of cinder v2 support also have removed volumev2 (and maybe also volume) from https://github.com/openstack/nova/blob/master/nova/context.py#L121 ? | |
| 16:24:50 | sean-k-mooney | i was booting from horizon and setting the Delete volumne on onstance delete option | |
| 16:25:01 | sean-k-mooney | so i assume that set it under the hood | |
| 16:25:58 | mriedem | efried: good point, probably yeah | |
| 16:26:05 | mriedem | did we remove volumev2 support already? | |
| 16:26:08 | mriedem | i think i pushed the change for it... | |
| 16:26:11 | efried | yeah | |
| 16:26:16 | efried | swhy I tagged ya | |
| 16:26:41 | mriedem | ah yeah https://github.com/openstack/nova/commit/eadbacbda628ecc969a980378faf55bc02f514bf#diff-d24869e36361bb430acd67658a83b0b5 | |
| 16:26:47 | mriedem | yeah we should remove volume and volumev2 from context | |
| 16:27:11 | efried | mriedem But include block-storage | |
| 16:27:23 | efried | since that's the official service type name | |
| 16:27:38 | efried | though I don't think devstack is setting that up yet. | |