Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-13
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 yeah
21:38:11 mriedem Max retries exceeded with url: /v2.1/6f70656e737461636b20342065766572//servers/ee29ce1f-ca1e-4466-84cd-23d57bb70206
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
00:47:13 openstackgerrit Merged openstack/nova master: Add host to API and Conductor https://review.openstack.org/556513
00:47:24 openstackgerrit Merged openstack/nova master: Address nits in Idf57fb5fbc611abb83943bd7e36d3cebf03b3977 https://review.openstack.org/561205
00:47:54 openstackgerrit Merged openstack/nova master: [placement] Parse forbidden traits in query strings https://review.openstack.org/556819
00:48:07 openstackgerrit Merged openstack/nova master: [placement] Filter resource providers by forbidden traits in db https://review.openstack.org/556472
00:48:32 openstackgerrit Merged openstack/nova master: [placement] Filter allocation candidates by forbidden traits in db https://review.openstack.org/556660
00:48:47 openstackgerrit Merged openstack/nova master: [placement] Support forbidden traits in API https://review.openstack.org/556820
01:25:35 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
01:25:36 openstackgerrit Jackie Truong proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
01:25:37 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
01:25:38 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
02:14:10 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
02:14:11 openstackgerrit Jackie Truong proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
02:14:12 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
02:14:13 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
02:23:02 openstackgerrit Merged openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078
02:28:15 Spaz-Work Morning Folks
02:46:54 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949

Earlier   Later