| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-20 | |||
| 09:54:11 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback https://review.openstack.org/521470 | |
| 10:09:46 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [Placement] Fix parameter order in placement API ref https://review.openstack.org/516233 | |
| 10:10:47 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix parameter order in placement API ref https://review.openstack.org/516233 | |
| 10:17:29 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | |
| 10:33:47 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix format in placement API ref https://review.openstack.org/521483 | |
| 10:54:18 | openstackgerrit | Maciej Kucia proposed openstack/nova-specs master: Non-unique network names in Servers IPs API response https://review.openstack.org/521392 | |
| 11:11:33 | openstackgerrit | Yikun Jiang proposed openstack/nova master: [WIP] Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | |
| 11:13:14 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | |
| 11:31:17 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placemant] Add 'CUSTOM_' prefix description in API ref https://review.openstack.org/521495 | |
| 11:32:03 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add 'CUSTOM_' prefix description in API ref https://review.openstack.org/521495 | |
| 12:08:43 | efried_zzz | cdent Greetings! How was your vacation time Down Under? | |
| 12:08:55 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add aggregate link note in API ref https://review.openstack.org/521502 | |
| 12:09:18 | cdent | hola efried_zzz : too short but otherwise very nice, I can link to some pics if you like | |
| 12:09:33 | efried | cdent Sure! | |
| 12:09:57 | cdent | https://t.co/BWtRih7zPU <- expands to a google pics link | |
| 12:11:51 | efried | cdent Wow, looks like the weather just sucked. | |
| 12:12:00 | cdent | yeah, was dreadful | |
| 12:12:16 | cdent | mid 20s and sunny, all the time. couldn’t be worse. | |
| 12:14:19 | efried | cdent I know how to get to the building docs - but I haven't been able to find the placement API ref from there. | |
| 12:14:52 | efried | cdent The links on the main page point to the real published docs. | |
| 12:16:28 | cdent | if you click on the right test, it goes straight to the placement docs: I named the test in my response, but can’t remember now | |
| 12:16:32 | cdent | or do you mean links somewhere else? | |
| 12:17:50 | efried | cdent ooohhh, crap, that's right, placement docs are a different build job. | |
| 12:18:17 | efried | I didn't get that far down - I just hit the first sphinx doc build job and was like, where tf is the placement ref?? | |
| 12:19:20 | cdent | It’s all part of this glorious future where placement is separate | |
| 12:20:07 | stephenfin | moshele: Hey | |
| 12:20:17 | stephenfin | What's up? | |
| 12:22:52 | moshele | stephenfin: regarding the revert https://review.openstack.org/#/c/518833/ | |
| 12:23:03 | stephenfin | Yup? | |
| 12:23:23 | moshele | stephenfin: so I have a fix https://review.openstack.org/#/c/519066/ | |
| 12:23:35 | stephenfin | moshele: Excellent. I'll take a look at that today :) | |
| 12:23:50 | moshele | stephenfin: but I didn't find the guy to say if it not breaking heat | |
| 12:23:55 | efried | cdent Am I missing the part of the series where GET /allocation_candidates returns the new format expected by PUT /allocations ? | |
| 12:24:30 | stephenfin | moshele: Hmm, that probably wouldn't be a bad idea. ramishra was the nick. I wonder if he's on #heat? | |
| 12:25:28 | moshele | stephenfin: I was looking for him in openstack-heat :) | |
| 12:25:49 | stephenfin | Ahh, yeah. Look at the welcome message ;) | |
| 12:27:01 | moshele | stephenfin: feel free to review the patch I should solve the problem | |
| 12:27:14 | moshele | s/I/ti/ | |
| 12:27:21 | stephenfin | Will do (y) | |
| 12:29:56 | cdent | efried: line 286ish https://review.openstack.org/#/c/510626/14/nova/api/openstack/placement/handlers/allocation.py | |
| 12:30:20 | cdent | oh sorry, that’s the put you want to know the get | |
| 12:30:56 | cdent | it’s in the _allocations_dict method | |
| 12:31:10 | cdent | GET was already nearly correct, it just needed the project_id and user_id added | |
| 12:31:21 | cdent | GET being different was why PUT was changed to be more like GET | |
| 12:31:46 | efried | cdent I'm talking about GET /allocation*_candidates* | |
| 12:32:07 | cdent | ah, sorry, names are hard on a monday | |
| 12:33:06 | efried | cdent The latest changes to the spec were that the resource request bit of the GET /allocation_candidates response was going to be changed to match what PUT /allocations expects so that you can still just turn it around and PUT it without looking at it (and converting it) | |
| 12:34:19 | cdent | oh, yeah, I think that’s just not done yet, as that was decided right before I started travelling | |
| 12:34:25 | cdent | so I’ll stick a -W on there somewhere | |
| 12:35:02 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [functional] Remove duplication of boot helper https://review.openstack.org/521071 | |
| 12:35:03 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | |
| 12:35:20 | efried | cdent Okay, cool. Didn't want to embarrass my self by +1ing that. Dodged a bullet, phew! | |
| 12:36:40 | cdent | My memory is starting to surface: I updated the spec and volunteered you or jay to do the work, but then you and jay fell in the refactoring hole related to resources etc and so now it is back to me. | |
| 12:36:41 | cdent | Or something | |
| 12:37:11 | efried | cdent Yes, that's the long and short of it. Though refactoring isn't the *only* thing that happened - see update email 41.75 :) | |
| 12:37:26 | cdent | that’s the “etc” part | |
| 12:37:47 | cdent | speaking of: thanks for doing those. you can carry on doing it if you’re feeling keen, or I’m happy to take it back if not | |
| 12:40:12 | efried | cdent I'd be fine either way. I've been happily slinging code for virt drivers getting hands on ProviderTree, but that stuff (as well as granular resource request work) is high atop other series work, so not urgent. Whereas symmetric stuff is pretty close to done. | |
| 12:40:58 | efried | cdent So if you're buried under other stuff, I can take it on for the sake of expediency. But if you've got the bandwidth to finish it, you should. | |
| 12:41:00 | cdent | the “those” I meant was the weekly updates | |
| 12:41:12 | efried | Hah. Oh. | |
| 12:41:22 | efried | You're welcome - and yes, please take them back. | |
| 12:41:40 | cdent | roger that. and I’ll take back the symmetric stuff | |
| 12:41:47 | efried | cdent I made sure to leave you #42 :) | |
| 12:41:53 | cdent | so kind | |
| 12:42:18 | efried | cdent And speaking of taking stuff over - you feel like running the scheduler meeting today? | |
| 12:42:36 | efried | edleafe- is out | |
| 12:44:14 | cdent | oh is he now | |
| 12:44:23 | cdent | yeah, I can do it | |
| 12:46:17 | efried | cdent Cool | |
| 12:46:44 | efried | cdent Being an American in the UK, are you going to be out later this week or no? | |
| 12:46:46 | cdent | just to check my calendar it is scheduled for a 1.25 hours from now? | |
| 12:46:55 | cdent | no | |
| 12:47:17 | cdent | Usually I take holiday back to the US to see family, but with sydney just now I couldn’t stand the idea of yet more international travel | |
| 12:47:51 | efried | cdent Okay. Yes, scheduler meeting is 1.25 hours from now if my UTC clock is sane. (DST is stupid) | |
| 12:48:01 | efried | cdent FYI I'll be out W-F | |
| 13:27:14 | niraj_singh | efried: i was going through ( https://review.openstack.org/#/c/512329/ ) this patch. A new exception is cretaed i.e. "ServiceUserTokenNoAuth". But i didn't find where this exception is handled. | |
| 13:36:10 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513717 | |
| 13:36:41 | efried | niraj_singh One sec. | |
| 13:37:54 | efried | niraj_singh Okay - what do you mean "handled"? | |
| 13:38:17 | efried | niraj_singh The motivation behind this patch was to make an exception that was easier for the user to understand. | |
| 13:38:32 | efried | niraj_singh So it's not actually supposed to be handled anywhere - it bubbles up to the user. | |
| 13:40:33 | efried | niraj_singh Sorry, by which I mean, it ends up in the logs where the operator can see it. | |
| 13:46:49 | stephenfin | moshele: Done | |
| 13:47:20 | stephenfin | Even though I suggested it, I'm not sure whether we should log an exception message without raising it. I'll let someone else comment on this | |
| 13:48:53 | niraj_singh | efried: I thought it should handle any where. ok thanks for information. | |
| 13:54:03 | openstackgerrit | priyaduggirala proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py https://review.openstack.org/508533 | |
| 13:57:41 | efried | Scheduler meeting in three minutes in #openstack-meeting-alt | |
| 13:57:46 | cdent | whoa | |
| 13:57:59 | efried | Did I just read your mind? | |
| 13:58:00 | cdent | I was, like, literally, typing, literally, like, the same, like thing | |
| 13:58:14 | efried | Great minds, and all that. | |
| 13:58:36 | cdent | we are merely expressions of the platonic nova scheduler dev | |
| 13:59:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref https://review.openstack.org/521541 | |
| 14:00:01 | efried | I had an almost-mother-in-law when I was a youngster who would always know the second half of pithy aphorisms, which was slightly annoying. In this case, "Great minds think alike" apparently concludes, "And fools seldom differ" | |
| 14:02:37 | stephenfin | sean-k-mooney: I'm +2 on this if you want to take a second look https://review.openstack.org/#/c/493082/ | |
| 14:03:43 | stephenfin | efried: nested resource providers are on my review radar for this week | |
| 14:03:58 | stephenfin | (RE: the review request you sent at some point last week :)) | |
| 14:04:06 | efried | stephenfin That's great to hear. Can you join the sched meeting in -alt ? | |
| 14:04:13 | stephenfin | Now? | |