| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-07 | |||
| 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: Test resource allocation during soft delete https://review.openstack.org/495159 | |
| 14:19:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | |
| 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 | |
| 14:43:15 | mriedem | ok | |
| 14:43:48 | mriedem | i have to modify something in the 2nd to bottom patch for sylvain so i was waiting to do that | |
| 14:47:35 | gibi | I'm OK with the first 4 patches in that series and just started reading the 5th | |
| 14:55:46 | dtantsur | hi folks! I'm afraid to imagine how often you've heard this question, but.. what's the difference between nova migrate and nova evacuate? | |
| 14:55:54 | dtantsur | I'm figuring out which one fits better into https://review.openstack.org/449155 | |
| 14:56:50 | edleafe | dtantsur: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/ | |
| 14:57:11 | dtantsur | thanks! | |
| 14:58:00 | edleafe | dtantsur: I keep that one handy, because after all these years it's still confusing | |
| 15:00:07 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721 | |
| 15:00:55 | stephenfin | sahid, bauzas, gibi: ^ addressed sahid's comments | |
| 15:00:58 | stephenfin | I think... | |
| 15:03:12 | gibi | stephenfin: I think so too | |
| 15:04:15 | bauzas | stephenfin: +Wiiii | |
| 15:08:49 | bauzas | mriedem: so, 2 points | |
| 15:09:11 | bauzas | mriedem: just remove the upgrade reno and then I'll +2 https://review.openstack.org/#/c/499399/6 | |
| 15:09:41 | bauzas | mriedem: also, about your skip_filters thoughts, just remember that we already have something called force_hosts | |
| 15:09:51 | bauzas | of course, that field is terribly named | |
| 15:10:22 | bauzas | but we could just not add yet again a new RPC API parameter, and rather just use the RequestSpec object for that | |
| 15:10:25 | gibi | stephenfin: I made an answer to your comment in https://review.openstack.org/#/c/463946/9/nova/tests/functional/notification_sample_tests/test_keypair.py@20 | |
| 15:10:34 | openstackgerrit | Merged openstack/nova-specs master: Convert consoles code to use objects framework https://review.openstack.org/500975 | |
| 15:11:03 | bauzas | mriedem: like, we could create a new field (and deprecate the legacy one) called forced_destination | |
| 15:11:12 | bauzas | having a Destination value | |
| 15:11:37 | bauzas | and so, if you pass a forced host, then the scheduler would claim | |