| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-26 | |||
| 17:45:18 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | |
| 17:45:41 | bauzas | dansmith: I forgot to make a note here | |
| 17:45:54 | bauzas | dansmith: I answered edleafe's on a previous PS | |
| 17:46:15 | bauzas | and sorry pidgin crashed when I was replying to you about the _enabled_filters | |
| 17:46:39 | bauzas | I preferred to explicitly ask for them in L48 given we could remove them from the default list later on | |
| 17:46:51 | dansmith | you use pidgin for irc like a n00b? :) | |
| 17:46:52 | dansmith | alright | |
| 17:47:41 | bauzas | dansmith: indeed, I love GUIs | |
| 17:47:51 | melwitt | that was a burn | |
| 17:48:05 | bauzas | and you know what ? I loooove Atom | |
| 17:48:10 | artom | dansmith, let me guess, you jiggle magnets around some wires to send your messages? | |
| 17:48:14 | bauzas | we're almost Friday | |
| 17:48:28 | bauzas | so I think I have the wildcard for telling that | |
| 17:48:52 | dansmith | bauzas: -1 on the middle patch | |
| 17:48:55 | dansmith | I think the test is broken | |
| 17:49:03 | bauzas | dansmith: ack, lookind | |
| 17:49:05 | dansmith | bauzas: I love guis too | |
| 17:49:20 | bauzas | oh "this" ans "that" | |
| 17:49:31 | bauzas | my English teacher should hit me | |
| 17:49:53 | dansmith | bauzas: that's the bottom, see the middle one | |
| 17:50:17 | bauzas | dansmith: I know :) | |
| 17:50:20 | dansmith | okay | |
| 17:50:22 | bauzas | dansmith: I just saw your comment :) | |
| 17:50:26 | bauzas | dansmith: for the middle patch | |
| 17:50:28 | bauzas | good points | |
| 17:50:32 | bauzas | I can respin quickly | |
| 17:50:39 | bauzas | for the 2nd test I'm adding | |
| 17:51:10 | dansmith | I would recommend waiting until we get a run so we know if the top one is going to work, but if you have to go before it will finish, then just push | |
| 17:51:15 | bauzas | that is not running init_host, that's rather a defensive patch for making sure anyone can't just change nova.conf.placement opt and add a default without being kicked in the ass | |
| 17:51:34 | bauzas | dansmith: well, I'll get an IndentationError anyway, right? | |
| 17:51:41 | bauzas | dansmith: so I'm pretty sure it would fail anyway | |
| 17:51:52 | bauzas | don't know how I left that FWIW | |
| 17:52:05 | bauzas | probably because I said I love Atom in public | |
| 17:52:12 | dansmith | oh I see, well, might be good to assert the positive case in another test anyway | |
| 17:53:55 | bauzas | dansmith: I can put that elsewhere | |
| 17:54:08 | bauzas | dansmith: if you have ideas | |
| 17:54:20 | bauzas | dansmith: I'm just about uploading a new Ps | |
| 17:54:28 | bauzas | dansmith: unless you disagree | |
| 17:54:39 | dansmith | don't disagree | |
| 17:57:40 | bauzas | k | |
| 17:57:44 | bauzas | shooting, then | |
| 18:02:14 | johnthetubaguy | melwitt: oh, resize is a bit of a bummer in that quota logic. | |
| 18:02:20 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Scheduler calling the Placement API https://review.openstack.org/417961 | |
| 18:02:21 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Block starting compute unless placement conf is provided https://review.openstack.org/425806 | |
| 18:05:12 | melwitt | johnthetubaguy: yeah, it's ugly. it would look a lot better with the check_deltas idea | |
| 18:05:46 | johnthetubaguy | melwitt: I was thinking, its not until resize confim that we update the DB with the new larger flavor so its counted correctly, thats quite a big window | |
| 18:06:09 | dansmith | bauzas: did you not see my comment on the last patch about enabling service version caching? | |
| 18:06:18 | bauzas | dansmith: it seems not | |
| 18:06:23 | dansmith | bauzas: okay, I'll fix | |
| 18:06:25 | bauzas | my bad | |
| 18:06:38 | dansmith | oh, | |
| 18:06:51 | dansmith | I thought you were leaving soon, it's only 1900 your time right? | |
| 18:07:18 | bauzas | dansmith: I'm still here for a couple of hours :) | |
| 18:07:19 | dansmith | either way, let us wait for a run of the top one anyway | |
| 18:07:27 | dansmith | and if it's after you leave I'll fix it | |
| 18:07:27 | melwitt | johnthetubaguy: ah, yeah. it's kind of like that now though, since it does a reserve right away, and quota is checked against reserved + in_use. the quota commit is in the confirm_resize but all that does it delete the reservation and increase the in_use | |
| 18:07:32 | bauzas | dansmith: I just need to bail out for 60 mins | |
| 18:07:36 | dansmith | okay | |
| 18:07:44 | bauzas | dansmith: but yeah, it's 7:07pm | |
| 18:07:59 | bauzas | dansmith: so I should be back around 9pm to midnight | |
| 18:08:05 | melwitt | johnthetubaguy: oh, you mean with the new way, the in_use doesn't increase until the confirm. yeah | |
| 18:08:05 | dansmith | okay | |
| 18:08:11 | bauzas | ie. 2000UTC to 2300UTC | |
| 18:08:24 | johnthetubaguy | melwitt: but the reservations count towards quota, so its a bit different | |
| 18:08:31 | bauzas | dansmith: I can do the fix related to your -1 | |
| 18:08:55 | johnthetubaguy | melwitt: yeah, I think the count instances just needs to use the new flavor if its a pending resize (I kinda hope there is a cleaner way though) | |
| 18:08:56 | bauzas | dansmith: but if you have time for it, go | |
| 18:08:57 | dansmith | bauzas: okay but let the top patch finish a run first | |
| 18:09:08 | bauzas | dansmith: okay, so I leave you implement that | |
| 18:09:12 | melwitt | johnthetubaguy: yeah, I get what you mean now. the old way reflects the increased resource usage immediately whereas the new way defers it until the confirm | |
| 18:09:18 | bauzas | dansmith: because I'll be off for 1.5h-ish | |
| 18:09:30 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | |
| 18:09:41 | bauzas | now the patches are in the gate, I'm bailing out but I'll be back | |
| 18:09:49 | johnthetubaguy | melwitt: yeah, sorry, thats what I mean | |
| 18:09:52 | bauzas | andreykurilin: where are we with the client side ? | |
| 18:09:53 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | |
| 18:10:21 | bauzas | andreykurilin: oh, it's missing another +2/+W | |
| 18:10:52 | melwitt | johnthetubaguy: I had been thinking it's good to not reflect the usage until it's really actually used. but, that opens the window for things to get past and go over quota when confirm happens later | |
| 18:10:53 | bauzas | andreykurilin: when are you planning to deliver a new revision ? | |
| 18:11:09 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (console) https://review.openstack.org/409402 | |
| 18:11:21 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (cfg drv) https://review.openstack.org/409404 | |
| 18:11:41 | bauzas | andreykurilin: oh, you're saying all johnthetubaguy's comments can be addressed in a follow-up | |
| 18:11:42 | andreykurilin | mriedem bauzas : let's release novaclient without merging fix for token+cli. It was broken for a year, so it looks like this feature is not in demand, but we need to release new version of novaclient now to release all warning messages which were introduced in 7.0.0 + we need full keystone v3 support | |
| 18:12:29 | bauzas | andreykurilin: the token thing is fixed, that's just the CLI which is not yet fixed, right? | |
| 18:12:45 | bauzas | andreykurilin: unless I'm wrong | |
| 18:13:00 | andreykurilin | bauzas: yes. Token with via python novaclient's API is fixed | |
| 18:13:08 | andreykurilin | *Token auth | |
| 18:13:31 | bauzas | andreykurilin: could you then split your CLI change and just leave the reno file in a separate change ? | |
| 18:13:37 | melwitt | johnthetubaguy: yeah, that would not be great to have to consider pending resize. I wonder if it could be as simple as checking count and quota in the confirm too. and if you are late with your confirm, you may not get to confirm it and have to revert | |
| 18:13:45 | bauzas | andreykurilin: I'd like to include that note for the 7.1.0 release notes | |
| 18:14:00 | bauzas | andreykurilin: and I feel brave enough for a single +2/+W | |
| 18:14:07 | melwitt | but I guess that's not a good experience | |
| 18:14:10 | andreykurilin | bauzas: sorry, I'm not near PC now:( | |
| 18:14:37 | bauzas | andreykurilin: okay, lemme do that then | |
| 18:15:00 | andreykurilin | bauzas Ok, it looks like I'll able to do it in 10-15 mini | |
| 18:15:06 | andreykurilin | *minutes | |
| 18:15:11 | openstackgerrit | Dirk Mueller proposed openstack/nova: [WIP] Handle boot_index in block_device_info also for volumes https://review.openstack.org/425813 | |
| 18:15:14 | johnthetubaguy | melwitt: I did wonder about that, it just feels like you could do loads of resizes to use too much quota, and just not bother doing and confirms so you get to keep the extra big instances | |
| 18:15:14 | bauzas | andreykurilin: I'll need to bail out for 1.5 hours | |
| 18:15:26 | bauzas | andreykurilin: so it's up to you, your call | |