| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-07 | |||
| 13:38:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452 | |
| 13:38:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501514 | |
| 13:38:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | |
| 13:39:31 | mriedem | alex raised a good question here https://review.openstack.org/#/c/501408/8/nova/conductor/manager.py@1049 | |
| 13:40:06 | mriedem | in that if the build request is already deleted after we create the instance in the cell but before we actually send the instance to the host to build, we cleanup the instance (local delete) but don't cleanup dest node allocations | |
| 13:40:34 | mriedem | separate bug, but a bug nonetheless | |
| 13:41:01 | mikal | sdague: so I need a reno when I remove a command from rootwrap's filters? Or only when I add one? | |
| 13:41:04 | openstackgerrit | Merged openstack/nova master: Fix test_rpc_consumer_isolation for oslo.messaging 5.31.0 https://review.openstack.org/501400 | |
| 13:42:20 | sdague | the adds are the critical bit. That's a config file which won't work if you don't add those lines | |
| 13:42:29 | sdague | if people left old things around, meh | |
| 13:43:07 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402 | |
| 13:43:38 | mriedem | who wants to look at https://review.openstack.org/#/c/501514/ and up? regression in pike that needs to be backported | |
| 13:43:42 | mriedem | vexxhost hit it yesterday | |
| 13:44:08 | jaypipes | mriedem: just FYI, we're evacuating this evening so I'll be OOO tomorrow and part of this afternoon. | |
| 13:44:23 | mriedem | jaypipes: ok, good luck | |
| 13:44:25 | jaypipes | mriedem: and I do mean evacuate, not rebuild. | |
| 13:44:29 | sdague | mikal: the general rule of thumb, assume someone is upgrading with pip install -U on the commit. Do they need to do something besides that to have usable environment | |
| 13:44:39 | mriedem | jaypipes: i hear ohio still has room for you | |
| 13:44:44 | mriedem | if you'll take it back | |
| 13:44:47 | jaypipes | mriedem: that's where we're headed. | |
| 13:45:04 | mriedem | nice | |
| 13:45:15 | mriedem | just in time for your beloved browns season to begin | |
| 13:45:22 | sdague | jaypipes: well, that means jeni's icecream when you arrive | |
| 13:45:24 | dansmith | jaypipes: good luck | |
| 13:45:30 | jaypipes | sdague: heh :) | |
| 13:45:43 | jaypipes | dansmith: thx. still smoldering where you are? | |
| 13:45:56 | dansmith | jaypipes: yeah | |
| 13:45:58 | sdague | jaypipes: we stopped in columbus north market on our drive to TN for the eclipse. Could not leave without some jeni's | |
| 13:46:14 | sdague | jaypipes: safe travels | |
| 13:46:21 | jaypipes | sdague: indeed. Jeni's is awesome. Graeters is also very good :) | |
| 13:46:26 | jaypipes | sdague: thx | |
| 13:46:36 | jaypipes | dansmith: how close to you are the fires? | |
| 13:46:54 | sdague | jaypipes: we going to see you in CO still? | |
| 13:47:08 | dansmith | jaypipes: the fires are at portland's watershed, if that tells you, but that's like 15 miles by the crow I think | |
| 13:49:03 | kashyap | dansmith: "by the crow" == shortest path I presume? | |
| 13:49:25 | dansmith | kashyap: yes, as in "as the crow flies" | |
| 13:49:30 | dansmith | apparently crows fly in straight lines | |
| 13:49:32 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/501745 | |
| 13:49:36 | kashyap | Yeah, I recall hearing that, just wanted to double-check :-) | |
| 13:49:47 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/501746 | |
| 13:49:48 | kashyap | Oh dear, the wildfire looks insane | |
| 13:50:44 | gibi | mriedem: hi! Is it OK for you If I rebase your patch https://review.openstack.org/#/c/498596 as mine top of it need to be adapted to a test case from master ? | |
| 13:56:33 | mriedem | gibi: sure | |
| 13:57:13 | mriedem | gibi: you were +1 on https://review.openstack.org/#/c/499718/ are you OK with it? sylvain is +2 and i'd like to move that series along as it's a regression that needs to be backported | |
| 13:57:37 | gibi | mriedem: looking... | |
| 13:57:53 | mriedem | looks like we need to talk about the other comments higher up the stack at some point too | |
| 13:58:00 | mriedem | after the meeting | |
| 13:58:16 | gibi | mriedem: yes, let's agree on the way forward with that series | |
| 14:00:08 | jaypipes | sdague: on line with Delta right now trying to redo flights | |
| 14:00:39 | bauzas | jaypipes: crossing fingers, mate | |
| 14:02:26 | mikal | sdague: yeah in that case I think a few of the renos need tweaking | |
| 14:02:38 | mikal | sdague: I will do that during the TWO HOURS OF KEYNOTES THIS ORNING | |
| 14:10:13 | sdague | mikal: but, you could miss some critical detail about synergizing! | |
| 14:19:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | |
| 14:19:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | |
| 14:19:55 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor ServerMovingTests for non-move tests https://review.openstack.org/498596 | |
| 14:26:07 | mriedem | gibi: i think you can remove your -1 on https://review.openstack.org/#/c/499237/ | |
| 14:26:45 | mriedem | gibi: i also replied to https://review.openstack.org/#/c/499874/4/nova/tests/functional/test_servers.py@1817 | |
| 14:27:12 | gibi | mriedem: reading... | |
| 14:28:31 | gibi | mriedem: what can we do with your dislike about rebuild after failed evac? | |
| 14:29:34 | mriedem | not sure i understand the question | |
| 14:30:01 | mriedem | gibi: i tend to agree with your point that we should handle a failure in the conductor (NoValidHost) the same as it's being handled in the compute when a claim fails or the spawn fails on the dest node, | |
| 14:30:03 | gibi | mriedem: I agree that rebuild after failed evac is not a major use case but I know about users depending on it. So I don't really want to prevent it | |
| 14:30:11 | mriedem | which is set the migration status to a state such that _destroy_evacuated_instances on the source node doesn't process it, | |
| 14:30:15 | mriedem | and delete the dest node allocation | |
| 14:31:02 | gibi | mriedem: OK. then I think we can set the migration to failed, and then nova will not try to delete source allocation either after the source compute is brought up | |
| 14:31:04 | mriedem | gibi: given that rebuild does not allocate resources or make a claim, i think in that case it's actually preferrable to *not* attempt to delete the source node allocations if the evacuate failed, | |
| 14:31:16 | gibi | mriedem: totally agree | |
| 14:31:17 | mriedem | because if the evac fails and you want to rebuild the instance on the source node after it comes back up, then the allocations are still there | |
| 14:31:37 | gibi | I think that we are on the same page now :) | |
| 14:31:49 | mriedem | and if the evac fails on the dest node, regardless of location (conductor or compute), we put the migration status into a state such that the source node won't process it | |
| 14:31:53 | mriedem | when it's restarted | |
| 14:32:06 | gibi | mriedem: agreee | |
| 14:32:17 | mriedem | yeah, so drop your -1 on https://review.openstack.org/#/c/499237/ and i'll +1 that | |
| 14:32:26 | gibi | mriedem: OK | |
| 14:33:17 | gibi | mriedem: done | |
| 14:33:35 | gibi | mriedem: this also means I have nothing agains the force evacuate test cases any more | |
| 14:33:48 | mriedem | yay | |
| 14:33:52 | gibi | mriedem: so I'm going to drop -1 there too | |
| 14:33:55 | jaypipes | ffs, delta wants $3700 to change my wife's flight. | |
| 14:34:32 | mnaser | that's outrageous. | |
| 14:34:47 | jaypipes | mnaser: welcome to price gouging. :) | |
| 14:35:03 | mnaser | don't know if it's any help but apparently jetblue's doing $99 flights -- dont know how much of those are still available (and im sure you heard of them) | |
| 14:35:22 | mriedem | when i was looking at flights to china in may, the price changed in one hour and went up like $7K | |
| 14:35:26 | jaypipes | mnaser: they want to charge me $800 for a single 50-minute flight from Columbus, OH to Atlanta, GA. | |
| 14:36:01 | jaypipes | looks like there's a possibility I might miss Denver... :( | |
| 14:36:49 | jaypipes | latest European ensemble is terrible... https://twitter.com/cajunweatherman/status/905792235656138753/photo/1 | |
| 14:38:09 | alex_xu | do we have wiki or etherpad for all the sessions and schedule in the PTG? I remember we have something in previous PTG | |
| 14:38:47 | jaypipes | alex_xu: I think ttx was putting together an ethercalc. | |
| 14:40:00 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | |
| 14:40:24 | alex_xu | jaypipes: ah, let me try to find it | |
| 14:40:34 | mriedem | alex_xu: https://etherpad.openstack.org/p/nova-ptg-queens | |
| 14:40:42 | mriedem | oh http://ptg.openstack.org/ | |
| 14:40:45 | mriedem | ^ is the home page | |
| 14:41:39 | alex_xu | mriedem: ah, thanks, surprise the home page change like that | |
| 14:41:50 | mriedem | well, it's the home page for devs | |
| 14:41:56 | mriedem | it's not the marketing glitzy homepage | |
| 14:42:47 | mriedem | gibi: don't forget this one https://review.openstack.org/#/c/499877/ | |
| 14:43:09 | gibi | mriedem: I'm in the process of changing my vote there :) | |
| 14:43:14 | mriedem | whew | |