| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-13 | |||
| 19:45:55 | imacdonn | mriedem: OK, well, just a heads-up in case you see it | |
| 19:46:04 | mriedem | ack | |
| 19:46:29 | imacdonn | other than that, ZFSSA CI seems to work (with multiattach) with the UCA | |
| 19:46:47 | mriedem | outstanding | |
| 19:47:55 | superdan | jgwentworth: sure seems reasonable | |
| 19:48:12 | jgwentworth | k, thanks | |
| 19:49:47 | openstackgerrit | melanie witt proposed openstack/nova-specs master: Document Rocky review priorities https://review.openstack.org/557796 | |
| 19:56:34 | mriedem | any opinions on removing the redundant unit tests from tssurya's patch here? https://review.openstack.org/#/c/554920/10/nova/tests/unit/compute/test_host_api.py@439 the functional test in that change will now cover that | |
| 19:57:54 | openstackgerrit | Jay Pipes proposed openstack/nova master: support multiple member_of qparams for GET /r-ps https://review.openstack.org/561315 | |
| 19:58:34 | leakypipes | superdan: that should help when we need >1 placement req filters... | |
| 20:00:24 | openstackgerrit | Jay Pipes proposed openstack/nova master: support multiple member_of qparams for GET /r-ps https://review.openstack.org/561315 | |
| 20:02:31 | openstackgerrit | Jay Pipes proposed openstack/nova master: support multiple member_of qparams for GET /r-ps https://review.openstack.org/561315 | |
| 20:08:41 | jgwentworth | mriedem: yeah, seems like that unit test can go | |
| 20:09:05 | jgwentworth | I don't see it covering anything additional unless I'm missing something | |
| 20:09:28 | superdan | leakypipes: that's implementing what we said int the spec revision right? | |
| 20:10:33 | leakypipes | superdan: yessir | |
| 20:10:45 | superdan | cools | |
| 20:16:10 | jgwentworth | wow, assert_called_once() is a thing. *mind blown* | |
| 20:42:44 | mriedem | using fixtures as context managers - put that in your bag of tricks too for when a test needs to conditionally use a fixture | |
| 20:43:35 | jgwentworth | I think I've done that before already with MonkeyPatch fixture | |
| 20:43:43 | jgwentworth | cause I'm cool like that | |
| 20:49:03 | mriedem | unfortunately our api fixtures might not be setup properly to handle running them as fixtures... | |
| 20:49:05 | mriedem | dagnabbit | |
| 20:55:15 | mriedem | cdent: i know you probably want to figure this out on your friday night, but it's not clear to me that we stop the wsgi app when PlacementFixture or OSAPIFixture tear down | |
| 20:56:41 | mriedem | specifically, i get a timeout b/c the request seems to hang on a connection error at L77 http://paste.openstack.org/show/719198/ | |
| 20:57:11 | mriedem | http://paste.openstack.org/show/719199/ | |
| 21:01:48 | openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | |
| 21:05:28 | jgwentworth | jackie-truong: hey, thanks for the quick update :) I'm going through the last patch now and to peruse the tempest test logs. sorry for the delay on review | |
| 21:06:18 | jackie-truong | jgwentworth: np! thanks :-) | |
| 21:21:29 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | |
| 21:22:18 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | |
| 21:34:37 | cdent | mriedem: Imma gonna need you to parse that a bit more for me, or I can think harder about it some other time, but at the moment, I don't get what you're saying supposed to be happening there | |
| 21:35:42 | mriedem | i'm just going to leave it as a note in my test for now, | |
| 21:35:51 | mriedem | but in that test i'm using the placement fixture and the compute api fixture, | |
| 21:36:03 | mriedem | but i need the placement fixture to be used conditionally based on what i'm doing, | |
| 21:36:25 | mriedem | so i use it when i'm creating a server because the compute service needs it, but then i don't want placement fixture in place when i delete the server | |
| 21:36:36 | mriedem | but when i go to delete the server i get this weird connection error, | |
| 21:36:48 | mriedem | so i'm not sure if/why the placement fixture is messing with the osapifixture | |
| 21:37:06 | mriedem | i figured maybe the placement fixture isn't cleaning up after itself properly | |
| 21:37:23 | cdent | that a url for the compute api though? | |
| 21:38:11 | mriedem | Max retries exceeded with url: /v2.1/6f70656e737461636b20342065766572//servers/ee29ce1f-ca1e-4466-84cd-23d57bb70206 | |
| 21:38:11 | mriedem | yeah | |
| 21:38:54 | cdent | is there a check behind that compute api call for the placement api? that is, at that step of the game, how is placement involved? | |
| 21:38:59 | mriedem | wsgi-intercept is working on unique endpoints between placement and osapifixture so i'm not sure how things could overlap | |
| 21:39:20 | cdent | is there any asynchrony/eventlet/similar magic going on? | |
| 21:39:22 | mriedem | the delete flow in this patch now adds a call to the scheduler report client to delete allocations for the instance | |
| 21:39:36 | mriedem | which i was hoping would bomb out and @safe_connect would handle it | |
| 21:39:49 | mriedem | since placement fixture wouldn't be in place when the server delete call happens | |
| 21:40:02 | mriedem | but...it's likely trying to hit ksa...? | |
| 21:40:47 | cdent | if you push the code up somewhere and add me to the review I'll look at it tomorrow (eagerly even, because I'm curious) | |
| 21:41:21 | mriedem | ok; i might know how to do this w/o the ugly stub i've got in place too, will test | |
| 21:41:23 | cdent | it's entirely possible there are some global bits in wsgi-intercept being naughty (it is old and gross) | |
| 21:41:37 | cdent | and if we can find them that would be a nice fix | |
| 21:42:27 | cdent | if nothing else, I'll play with the paste tomorrow | |
| 21:43:15 | mriedem | it's bugging me so i'll have something up tonight most likely and just add yo uto the review | |
| 21:43:58 | cdent | I'm going to not get sucked in for of fear of being glared at by family | |
| 21:54:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Block deleting compute services which are hosting instances https://review.openstack.org/560674 | |
| 21:54:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete allocations from API if nova-compute is down https://review.openstack.org/560706 | |
| 21:54:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup RP and HM records while deleting a compute service. https://review.openstack.org/554920 | |
| 22:47:06 | openstackgerrit | Merged openstack/nova master: Rename network.utils to network.linux_utils. https://review.openstack.org/559197 | |
| 22:47:24 | openstackgerrit | Merged openstack/nova master: Add microversion to support extra_specs in flavor API. https://review.openstack.org/554159 | |
| 22:47:36 | openstackgerrit | Merged openstack/nova stable/queens: Remove the branch specifier from the nova-multiattach job https://review.openstack.org/560931 | |
| 22:47:48 | openstackgerrit | Merged openstack/nova master: Skip placement on rebuild in same host https://review.openstack.org/546357 | |
| 23:00:19 | jgwentworth | mriedem: fyi I see the ceph job on master is busted, looking into what's going on http://logs.openstack.org/49/479949/58/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/eecfbb1/logs/devstacklog.txt.gz#_2018-04-13_21_24_31_947 | |
| 23:00:57 | jgwentworth | I don't see anything wrong with that identity URL | |
| 23:27:40 | lbragstad | leakypipes: thanks for the review on https://review.openstack.org/#/c/553613/2 | |
| 23:28:04 | lbragstad | i tried to answer some of your comments, but let me know if bits are still unclear | |
| 23:34:17 | lbragstad | melwitt: http://logs.openstack.org/49/479949/58/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/eecfbb1/logs/screen-keystone.txt.gz#_Apr_13_21_24_28_551060 looks new | |
| 23:36:09 | lbragstad | looks like a missing dependency of paste | |
| 23:47:06 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | |
| 23:53:43 | melwitt | lbragstad: oh, thank you! | |
| 23:54:01 | melwitt | for whatever reason, I didn't think to look in the keystone log. my bad | |
| 23:54:12 | lbragstad | melwitt: yep - not sure what would have caused that though? | |
| 23:54:35 | melwitt | yeah, that I don't know. it's really recent, started within the last couple of days I think | |
| 23:54:44 | lbragstad | huh | |
| 23:54:53 | lbragstad | sounds like it's consistent? | |
| 23:55:24 | melwitt | I think it is, yeah. I've seen a few examples so far | |
| 23:55:45 | melwitt | I feel like something like this has happened before too and I can't remember what it was, of course | |
| 23:56:34 | melwitt | at least I have something to go on now, so I'll find it eventually | |
| #openstack-nova - 2018-04-14 | |||
| 00:00:25 | melwitt | this is what I was thinking of https://review.openstack.org/#/c/510660/ | |
| 00:01:22 | lbragstad | oh intereting | |
| 00:01:27 | lbragstad | interesting* | |
| 00:01:41 | melwitt | must be something similar since we updated to the queens uca this week | |
| 00:01:49 | lbragstad | oh | |
| 00:01:51 | melwitt | except urlmap this time | |
| 00:02:03 | lbragstad | sure | |
| 00:02:41 | lbragstad | looks like it happened with PasteDeploy==1.5.2 | |
| 00:04:18 | lbragstad | nevermind, just read sean's comment | |
| 00:08:29 | melwitt | versions of paste and pastedeploy are the same between a successful (different job) and my failing run :\ | |
| 00:08:59 | lbragstad | weird - https://lists.launchpad.net/openstack/msg05092.html | |
| 00:09:16 | lbragstad | circa 2011 | |
| 00:09:19 | melwitt | heh | |
| 00:09:52 | melwitt | well, I think it's the same thing where paste is installed from the uca and pastedeploy from pypi and so pastedeploy can't find paste that it depends on, I think | |
| 00:10:01 | lbragstad | yeah - i think you're right | |
| 00:10:03 | melwitt | maybe the force update is actually screwing us now | |
| 00:10:08 | melwitt | the old hack | |
| 00:10:11 | lbragstad | go figure | |
| 00:10:21 | melwitt | yeah, lol. they "fixed the glitch" in the uca | |
| 00:11:07 | melwitt | gonna try a revert of that and see if it passes. thanks for helping look at this lbragstad | |
| 00:11:23 | lbragstad | melwitt: anytime | |