| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-24 | |||
| 13:55:57 | efried | unless argparse is doing something magical... | |
| 13:56:03 | dansmith | efried: what? | |
| 13:56:16 | efried | see comment on https://review.opendev.org/661002 | |
| 13:56:52 | aspiers | Yeah, that comment looks right | |
| 13:56:57 | dansmith | oh yeah | |
| 13:57:22 | openstackgerrit | Dan Smith proposed openstack/nova master: Make nova-next archive using --before https://review.opendev.org/661002 | |
| 13:57:26 | aspiers | efried: BTW I can't believe you are still using a W520 | |
| 13:57:41 | aspiers | I mean, I am too, but only as a VPN server hidden under my desk | |
| 13:58:10 | efried | aspiers: Heh, it was my IBM-provided workstation, did a nice SSD and memory upgrade, it was really stable and solid, so I bought it back when I left. | |
| 13:58:15 | aspiers | Mine weighed a ton and the battery used to last 2 hours on a good day | |
| 13:58:39 | efried | My Intel-provided HP thingy has roughly equivalent specs, weighs about a quarter as much... and performs like a dog. | |
| 13:58:57 | efried | ...and won't run not-Windows. | |
| 13:59:06 | efried | (because of Intel gorp) | |
| 13:59:19 | efried | (without major hoops of fire) | |
| 14:05:33 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Fixes multi-registry config in Quobyte driver https://review.opendev.org/660706 | |
| 14:10:14 | mriedem | efried: we might want to see https://review.opendev.org/#/c/661002/ working in action before +2ing :) | |
| 14:11:19 | efried | mriedem: It worked at a previous PS | |
| 14:11:22 | efried | as in, didn't blow up | |
| 14:11:28 | efried | syntax weirdness notwithstanding | |
| 14:11:47 | efried | i.e. archived rows older than [earlier date than intended] | |
| 14:12:15 | efried | presumably all the rows in that job are newer than "yesterday" anyway, so the change is a no-op? | |
| 14:12:17 | mriedem | http://logs.openstack.org/02/661002/1/check/nova-next/d88596c/logs/devstack-gate-post_test_hook.txt.gz#_2019-05-23_23_53_30_031 | |
| 14:12:21 | mriedem | nova-manage: error: unrecognized arguments: --max-rows | |
| 14:12:48 | efried | oh, that doesn't cause the test to fail? Guess that should be fixed :) | |
| 14:12:53 | mriedem | we should probably have a set -e around this b/c it's blowing up but not failing | |
| 14:13:08 | efried | okay | |
| 14:14:49 | mriedem | set -e in a separate change so we can backport it imo | |
| 14:15:29 | finucannot | efried: I did, yup, cheers. The kolla patch has a +2 already too. Anything else needed to get that approved now? :) | |
| 14:21:16 | openstackgerrit | Dan Smith proposed openstack/nova master: Make nova-next archive using --before https://review.opendev.org/661002 | |
| 14:30:51 | melwitt | dansmith: do you recall whether we exclude certain exceptions from incrementing the build_failed counter for the auto-disabling of computes? I thought it was discussed in the past but looking at the code I don't see anything like that https://github.com/openstack/nova/blob/stable/queens/nova/compute/manager.py#L1789 | |
| 14:31:24 | dansmith | yes, there are two classes | |
| 14:32:30 | dansmith | so things we catch and handle in a certain way don't count, | |
| 14:32:41 | dansmith | and anything we don't catch counts | |
| 14:32:48 | dansmith | and then some gray in the middle | |
| 14:33:12 | dansmith | and part of the problem is we don't do nearly as good at distinguishing as we thought we did | |
| 14:34:07 | dansmith | hmm, although looking again, maybe we lumped them all together in the end after that realization | |
| 14:34:56 | melwitt | yeah, I don't remember this, I thought there was some kind of whitelist that wouldn't increment the counter but not seeing it now | |
| 14:35:32 | dansmith | I think it was the failed/rescheduled result | |
| 14:35:40 | dansmith | but we increment for both now | |
| 14:35:55 | melwitt | I see | |
| 14:36:47 | openstackgerrit | Eric Fried proposed openstack/nova master: Clarify --before help text in nova manage https://review.opendev.org/661289 | |
| 14:36:50 | mriedem | there is no whitelist | |
| 14:36:52 | mriedem | we talked about it | |
| 14:36:55 | mriedem | but it got out of hand | |
| 14:37:08 | mriedem | i had a wip patch for a bit for one type of exception but it was gross and dropped it | |
| 14:37:32 | mriedem | there are about a billion ways you can get an exception in that build path in the compute | |
| 14:37:41 | dansmith | yeah, that's what I mean by we thought it would be clearer which things are terminal failures or not, but it's not | |
| 14:37:48 | mriedem | and since it's python and not strictly typing any of that stuff, we can't really just catch one | |
| 14:38:10 | dansmith | well, I dunno about _that_ :) | |
| 14:39:02 | mriedem | well, meaning like in java where you have to declare and explicitly handle exceptions that get raised up during compiling | |
| 14:39:14 | dansmith | ah sure | |
| 14:39:27 | dansmith | i.e. "the most annoying part of java" | |
| 14:39:35 | mriedem | zvm driver could add a new MainframeKaputError to raise and we'd not handle it | |
| 14:39:43 | mriedem | if you're ocd like me it's not so bad :) | |
| 14:40:30 | mriedem | i'd like some credit for my witty use of a german word in that zvm joke please... | |
| 14:40:46 | melwitt | ok, this is sounding familiar now, I vaguely remember your wip patch about it | |
| 14:40:58 | melwitt | oh yeah, kaputt, nicely done | |
| 14:41:11 | mriedem | melwitt: so is your customer on just a release that is too old to have the weigher? | |
| 14:41:48 | efried | finucannot: If two out of three of melwitt, mriedem, and sean-k-mooney ack it, I'll be happy to approve it (nova-consoleauth removal bp) | |
| 14:41:48 | melwitt | no, they have the weigher, it's just incrementing the counter for user-caused build failures we think | |
| 14:42:02 | dansmith | that's the whole thing | |
| 14:42:07 | dansmith | that's why we moved it to a weigher | |
| 14:42:21 | dansmith | because we suck so bad at knowing why a thing failed that auto-disable isn't really sane | |
| 14:42:23 | mriedem | melwitt: malicious user? | |
| 14:42:41 | dansmith | so we made it a weigher so it doesn't take it out of rotation and you can adjust how much score you give to that decision | |
| 14:42:51 | melwitt | mriedem: I doubt it but not sure | |
| 14:43:22 | mriedem | so jed at the bank wrote a script to boot 100 vms at once and it's DoSing their private cloud | |
| 14:43:58 | melwitt | dansmith: got it. yeah, IIUC this customer is trying to pack instances and they see it start spreading instances when they don't want it to, because of the user-caused failures | |
| 14:44:21 | dansmith | melwitt: right so null out that weigher and move on | |
| 14:45:51 | melwitt | mdbooth: turns out I was mistaken. we had tried some exception whitelist patches but it got unwieldy, so that idea was abandoned. the recommended way to handle this scenario is to disable the failed build weigher | |
| 14:46:30 | mdbooth | melwitt: Ack, thanks. | |
| 14:48:27 | mriedem | once again let me state i dislike the non-border table rendering in our docs now https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#id17 | |
| 14:51:53 | gibi | mriedem: do you have opinion about the CLI syntaxt propsal in https://review.opendev.org/#/c/651783/3/osc_placement/resources/resource_provider.py@130 ? | |
| 14:54:13 | mriedem | sec | |
| 14:54:18 | gibi | thanks | |
| 15:02:42 | finucannot | mriedem: I'll trade you a fix for that table issue for your ack on my remove-consoleauth bp | |
| 15:03:04 | finucannot | been meaning to get to that for weeks now | |
| 15:17:09 | aspiers | efried: asking on #openstack-requirements about the contextlib2 issue, I have not much clue what's going on | |
| 15:17:20 | aspiers | bizarrely it's moaning about sphinx too | |
| 15:17:36 | aspiers | where's stephenfin when you need him? | |
| 15:18:06 | finucannot | aspiers: I'm here (it's Friday) | |
| 15:18:12 | aspiers | hah | |
| 15:18:59 | openstackgerrit | Arnaud Morin proposed openstack/nova master: Force refresh instance network info on deletion https://review.opendev.org/660761 | |
| 15:19:28 | finucannot | What's the specific Sphinx issue? | |
| 15:19:31 | finucannot | aspiers: ^ | |
| 15:19:43 | aspiers | finucannot: see #openstack-requirements | |
| 15:20:45 | finucannot | Man, I wish we'd drop Python 2 this cycle instead of in U :( | |
| 15:22:28 | amorin | hi efried, mriedem and others. I updated the patch to refresh instance network cache on deletion, thanks for commenting/reviewing! | |
| 15:23:48 | ganso | dansmith: Hi! When you have a few minutes, could you please take a look at and sign off https://review.opendev.org/#/c/659338/ if everything is ok? thanks in advance =) | |
| 15:24:34 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 15:24:58 | dansmith | ganso: I was going to challenge the directness of your query, but I see tonyb threw me under the bus, so.. okay :) | |
| 15:25:15 | mriedem | efried: finucannot: i've put my ack in https://blueprints.launchpad.net/nova/+spec/remove-consoleauth | |
| 15:30:48 | mriedem | amorin: questions in there, | |
| 15:31:05 | mriedem | but without the force will this even fix the case in your bug? have you tested it without the force_refresh=True? | |
| 15:31:27 | mriedem | b/c the point of the force flag was to rebuild the cache from neutron rather than the cache itself | |
| 15:33:56 | amorin | mriedem: good point | |
| 15:34:03 | amorin | hum | |
| 15:34:33 | amorin | I need to test that, it worked in my dev infra, but I need to double check that | |
| 15:35:11 | amorin | I wont have time today, but I will do next week | |
| 15:35:43 | mriedem | "it" being the original patch with force_refresh=True? | |