Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-17
21:34:25 jaypipes mriedem: k, +2 from me.
21:34:50 jaypipes mriedem: that one would be great starter work for a newcomer to Nova.
21:35:07 jaypipes mriedem: do you mind if I reach out on the ML to find someone for that one?
21:35:20 jaypipes mriedem: not sure if you'd already talked to anyone.
21:35:34 efried mriedem jaypipes I threw a comment in there
21:36:00 efried And am suspiciously present as you're discussing giving an assignment to a newcomer...
21:36:22 jaypipes efried: you consider yourself a newcomer? :)
21:36:51 efried Been sequestered in my powervm cubby for two or three years. Know practically nothing about nova at large.
21:36:59 jaypipes efried: you've been around IRC for at least a month now. That makes you an old Nova hat :P
21:37:16 efried Oh, okay. One month is long time, got it.
21:37:21 jaypipes efried: if you're interested in that one, and mriedem says cool, go for it ;)
21:38:21 efried If it would be more appropriate for an even noobier noob, I won't cry in my beer.
21:40:32 jaypipes efried: no, honestly, it's a good chunk of work that really needs doing. so if you're interested, I'd recommend you pick it up.
21:41:03 efried jaypipes Roger wilco. Which ML, btw? I've been following 'openstack@'. Is there another one I should be watching? openstack-dev?
21:41:59 jaypipes ya, openstack-dev@
21:45:02 efried jaypipes Is this the kind of thing that other services should also be doing?
21:52:26 jaypipes efried: yup.
21:53:44 mriedem jaypipes: efried: i didn't have anyone assigned
21:53:46 mriedem so go nuts
21:58:32 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Use service catalog to get endpoint URLs https://review.openstack.org/454983
22:01:27 mriedem efried: it's all yours, thanks
22:01:38 mriedem efried: btw, raj_singh already has a start on this for glance connections inside nova
22:02:06 mriedem https://review.openstack.org/#/c/412634/
22:04:15 efried mriedem Thanks
23:20:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (2/4) https://review.openstack.org/455568
23:21:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (3/4) https://review.openstack.org/455570
23:21:36 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (4/4) https://review.openstack.org/455573
23:21:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565
23:56:24 openstackgerrit Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230
#openstack-nova - 2017-04-18
00:24:25 openstackgerrit STEW TY proposed openstack/nova master: Transform instance.unrescue notifications https://review.openstack.org/388275
01:23:17 openstackgerrit shaofeng cheng proposed openstack/nova master: Update etherpad url https://review.openstack.org/457198
01:31:52 openstackgerrit Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230
01:57:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix response code and parameters in evacuate https://review.openstack.org/457167
02:10:34 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova master: Remove json-schema extension variable for resize https://review.openstack.org/457430
02:11:08 oomichi gmann: alex_xu: ^^^ is easy one
02:14:42 alex_xu oomichi: got it
02:19:49 gmann oomichi: +1, i did search final schema for resize due to that name, thanks
02:20:16 alex_xu oomichi: done
02:20:36 oomichi gmann: alex_xu: thanks :)
02:21:00 alex_xu oomichi: np
02:21:13 oomichi gmann: yeah, I found this on the review of https://review.openstack.org/455570
02:22:09 oomichi gmann: I guessed the resize jsonschema was extended from the name, but actually not. so it would be better to rename
02:22:24 gmann oomichi: yea
02:28:28 oomichi takashin: can you take a look at https://review.openstack.org/#/c/457167 ? I am not sure the error status code 503 also is necessary for evacuate API
02:29:14 takashin oomichi: Thank you for your review.
02:30:08 takashin oomichi: I don't know 503 is necessary or not, too.
02:33:24 oomichi takashin: I did some investigation now and actually APIs which could return 503 are fping and tenant-network only
02:33:45 oomichi takashin: the corresponding api-ref contains it on these APIs.
02:34:05 oomichi takashin: the other api-ref which includes this case should be invalid I think now
02:34:26 takashin oomihi: Thanks. I will remove 503 in evacuate API reference.
02:34:35 oomichi takashin: cool, thanks
02:48:25 oomichi gmann: about https://review.openstack.org/#/c/456430 I did think about it before, but I don't have strong opinion about that
02:48:57 oomichi gmann: the enum list clearly says the type is string even if not having 'type'
02:49:28 oomichi gmann: it is also an option to remove 'type' from such definitions instead
02:49:41 oomichi for consistency
02:50:54 gmann oomichi: actually, enum values can be accessed with index so it was confusing to me. but yes json schema defien enum always a string i think
02:53:34 oomichi gmann: it also would be fine to add type to all schema, but current patch seems necessary to be updated anyways
02:55:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix response code and parameters in evacuate https://review.openstack.org/457167
02:55:21 gmann oomichi: did not get, you mean to updated with takashin comments ?
02:55:34 oomichi gmann: yep
02:56:09 gmann oomichi: yea i can incorporate those separately if takashin is ok :)
02:56:21 gmann takashin: ^^ what you say?
03:02:00 takashin gmann: You can do it separately. But it is small changes. So it is good for reviewers to do it together.
03:09:29 gmann takashin: sure, ll do
03:30:02 oomichi takashin: one question about https://review.openstack.org/#/c/414926/39/nova/tests/functional/test_servers.py
03:30:27 oomichi takashin: why do we need to specify a certain image_uuid for these test cases?
03:31:15 takashin oomichi: Just a moment, please.
03:31:22 oomichi takashin: New tests seem to require the same host only, so I could not understand why we need to specify image_uuid
03:54:22 openstackgerrit Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi V2 in nova https://review.openstack.org/453493
03:55:20 takashin oomichi: Image API was deprecated in API version 2.36.
03:55:50 takashin oomichi: So an error is raised in https://github.com/openstack/nova/blob/5a556a720f5a7394cab4c84fa6202976c6190b23/nova/tests/functional/integrated_helpers.py#L141 .
03:56:51 oomichi takashin: does that mean 2.36+ functional tests should specify image_uuid, right?
03:57:02 takashin oomichi; And I modified _build_minimal_create_server_request method in _IntegratedTestBase,
03:57:55 takashin oomichi: Yes. (in my modification.)
03:58:37 oomichi takashin: I see, thanks. Then it would be better to have NOTE on https://review.openstack.org/#/c/414926/39/nova/tests/functional/integrated_helpers.py
03:58:50 oomichi to explain the above your comment
03:59:11 takashin oomichi: Thank you for your review. I will add a note.
03:59:21 oomichi takashin: thanks again :)
05:14:46 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration https://review.openstack.org/451657
05:37:18 openstackgerrit Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi V2 in nova https://review.openstack.org/453493
05:56:33 openstackgerrit Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi V2 in nova https://review.openstack.org/453493
06:24:52 openstackgerrit Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi V2 in nova https://review.openstack.org/453493
06:26:49 openstackgerrit Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi V2 in nova https://review.openstack.org/453493
07:28:36 openstackgerrit Alex Xu proposed openstack/nova master: Deprecate Multinic, floatingip action and os-virtual-interface API https://review.openstack.org/457181
07:35:59 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot https://review.openstack.org/394321
07:39:24 openstackgerrit shaofeng cheng proposed openstack/nova master: Update nova version newton to ocata in upgrade.rst https://review.openstack.org/457197
07:39:55 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801
07:43:58 gcb alex_xu: please help review https://review.openstack.org/457188 , that fixes http://logs.openstack.org/periodic/periodic-nova-py35-with-oslo-master/74800d1/testr_results.html.gz
07:45:03 gcb alex_xu: this blocks we release oslo.config 4.0, hope we can fix it in Nova side before we release new version of oslo.config
08:27:05 openstackgerrit Lei Zhang proposed openstack/nova master: Add sync traits command for placement https://review.openstack.org/450125
08:27:46 openstackgerrit LiChaoLei proposed openstack/nova master: Fix bug of delete image_base_image_ref after rebuild instance https://review.openstack.org/457514
08:50:13 openstackgerrit Gábor Antal proposed openstack/nova master: [WIP] Transform instance.live_migration_rollback notification https://review.openstack.org/402124
09:02:40 openstackgerrit Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi V2 in nova https://review.openstack.org/453493
09:09:42 openstackgerrit Roman Podoliaka proposed openstack/osc-placement master: tests: add a hook for functional testing in the gate https://review.openstack.org/452122
09:09:43 openstackgerrit Roman Podoliaka proposed openstack/osc-placement master: CLI for resource providers https://review.openstack.org/457532
09:09:43 openstackgerrit Roman Podoliaka proposed openstack/osc-placement master: CLI for inventories https://review.openstack.org/457533
09:09:44 openstackgerrit Roman Podoliaka proposed openstack/osc-placement master: CLI for allocations https://review.openstack.org/457534
09:09:44 openstackgerrit Roman Podoliaka proposed openstack/osc-placement master: CLI for usages https://review.openstack.org/457535
09:13:13 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot https://review.openstack.org/394321
09:23:37 alex_xu gcb: done

Earlier   Later