| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-29 | |||
| 18:06:31 | mriedem | deleted things | |
| 18:06:32 | melwitt | it deletes it from the main tables | |
| 18:06:33 | superdan | without constraints | |
| 18:06:35 | cburgess | Oh.. | |
| 18:06:45 | cburgess | That would do what I needed. | |
| 18:06:52 | mriedem | cburgess: nova-manage db archive_deleted_rows --verbose --until-complete | |
| 18:07:03 | cburgess | Cool. I'll look into it. Thanks. | |
| 18:07:18 | cburgess | Oh this.. https://blueprints.launchpad.net/nova/+spec/archive-deleted-rows-all | |
| 18:07:22 | superdan | mriedem: wanna peek behind my curtain? | |
| 18:07:23 | mriedem | it's not multicell aware, but you're on folsom so that doesn't matter for you | |
| 18:07:24 | cburgess | Looks like melwitt Worked on it. | |
| 18:07:31 | cburgess | I'm not on folsom.. god.. | |
| 18:07:39 | mriedem | superdan: which curtain? | |
| 18:07:39 | cburgess | We never used Folsom. | |
| 18:07:54 | superdan | mriedem: this one over here.. the one that hides my current git tree | |
| 18:08:05 | melwitt | yeah, there might be some issues if you keep the archived data indefinitely. recently I learned primary keys can be reused after a DB restart with innodb, so archive can fail if there are dupes | |
| 18:08:22 | superdan | melwitt: cburgess wants to dump the shadow tables immediately though, so no problem | |
| 18:08:35 | melwitt | yep +1 | |
| 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 | a3d7d30 Merge build requests into the sortmaster 9000 | |
| 18:09:02 | superdan | % gsl HEAD^.. | |
| 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 | 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 | |