| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-29 | |||
| 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 | http://zuulv3.openstack.org/ | |
| 18:13:04 | mriedem | if you're looking for queued stuff | |
| 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 | mriedem | i want to say ocata | |
| 18:44:51 | cburgess | I can figure out when that went in then. | |
| 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 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Ensure instance can migrate when launched concurrently https://review.openstack.org/508591 | |
| 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) | |
| 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 | |
| 19:29:33 | mriedem | not trying to block, just want to make sure i know what's going on with this stuff before it merges and i'm lost later | |
| 19:31:57 | superdan | mriedem: I thought we didn't do _LI( but we don't do _( at all anymore? | |
| 19:32:13 | mriedem | we don't translate log messages at all anymore | |
| 19:32:21 | superdan | really thought I was getting pep8 fails | |
| 19:32:31 | superdan | christ, I can never keep it straight | |
| 19:33:53 | mriedem | https://docs.openstack.org/oslo.i18n/latest/user/guidelines.html | |
| 19:34:02 | mriedem | i think that top paragraph is the new guideline | |
| 19:34:24 | mriedem | and https://docs.openstack.org/oslo.i18n/latest/user/guidelines.html#log-translation | |
| 19:37:17 | superdan | I believe you, I just can't keep track of it | |
| 19:37:51 | mriedem | i had to look it up too, wasn't sure about _() when you asked but was pretty sure | |
| 19:38:40 | superdan | I figure if I just pick some behavior I'll be right 20% of the time when we've circled back to that as the preferred one | |
| 19:39:10 | mriedem | depends on the current ibm corporate wide software guidelines at the time | |
| 19:39:43 | superdan | mriedem: so, this isn't in the gate yet so do you want me to fix the bottom patch or tack on to the end? this set is pretty fragile so if I do the bottom it'll likely percolate awesomeness up the stack pretty good | |