| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-22 | |||
| 13:45:19 | mriedem | kashyap: april 19 | |
| 13:45:43 | kashyap | mriedem: Ah, I'll be starting tomm or at most Monday. | |
| 13:45:57 | jaypipes | cdent: cool. looks like a relatively simple fix. | |
| 13:46:02 | cdent | efried: I pretty much agree with you, but what do we do any extant providers that are the same uuid with different reps and have since diverged? | |
| 13:46:33 | cdent | jaypipes: if you look above in scrollback there are some differing opinions on the right fix, but not vastly so | |
| 13:46:37 | kashyap | mriedem: In your "copious free time", wonder if you can punch any holes in the above change: https://review.openstack.org/#/c/534384 | |
| 13:46:51 | kashyap | If you can't get to it; it's fine. Got enough attention so far, can wait | |
| 13:46:59 | mriedem | kashyap: i already did wrt backports | |
| 13:47:04 | mriedem | so don't really want to talk about that one honestly | |
| 13:47:30 | kashyap | Okay, no prob; dansmith suggested on the post a bit more palatable. | |
| 13:47:31 | efried | cdent: That's a neat question. Let the user clean 'em up? | |
| 13:47:48 | kashyap | (But, IMHO, it just seems like (even Tony) is trying to overly stick to the "letter of the law") | |
| 13:47:55 | efried | cdent: ...which entails allowing DELETE APIs to accept and use non-normalized UUIDs... | |
| 13:48:05 | lyarwood | mriedem: sorry missed your ping earlier, looking now | |
| 13:48:31 | mriedem | dansmith: question in tssurya's scheduler sighup patch about locking https://review.openstack.org/#/c/550527/ | |
| 13:48:42 | mriedem | dansmith: i have a feeling we should be locking on that new attribute when we're resetting it | |
| 13:49:47 | efried | cdent: Gotta say it's pretty doubtful that something like this would have happened IRL, because the client - whatever it is - will generally be using one code path to create providers. So they'll have created 'em all with the same UUID format. | |
| 13:50:18 | dansmith | mriedem: okay I failed to circle back to those so I'll try to do that this morning dodging meetings | |
| 13:51:17 | cdent | efried: that's probably true, I'm just asking the questions for completeness. Also, just sake of the record and all that (in the sense that it is not really germane for today's reality): the whole point of having an http api is so that there are and will be multiple clients and different code | |
| 13:51:46 | efried | yup, I get that. | |
| 13:52:34 | efried | cdent: I say we just fix the glitch, and not bother making a cleanup tool until someone claims they need it. | |
| 13:53:42 | efried | cdent, melwitt, mriedem: This would be a good bug for new contributors. | |
| 13:54:18 | edleafe | efried: +1 on not worrying about it until we need to worry about it | |
| 13:55:23 | mriedem | tag it with low-hanging-fruit then | |
| 13:57:04 | efried | done | |
| 13:58:08 | sq4ind | sean-k-mooney, efried any idea ? | |
| 13:58:31 | efried | sq4ind: None here, sorry. | |
| 13:58:47 | sq4ind | efried, thanks | |
| 13:58:57 | efried | sq4ind: But you may want to recap for jaypipes - this sounds up his alley to me. | |
| 13:59:04 | gibi | nova meeting starts in about a minute | |
| 14:03:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Create volume attachment during boot from volume in compute https://review.openstack.org/541420 | |
| 14:04:04 | efried | mriedem: Poetic patch title ^ | |
| 14:04:41 | mriedem | is it? | |
| 14:10:48 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813 | |
| 14:13:30 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add request_id to isntance action notifications https://review.openstack.org/553288 | |
| 14:25:00 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add request_id to instance action notifications https://review.openstack.org/553288 | |
| 14:25:42 | bauzas | mriedem: come to France and you'll enjoy the day | |
| 14:25:57 | bauzas | well, if you can... | |
| 14:28:36 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add request_id to instance action notifications https://review.openstack.org/553288 | |
| 14:33:23 | mriedem | need another core for the bottom 2 patches in this series to move the nova-cells-v1 job in-tree and then change it to use neutron, which is the first part of removing nova-network https://review.openstack.org/#/c/549780/ | |
| 14:36:03 | stephenfin | mriedem: I'll grab em | |
| 14:44:09 | sq4ind | I think I've found why I was having issues... basically when listing cells in placement I had one out of three rabbitmq servers in the transport url, and only hosts connected to this rabbitmq server were updating their resource usage. Now I've fixed the url, I am also moving VMs from hosts and removing hosts from placement cells, and then recreating them. After that everything works as it should be | |
| 14:44:16 | sq4ind | hope that makes sense | |
| 14:46:38 | efried | sq4ind: Glad you figured it out - sorry we couldn't be more help. | |
| 14:55:40 | sq4ind | efried, no problem :D I like to dig more and more into openstack internals :D And you were very helpful: you've gave some clues where to look at it :D, thanks ! :) | |
| 15:01:21 | openstackgerrit | Eric Fried proposed openstack/nova master: Support extending attached ScaleIO volumes https://review.openstack.org/554679 | |
| 15:01:26 | mriedem | stephenfin: thanks for hitting those cells ci patches | |
| 15:03:17 | stephenfin | mriedem: np | |
| 15:06:50 | mriedem | edmondsw: we don't need both a powervm-resize and powervm-cold-migrate blueprint | |
| 15:06:55 | mriedem | if you implement resize, you have to have cold migrate | |
| 15:07:07 | mriedem | so i'm going to mark the cold migrate one as superseded | |
| 15:07:18 | edmondsw | mriedem yep | |
| 15:07:22 | mriedem | esberglu: ^ | |
| 15:07:56 | edmondsw | rebuild and evac would go in there as well | |
| 15:08:07 | edmondsw | esberglu update the description on that one? | |
| 15:08:14 | edmondsw | (resize) | |
| 15:08:19 | mriedem | eh? | |
| 15:08:23 | mriedem | those are not the same as resize | |
| 15:08:27 | mriedem | nor do they require a blueprint | |
| 15:08:32 | openstackgerrit | Chris Dent proposed openstack/nova master: Use microversion parse 0.2.1 https://review.openstack.org/550265 | |
| 15:08:34 | mriedem | if your driver can spawn and destroy, you support rebuild | |
| 15:09:44 | mriedem | https://blueprints.launchpad.net/nova/rocky should be accurate now right? | |
| 15:09:50 | mriedem | 5 powervm-* blueprints | |
| 15:10:25 | edmondsw | mriedem sorry otp | |
| 15:14:26 | openstackgerrit | Raoul Hidalgo Charman proposed openstack/nova master: Expose shutdown retry interval as config setting https://review.openstack.org/552483 | |
| 15:14:34 | mriedem | does anyone from virtuozzo still work on nova? | |
| 15:14:38 | mriedem | because their CI is borked | |
| 15:14:47 | edmondsw | mriedem yeah, looks right | |
| 15:15:07 | esberglu | edmondsw: mriedem: Will update resize | |
| 15:15:11 | edmondsw | mriedem I meant evac would come only when we support migration | |
| 15:15:19 | edmondsw | esberglu I think mriedem wants it left as-is | |
| 15:15:20 | mriedem | edmondsw: those aren't the same | |
| 15:15:22 | mriedem | or related | |
| 15:15:43 | mriedem | evac is not the same as cold migrate, | |
| 15:15:49 | mriedem | it's a rebuild of the instance on another host | |
| 15:15:55 | mriedem | using spawn and destroy | |
| 15:16:00 | edmondsw | oh, right... | |
| 15:16:29 | edmondsw | sorry, too many different uses of the same term | |
| 15:16:56 | dansmith | mriedem: should I link him or will you? | |
| 15:17:17 | dansmith | edmondsw: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/ | |
| 15:17:30 | edmondsw | dansmith +1 | |
| 15:17:57 | edmondsw | I remember reading that when it was new | |
| 15:18:22 | edmondsw | esberglu ^ | |
| 15:18:28 | kashyap | dansmith: At this point it warrants to be put in the channel topic ;-) | |
| 15:18:32 | dansmith | heh | |
| 15:18:42 | kashyap | "Evacuate", please see: $short-url | |
| 15:19:08 | kashyap | dansmith: Then you can start chiding people: "Don't you read channel topic?!" | |
| 15:19:29 | dansmith | kashyap: make a bot, that will waste more time | |
| 15:20:04 | kashyap | True; it reminds me of the bot on #fedora-devel, where if someone just says: | |
| 15:20:15 | kashyap | Person: Ping | |
| 15:20:17 | kashyap | Bot: https://fedoraproject.org/wiki/No_naked_pings | |
| 15:22:14 | efried | wow | |
| 15:22:19 | efried | TIL | |
| 15:22:29 | dansmith | efried: not everyone agrees with those assertions (like myself) | |
| 15:22:33 | efried | ...that some people are just too damn touchy | |
| 15:22:36 | dansmith | hehe | |
| 15:22:38 | dansmith | yeah that | |
| 15:22:52 | eric-young | efried: thanks for the quick edit on https://review.openstack.org/554679 | |
| 15:22:56 | mriedem | can we now say that nova people aren't as bad as fedora people? | |
| 15:23:07 | efried | eric-young: Yahyoubetcha | |