| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-17 | |||
| 20:08:48 | mriedem | i figured it was older | |
| 20:08:53 | mriedem | what i refer to as "the before times" | |
| 20:08:59 | comstud | haha | |
| 20:09:15 | comstud | well, it's new enough that I don't remember much about it | |
| 20:09:19 | mriedem | nova, BC (before comstud) | |
| 20:09:20 | comstud | it is familiar to me, so I know it must have existed | |
| 20:09:35 | comstud | but I might have been more focused on Ironic at the time | |
| 20:09:40 | mriedem | ok, | |
| 20:09:43 | mriedem | well, fyi, don't use it | |
| 20:10:03 | comstud | haha | |
| 20:10:08 | comstud | don't use openstack? | |
| 20:10:09 | comstud | yeah, I know. | |
| 20:10:23 | mriedem | ouch | |
| 20:11:29 | comstud | in any case, i remember feeling like the quota code needed a complete re-do | |
| 20:11:44 | mriedem | that's what melwitt is working on | |
| 20:12:01 | mriedem | taking out reservations and commit/rollback | |
| 20:12:34 | mriedem | if i mention her enough, she's bound to show up too :) | |
| 20:13:48 | comstud | hehe | |
| 20:16:48 | Apoorva | Cinder CI tests are failing with following error http://paste.openstack.org/show/606920/ Can someone help? | |
| 20:17:10 | Apoorva | Failures are in n-cpu.log http://openstack-ci.tintri.com/tintri/refs-changes-58-430858-3/logs/screen-n-cpu.log.txt | |
| 20:17:26 | mriedem | Apoorva: is your CI using the ubuntu cloud archive? | |
| 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 | |