Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-20
05:57:39 jianghuaw_ gibi, will you help to review this patch which we mentioned in last Thursday? https://review.openstack.org/#/c/512580/
06:14:57 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
06:30:33 moshele stephenfin: hi
06:36:09 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190
06:44:49 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
06:45:18 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Clean up exception caught in _validate_and_build_base_options https://review.openstack.org/463253
07:26:42 openstackgerrit Takashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513717
07:27:24 openstackgerrit Takashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513717
08:12:30 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205
08:18:12 openstackgerrit Chris Dent proposed openstack/nova master: Include project_id and user_id in AllocationList.get_all_by_consumer_id https://review.openstack.org/512420
08:18:13 openstackgerrit Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
08:18:13 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid} https://review.openstack.org/510626
08:44:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Implement query param schema for sec group APIs https://review.openstack.org/521353
08:56:44 moshele stephenfin: hi
09:42:44 yankcrime webdev
09:53:36 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid} https://review.openstack.org/510626
09:53:37 openstackgerrit Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
09:54:10 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove reserve_quota_delta https://review.openstack.org/521469
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.

Earlier   Later