Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-24
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?
15:35:50 mriedem or the current version of https://review.opendev.org/#/c/660761/3 ?
15:36:15 mriedem if terminate is just racing with build and the copy of the instance during terminate just doesn't have the info cache data from the db, then the refresh should fix your problem i think,
15:36:35 mriedem but if the info cache in the db is empty when you refresh, that bottom change isn't going to help
15:36:47 amorin the cache is populated in the DB
15:36:47 mriedem it won't make things worse of course
15:36:55 mriedem ok, well in that case you might be saved,
15:37:00 amorin but after the copy of instance is used for deletion
15:37:14 amorin yup, anyway I will double check that point
15:37:15 melwitt dansmith: do you remember where we landed when we were talking about disabling oslo.messaging heartbeats in nova-api, as far as whether we would recommend running that configuration in our upstream docs?
15:37:20 mriedem because https://github.com/openstack/nova/blob/976d1b89c2b754729903291050e01c9bf49704b9/nova/network/neutronv2/api.py#L1757
15:37:33 sean-k-mooney efried: sorry was on a call for the last 90 mins
15:37:43 amorin _gather_port_ids_and_networks is using the nova cache DB right?
15:37:53 sean-k-mooney efried: did ye reach an agreement on the nova-console auth blueprint
15:37:56 openstackgerrit Adam Spiers proposed openstack/nova master: Move selective patching of open() to nova.test for reuse https://review.opendev.org/661266
15:38:12 melwitt dansmith: this is about the idle nova-api wsgi app heartbeat error messages stuff
15:38:26 mriedem amorin: yeah
15:38:47 amorin mriedem, by the way, thanks for the tip about mocking in setup, I was pretty sure that this was possible
15:38:51 mriedem amorin: yeah so tl;dr if you're just racing to refresh from the db on delete then your fix is probably ok
15:38:55 amorin but I am beginning in that subject
15:38:57 mriedem np
15:39:03 mriedem i've only recently-ish started using that trick
15:39:09 efried sean-k-mooney: It sounds like all the deployment projects are either sorted or have patches in flight to sort. I was looking for you & others to vet and +1 basically so I could approve the bp
15:39:54 melwitt efried, finucannot: I'm also +1 on removing nova-consoleauth since it looks like everything is sorted with deployment tools
15:40:02 sean-k-mooney efried: kolla has an open bug but no progess in a while
15:40:22 sean-k-mooney if that is the only blocker i can go write the patches needed
15:40:47 sean-k-mooney it is already optional but it would jsut invovle deleting the container on upgrade an removing the deployment code
15:41:28 efried finucannot: do you have any further words about kolla ^ ?
15:41:57 sean-k-mooney so yes im +1 on removing it and can help the kolla folks as needed since i used to work on kolla and kolla ansibel a few release ago
15:42:06 finucannot sean-k-mooney: So there's more to it than this? https://review.opendev.org/#/c/661251/
15:42:09 finucannot efried: ^
15:42:33 sean-k-mooney that removed it from kolla
15:42:44 sean-k-mooney but they also need to remove it in kolla ansible
15:42:53 sean-k-mooney e.g. you just deleted teh container
15:43:16 sean-k-mooney but the upgrade playbook need to koll all running instance of the contain
15:43:21 sean-k-mooney *kill
15:43:54 finucannot Ah, I figured there was more to it than that. I can try drafting those patches based on what they did in OSA. I'd need one to kill the running instances (which would be backported to stable/stein) then a follow up to remove all references entirely
15:44:33 sean-k-mooney we need to kill this handeler https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/nova/handlers/main.yml#L112-L136
15:44:48 efried finucannot: Okay, mriedem, melwitt, and sean-k-mooney are all +1, I'm approving the bp assuming the above is fairly easy and will be taken care of before we pull the trigger.
15:44:54 sean-k-mooney and remove like 2 or 3 other things but its a fairly small patch
15:46:08 finucannot sean-k-mooney: I'll do that before I clock off
15:47:55 sean-k-mooney ok you need to kill the service default https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/nova/defaults/main.yml#L83-L93 and as i sad just update https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/nova/tasks/rolling_upgrade.yml and https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/nova/tasks/legacy_upgrade.yml to nuke the running container on
15:47:57 sean-k-mooney upgrade
15:49:02 sean-k-mooney let me know if you need any help navagaing kolla-ansible
15:50:55 mriedem gibi: replied, i think your suggestion is ok
15:52:42 finucannot mriedem, efried: https://review.opendev.org/661344
15:53:25 dansmith melwitt: I think disabling heartbeats will increase recovery time if the connection goes stale, and will cause stale connections to stay open on the rabbit side
15:53:34 dansmith melwitt: so I think threads=1, leave heartbeat behavior
15:54:23 melwitt sean-k-mooney, mdbooth ^
15:54:52 aspiers efried: I'm pondering whether my helpers fall foul of https://docs.python.org/3/library/unittest.mock.html#where-to-patch
15:54:52 melwitt dansmith: thanks
15:56:06 aspiers efried: although that wouldn't explain why they have been working so far
15:56:33 gibi mriedem: thanks.
15:57:07 sean-k-mooney dansmith: if we leave teh hearbeat enable should we at least change the log level in the nova-api for the oslo messaging logs?
15:57:19 sean-k-mooney or the heart beat on in partiaclar
15:59:16 efried finucannot: why does starlingx get different zebra stripe?
15:59:18 dansmith I dunno, but we might want to soften the log level in o.msg where it raises holy hell about the reconnct
15:59:22 efried *I* want special zebra stripe
16:00:09 efried aspiers: I'm going to have to get back to you (possibly much) later
16:00:35 aspiers efried: it's OK. Attacking it with a debugger now
16:00:51 sean-k-mooney dansmith: so we suggested that and got some push back to droping it to info in oslo
16:01:38 finucannot efried: Whoops, missed that
16:01:41 finucannot fixed
16:01:49 dansmith sean-k-mooney: so what is your thought, to specifically set o.msg level to error in nova-api?
16:02:24 sean-k-mooney dansmith: effectivly but i think we can be more granular then that. we do this for some privesep stuff right?
16:02:43 dansmith sean-k-mooney: we can set defaults on the logger name level, I think
16:02:59 sean-k-mooney dansmith: actully setting it to error wont help as o.msg is currently loggin it as an error
16:03:16 sean-k-mooney so more we woudl have to install a log filter to speficaly drop that error

Earlier   Later