Earlier  
Posted Nick Remark
#openstack-nova - 2017-09-29
18:08:38 cburgess Yeah cool.
18:08:45 cburgess I will investigate this feature. Thanks team!
18:08:54 melwitt just sharing my recent revelation, just in case
18:09:01 superdan mriedem:
18:09:02 superdan % gsl HEAD^..
18:09:02 superdan a3d7d30 Merge build requests into the sortmaster 9000
18:09:32 superdan mriedem: you may ooh and ahh now
18:10:26 mriedem i don't know what you're talking about
18:10:38 mriedem oh
18:10:39 superdan bah
18:10:40 mriedem now i get it
18:11:07 mriedem jesus, is there anything the sortmaster 9000 can't do?
18:11:15 mriedem oh i know
18:11:31 mriedem RFE: please make the sortmaster 9000 allow me to pass a volume type during boot from volume
18:12:57 mriedem oh btw
18:13:04 mriedem if you're looking for queued stuff
18:13:04 mriedem http://zuulv3.openstack.org/
18:14:44 mriedem this pike regression fix needs another +2 https://review.openstack.org/#/c/507938/
18:14:47 mriedem the zuul failure is the nnet job
18:15:49 superdan which hasn't merged yet right?
18:15:51 superdan the nnet fix?
18:15:57 mriedem i'm watching it
18:16:00 mriedem but no
18:16:52 melwitt I have a tox.ini patch up for adding stderr/stdout capture and test timeout to the functional and api_sample_tests https://review.openstack.org/#/c/507976
18:17:12 melwitt if anyone wants to help get stdout capturing back
18:19:18 mriedem melwitt: commented,
18:19:23 mriedem there might be a way to avoid the duplication
18:20:44 melwitt thanks
18:36:54 mriedem ok i think these nnet zuul job changes are all passing
18:36:58 mriedem so i'll bug infra after that
18:44:22 cburgess mriedem melwitt Is there a minimum reliable version that supported the archive thing (as in not before liberty etc)? Looks like the command first went in for grizzly.
18:44:35 cburgess superdan too
18:44:37 mriedem looking
18:44:41 mriedem you want the --until-complete thing
18:44:46 cburgess OK.
18:44:51 cburgess I can figure out when that went in then.
18:44:51 mriedem i want to say ocata
18:45:05 cburgess Prior to that it didn't work at all or... it was just inefficient?
18:45:28 mriedem https://review.openstack.org/#/c/378718/
18:45:37 mriedem yeah ocata
18:45:44 superdan didn't work at all really
18:45:49 mriedem before that i think there were some random foreign key constraints that could mess it up for totally completing
18:45:57 mriedem plus you had to always specify the batch number
18:46:33 cburgess RAD...
18:46:50 cburgess So.. not as sad a panda as I was before, still work for me to do. Thanks.
18:47:20 mriedem also https://review.openstack.org/#/q/I77255c77780f0c2b99d59a9c20adecc85335bb18
18:47:29 mriedem so going back to mitaka i think you can delete everything in batches
18:47:31 mriedem per dan's fix
18:48:46 cburgess Cool.
18:49:05 cburgess I doubt I'm going to enjoy backporting this to Icehouse. *sigh*
18:49:30 melwitt icehouse? ouch
18:49:42 cburgess melwitt Don't judge.... ok well judge.
18:50:01 melwitt not judging, just imagining the pain
18:50:08 mriedem cburgess: i expect it's actually not bad
18:50:14 mriedem since that code is isolated
18:50:21 mriedem and not a lot of changes over time
18:50:27 melwitt that reminds me, I wanted to resurrect my old patch to turn on FK constraint enforcement to sqlite
18:50:44 cburgess melwitt My world is pain.. nothing but pain. I'm a purveyor of fine vintage clouds (actually I purvey new ones, but I have to support vintage ones).
18:50:52 melwitt hah
18:52:52 openstackgerrit melanie witt proposed openstack/nova master: Make setenv consistent for functional and api_samples https://review.openstack.org/507976
18:54:36 openstackgerrit melanie witt proposed openstack/nova master: Make setenv consistent for functional and api-samples https://review.openstack.org/507976
18:57:51 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Add a regression test for bug 1718455 https://review.openstack.org/508590
18:57:53 openstack bug 1718455 in OpenStack Compute (nova) "[pike] Nova host disable and Live Migrate all instances fail." [Medium,In progress] https://launchpad.net/bugs/1718455 - Assigned to Matt Riedemann (mriedem)
18:57:53 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Ensure instance can migrate when launched concurrently https://review.openstack.org/508591
19:09:18 mriedem superdan: dead code now right? https://github.com/openstack/nova/blob/ae4b5d0147cb3e345bf57034221e9c8fedf3cad2/nova/conductor/manager.py#L261
19:09:28 mriedem oh nvm
19:09:30 mriedem we're at 1.17
19:09:44 superdan right
19:09:50 superdan we never bumped
19:10:10 mriedem we should remember to do that this release, across the board
19:10:11 mriedem we're due
19:10:27 mriedem and by we i mean you, because i'd just mess it up
19:11:30 superdan heh
19:16:12 openstackgerrit Dan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000 https://review.openstack.org/508595
19:16:34 superdan mriedem: ^ probably needs a few more test cases, but you love pointing those out so I'll indulge you
19:17:11 superdan I'm trying to decide if I should parallelize fetching the buildrequests as well, to try to further lower the latency
19:17:31 superdan it'd be hard to prove that is worth it though, since it's hard to have many instances in that state for long without bugs
19:18:05 mriedem i'm currently doing a review of https://review.openstack.org/#/c/498948/ before it merges
19:18:28 superdan another favorite pasttime of yours
19:18:31 mriedem yeah the only way to do that really is via fixtures, to hold up the build requests
19:18:50 superdan well, I meant in a devstack type environment, but yeah
19:19:06 mriedem yeah i'm not sure how you'd reliably test that
19:19:18 superdan that's my point yeah
19:19:25 mriedem unless you booted so many at once, and scheduling was slow enough, that you could see a difference
19:19:33 superdan and probably not much gain for people
19:19:47 mriedem single node devstack is probably not going to work there
19:19:52 mriedem you'd have to fake out like 1000 computes or something
19:19:56 mriedem something that placement has to chew on
19:20:13 superdan nah, just shut down scheduler so we block on making that decision and then try to list before the timeout
19:20:17 superdan but still, not great
19:21:33 mriedem true, set the rpc timeout to 10 minutes :)
19:22:22 superdan the problem is, because we don't have real threads, we're already pegging the cpu just processing the results from the DBs as it is, so, not likely the even smaller overlap of operations is really going to help
19:22:42 melwitt how can we get real threads
19:23:06 superdan I'm not sure eventlet is threadsafe, so I'm not sure we can without bigger changes
19:23:39 superdan that was the problem with us using real threads for db workers ages ago, and it was never fixed, AFAIK, we just worked around it by moving to an all-python db driver
19:24:34 superdan in aggregate it's not as big of a deal because multiple requests will be running at once in our workers, so more overall work gets done with more cores, just not in a single-request sort of environment
19:25:00 superdan any environment that only has one concurrent request ever is either (a) dan's test box or (b) probably not worried about listing thousands of instances at a time :)
19:25:15 melwitt heh
19:29:06 mriedem superdan: some comments/questions in https://review.openstack.org/#/c/498948/
19:29:27 openstackgerrit priyaduggirala proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py https://review.openstack.org/508533

Earlier   Later