| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-17 | |||
| 20:17:49 | mriedem | this came up in the ML, something about the version of libvirt and python-libvirt not matching | |
| 20:17:50 | Apoorva | mriedem I recently upgraded to Ubuntu 16.04 | |
| 20:19:15 | mriedem | Apoorva: see http://lists.openstack.org/pipermail/openstack-dev/2017-April/114912.html - i think somewhere in that thread someone mentions this issue with python-libvirt | |
| 20:20:24 | rfolco | Apoorva, we solved this by rebuilding python-libvirt as mriedem said. Basically our pip wheels were out of sync. | |
| 20:21:25 | mriedem | gah the CellDatabases fixture isn't in newton | |
| 20:40:16 | Apoorva | rfolco thanks. I have libvirt-python (3.2.0) installed on the machine running devstack | |
| 20:55:47 | mriedem | jaypipes: commented on the claims in scheduler spec too https://review.openstack.org/#/c/437424/ | |
| 21:00:04 | openstackgerrit | Monty Taylor proposed openstack/python-novaclient master: Add novaclient client_name and client_version to user-agent https://review.openstack.org/456638 | |
| 21:25:17 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Use service catalog to get endpoint URLs https://review.openstack.org/454983 | |
| 21:25:19 | mriedem | jaypipes: updated ^ | |
| 21:25:32 | jaypipes | mriedem: cool, thank you sir. | |
| 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 | |