| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-17 | |||
| 20:05:52 | comstud | the soft deleting of instances in particular caused issues | |
| 20:06:23 | mriedem | yes, soft delete is terrible | |
| 20:06:24 | comstud | i know I'd made a number of comments on how to fix it | |
| 20:06:27 | comstud | but | |
| 20:06:35 | mriedem | well, | |
| 20:06:35 | comstud | i don't remember exactly what they were | |
| 20:06:38 | comstud | i think it was... | |
| 20:06:44 | mriedem | soft delete + quotas + local delete code in the api | |
| 20:06:47 | comstud | move some code very close to the real delete. | |
| 20:07:01 | comstud | but that's the other issue... | |
| 20:07:10 | comstud | edge cases when compute host is down | |
| 20:07:13 | comstud | etc | |
| 20:07:29 | mriedem | yeah see https://review.openstack.org/#/c/386756/ | |
| 20:07:40 | mriedem | we've also duplicated a lot of that local delete logic too | |
| 20:07:47 | mriedem | since you've been gone | |
| 20:08:10 | mriedem | oh, let's not forget shelve offloaded | |
| 20:08:33 | comstud | ya, i dunno when shelved was added | |
| 20:08:41 | comstud | i think it was after I stopped focusing on nova | |
| 20:08:46 | mriedem | really? | |
| 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 | |