| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-16 | |||
| 20:26:17 | efried | yeah... | |
| 20:26:27 | mriedem | see how many times someone named Eric Fried blindly rechecked it :) | |
| 20:26:44 | mriedem | i'll put up a patch | |
| 20:26:52 | efried | If only I had full debuggability from my phone. | |
| 20:27:39 | mriedem | or make shilpa responsible for investigating the failures and rechecking them | |
| 20:27:45 | mriedem | but whatever, done is done | |
| 20:30:28 | efried | mriedem: looks to me like everything (except the super()) through the self.computes = {} can be removed, yah? | |
| 20:30:51 | mriedem | yup, just did it, running the tests | |
| 20:31:11 | efried | k, just prepping for review :P | |
| 20:31:25 | efried | +2 "that's how I would have done it" | |
| 20:37:16 | efried | tests pass locally for me | |
| 20:37:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove redundancies from AggregateRequestFiltersTest.setUp https://review.opendev.org/682475 | |
| 20:38:11 | efried | mriedem: two more lines | |
| 20:38:52 | mriedem | ack | |
| 20:39:42 | efried | mriedem: why only partial-bug? | |
| 20:40:02 | mriedem | b/c i don't know if it's the root fix | |
| 20:40:12 | mriedem | hence the "at least rule it out" comment | |
| 20:40:28 | mriedem | if we see the hits drop off in e-r by next week we can close the bug | |
| 20:40:45 | efried | this way requires us to remember to go back at the bug :) | |
| 20:40:48 | efried | but okay | |
| 20:40:58 | mriedem | i will when i cleanup old e-r queries | |
| 20:41:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove redundancies from AggregateRequestFiltersTest.setUp https://review.opendev.org/682475 | |
| 20:42:57 | efried | thanks for the quick fix mriedem <high five> | |
| 20:43:08 | mriedem | <down low> | |
| 20:45:02 | mriedem | i wonder if the fake messaging driver doesn't do rpc fanout cast properly, if we should have some sort of fixture that blows up if you try to start more than one non-nova-compute service in a functional test | |
| 20:45:25 | mriedem | since that's twice in 2 weeks that we've had some issue from functional test inheritance doubling up on fixtures and stuff | |
| 20:46:02 | sean-k-mooney | are you suggestion on;ly allowing 1 comptue service in the fucntional tests | |
| 20:46:21 | efried | no | |
| 20:46:23 | sean-k-mooney | or havign a fixture to detect when you have more then one but have not set things up correctly | |
| 20:46:35 | efried | he's suggesting allowing only one *non* nova-compute service. | |
| 20:46:52 | efried | we clearly need multiple compute services for many tests. | |
| 20:47:01 | sean-k-mooney | oh so only one conductor or scheduler | |
| 20:47:01 | efried | but we should never need more than one superconductor | |
| 20:47:04 | efried | right | |
| 20:47:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove SchedulerReportClient from AggregateRequestFiltersTest https://review.opendev.org/682480 | |
| 20:47:38 | mriedem | correct, one controller service | |
| 20:47:43 | mriedem | one of each | |
| 20:50:53 | mriedem | i think multiple apis and conductors are probably ok since those are stateless | |
| 20:51:02 | mriedem | but the scheduler has some stateful crap in the HostManager | |
| 20:51:06 | mriedem | like aggregate info | |
| 20:52:12 | mriedem | b'2019-09-16 15:10:19,491 INFO [nova.service] Starting scheduler node (version 19.1.0)' | |
| 20:52:12 | mriedem | but yeah the test was definitely starting 2 schedulers | |
| 20:52:17 | mriedem | b'2019-09-16 15:10:20,156 INFO [nova.service] Starting scheduler node (version 19.1.0)' | |
| 21:01:13 | efried | mriedem: what was the other bug with the duplicated services? | |
| 21:01:26 | mriedem | duplicated placement fixture | |
| 21:01:27 | mriedem | sec | |
| 21:01:41 | mriedem | https://github.com/openstack/nova/commit/5e1b096894f6de4cfbca254cf74dcfcf56358ea5#diff-5befe429f14247314e6ca487aa4e13bd | |
| 21:02:07 | efried | thank you sir | |
| 21:02:13 | efried | I'm posting a poison patch now | |
| 21:02:23 | mriedem | heh i was just writing my commit message ofr one | |
| 21:02:25 | mriedem | *for | |
| 21:02:30 | efried | oh, no bug for I057a07c8d0b880c8d09fc2e618ce1f7fc885beda ? | |
| 21:02:39 | mriedem | guess not | |
| 21:03:20 | efried | hm, if that one was placement fixture, my fix isn't going to hit it. | |
| 21:03:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Do not allow mutiple nova-scheduler workers https://review.opendev.org/682485 | |
| 21:04:06 | mriedem | efried: ^ is what i was thinking of | |
| 21:04:28 | efried | whoah | |
| 21:04:46 | efried | Mine is way dumber, but more complete, and faster. | |
| 21:05:43 | openstackgerrit | Eric Fried proposed openstack/nova master: Only allow one non-compute service in tests https://review.opendev.org/682486 | |
| 21:05:46 | efried | mriedem: ^ | |
| 21:09:21 | mriedem | you're going to at least have failures on tests that intentionally start multiple api fixtures with different projects and roles (admin vs non-admin, project1 and projec2 for filtering, etc) | |
| 21:10:35 | sean-k-mooney | if we have multi cell tests then those would also fail correct | |
| 21:11:13 | efried | yeah, I guess I thought the cell conductors would have different names, but that doesn't make sense. | |
| 21:11:14 | sean-k-mooney | so we need a way to allow it | |
| 21:11:22 | mriedem | the api/conductor/scheduler services don't get registered in the cell dbs | |
| 21:11:24 | mriedem | so i don't think that matters here | |
| 21:12:03 | mriedem | but i'm also not sure how sophisticated our fixture stuff is to know where to create those services (they should go into cell0 but might not) | |
| 21:12:27 | mriedem | the CellDatabase fixture has a default of cell1 so the stuff probably just gets created there | |
| 21:12:40 | mriedem | sean-k-mooney: also btw i added https://review.opendev.org/#/c/670585/2 to https://review.opendev.org/#/c/670585/2 https://etherpad.openstack.org/p/nova-train-release-todo | |
| 21:12:43 | mriedem | so we don't forget | |
| 21:13:36 | sean-k-mooney | ok cool thanks. it would not be the end of the world if it sliped but im hopeing that we can drop supprot for this downstream before our next LTS | |
| 21:14:02 | sean-k-mooney | so in 24-18 months when people start using it we dont have to support it | |
| 21:15:20 | artom | sean-k-mooney, are you really though? :P | |
| 21:15:42 | sean-k-mooney | only by the expect it to work and get new features bit | |
| 21:16:01 | sean-k-mooney | they can choose one but not both if they want something that old | |
| 21:17:05 | mriedem | they expect you to make it work and provide new features when they shovel out the $$$ to your sales guys | |
| 21:17:07 | mriedem | and gals | |
| 21:17:22 | mriedem | and then the shit rolls down hill to you, the developer | |
| 21:18:21 | sean-k-mooney | ya well we try to do both but when you cant backport api,db or object changes addinf features that arent bug fixs to old release is | |
| 21:18:25 | sean-k-mooney | challanging | |
| 21:19:56 | efried | mriedem: I don't see anywhere we start cell conductor services. What would that look like? | |
| 21:20:08 | efried | Or do we do it implicitly when we start a compute service in a specific cell? | |
| 21:20:55 | sean-k-mooney | if we are starting only one conductor the we are running it in the non super conductor toplogy | |
| 21:21:16 | openstackgerrit | Eric Fried proposed openstack/nova master: Only allow one scheduler service in tests https://review.opendev.org/682486 | |
| 21:21:31 | sean-k-mooney | i dont know if we create multiple conductors in the current fucntional test but the cross cell migration code might have some? | |
| 21:22:40 | efried | we don't have any cross-cell migration code | |
| 21:22:47 | efried | tssssssss | |
| 21:23:31 | sean-k-mooney | i ment there might be case in mriedem series | |
| 21:24:42 | sean-k-mooney | we might be relying on https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L118 | |
| 21:24:55 | mriedem | efried: we don't do the whole super conductor / cell conductor thing in functional tests | |
| 21:24:57 | mriedem | it's all just one | |
| 21:24:58 | sean-k-mooney | to start the conductor in the functional test | |
| 21:25:06 | mriedem | efried: meaning we get away with shit in functional tests that we wouldn't in devstack | |
| 21:25:47 | mriedem | e.g. we do stuff like have the CheatingSerializer so it all looks like one RPC | |
| 21:25:54 | sean-k-mooney | i think devstack has the abiltiy to deploy in the legacy mode too but i dont know if thats tested still so it could be broken | |
| 21:25:58 | mriedem | which has been a problem in my cross-cell series with functional tests | |
| 21:26:10 | mriedem | sean-k-mooney: all grenade jobs are non-superconductor | |
| 21:26:39 | sean-k-mooney | ah ok. is that still true for the new zuulv3 version? | |
| 21:27:07 | mriedem | i would expect so but haven't looked | |
| 21:27:22 | sean-k-mooney | i tried using that to do a greade version of my numa jobs but i found i could not set local.conf diffrently for new/old nodes | |