| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-25 | |||
| 21:33:30 | dansmith | oh we actually have a .services on instance | |
| 21:33:37 | dansmith | I didn't remember | |
| 21:35:55 | mriedem | yeah | |
| 21:35:55 | mriedem | https://bugs.launchpad.net/nova/+bug/1719460 | |
| 21:35:56 | openstack | Launchpad bug 1719460 in OpenStack Compute (nova) "instance.services loading when listing instances regardless of microversion" [Medium,Triaged] | |
| 21:35:58 | mriedem | i think ^ sums it up | |
| 21:36:09 | openstackgerrit | Ed Leafe proposed openstack/nova-specs master: Re-propose nested resource providers spec https://review.openstack.org/505209 | |
| 21:37:14 | mriedem | btw, deleting 1000 instances via the API takes awhile | |
| 21:49:07 | openstackgerrit | Merged openstack/nova master: [placement] correct error on bad resource class in allocation https://review.openstack.org/502205 | |
| 21:52:47 | efried | mriedem Can I get some love on https://review.openstack.org/#/c/488137/ so I can get the rest of the series moving? | |
| 21:53:15 | mriedem | how about tough love | |
| 21:53:32 | efried | I expect nothing less from you | |
| 22:01:36 | mriedem | efried: your unit tests failed | |
| 22:02:15 | efried | sonofa... Okay, I think I know what happened. | |
| 22:03:51 | efried | Okay, since I'm going to need an edit anyway, I guess I might as well rebase... | |
| 22:04:10 | openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | |
| 22:06:12 | openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | |
| 22:06:21 | openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | |
| 22:06:32 | efried | mriedem That ought to fix it ^ | |
| 22:08:16 | mriedem | well, meanwhile +2 on https://review.openstack.org/#/c/506370/ again | |
| 22:08:55 | efried | whee! | |
| 22:09:01 | mriedem | good god it takes a long time to delete 1000 vms | |
| 22:10:53 | dansmith | mriedem: is it actually deleting things or hung? | |
| 22:11:05 | mriedem | down to 512 left | |
| 22:15:01 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | |
| 22:15:26 | openstackgerrit | Eric Fried proposed openstack/nova master: Don't fix protocol-less glance api_servers anymore https://review.openstack.org/505317 | |
| 22:17:39 | jaypipes | mriedem: but don't you know? each launch of a VM does 400 calls to the database... | |
| 22:23:56 | jaypipes | dansmith: easy one from cdent: https://review.openstack.org/#/c/495913/ | |
| 22:32:28 | dansmith | jaypipes: I'm pretty meh on stuff like that in o.vo | |
| 22:32:47 | dansmith | doesn't save anything having that in the library and makes it harder to trace why you have these two fields in your object | |
| 22:32:49 | dansmith | but whatever | |
| 22:32:58 | jaypipes | dansmith: yeah? seems like it's generally useful, no? | |
| 22:33:36 | dansmith | no, it "saves" two lines from a single object in a project, or four for multiples, but makes chasing down the definitions a cross-repo venture | |
| 22:33:43 | jaypipes | dansmith: I had asked cdent to submit that to ovo, so blame me :) | |
| 22:33:49 | dansmith | I blame you. | |
| 22:33:53 | jaypipes | :) | |
| 22:40:49 | mriedem | jaypipes: this is delete | |
| 22:41:08 | mriedem | your callback joke doesn't fly with me | |
| 22:42:12 | jaypipes | mriedem: :) | |
| 22:43:13 | mriedem | ooo cpu pinning failure in the ML | |
| 22:44:57 | mriedem | how many db calls are made when we create a server? | |
| 22:45:02 | mriedem | 375? | |
| 22:46:01 | dansmith | 375, 376, however many it takes | |
| 22:49:09 | mriedem | 7.5 deletes per minute, just counted | |
| 22:49:11 | mriedem | blazing | |
| 22:49:34 | dansmith | you could restack quicker | |
| 22:49:56 | mriedem | probably | |
| 22:50:04 | mriedem | do unstack/stack work? or restack? | |
| 22:53:39 | openstackgerrit | Merged openstack/nova master: Add get_node_uuid() helper to ResourceTracker https://review.openstack.org/506730 | |
| 22:53:40 | dansmith | reboot, stack works :) | |
| 22:54:00 | dansmith | I don't know what still works for teardown after the systemd thing | |
| 22:54:28 | cfriesen | mriedem: do you know where the cost is? cuz that seems higher than it should be. | |
| 22:58:28 | jaypipes | edleafe: hopefully I explained the whole server-side PUT /allocations/{consumer_uuid} API microversion mismatch scenario properly in that last comment on the alternate hosts spec. | |
| 23:00:53 | jaypipes | efried: we get all your questions answered on the nested resource providers spec? | |
| 23:04:06 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 23:11:03 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add alternate hosts https://review.openstack.org/486215 | |
| 23:11:03 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | |
| 23:12:13 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 23:19:20 | openstackgerrit | Merged openstack/nova master: Deprecate idle_timeout in api_database https://review.openstack.org/507006 | |
| 23:24:11 | mriedem | cfriesen: this is a devstack env fwiw | |
| 23:29:53 | edleafe | jaypipes: "++ let's ship it. :)" doesn't explain that much :) | |
| 23:30:25 | jaypipes | edleafe: sorry, meant the selection object spec | |
| 23:31:05 | edleafe | jaypipes: just funnin' ya | |
| #openstack-nova - 2017-09-26 | |||
| 00:04:02 | mriedem | gah | |
| 00:04:09 | mriedem | superconductor foils me again | |
| 00:04:25 | mriedem | nova-manage db archive_deleted_rows in devstack uses nova.conf by default, which is cell0 in devstack | |
| 00:07:01 | mriedem | http://logs.openstack.org/58/506458/2/check/gate-tempest-dsvm-neutron-nova-next-full-ubuntu-xenial-nv/0d589b6/logs/devstack-gate-post_test_hook.txt.gz | |
| 00:07:02 | mriedem | yar | |
| 00:08:21 | mriedem | we could make nova-manage db archive_deleted_rows use the api database to get the cells... | |
| 00:12:08 | mriedem | https://bugs.launchpad.net/nova/+bug/1719487 | |
| 00:12:09 | openstack | Launchpad bug 1719487 in OpenStack Compute (nova) "nova-manage db archive_deleted_rows is not multi-cell aware" [Medium,Triaged] | |
| 00:24:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: For map_instances argument destination is not defined https://review.openstack.org/502236 | |
| 00:26:02 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix --max-count handling for nova-manage cell_v2 map_instances https://review.openstack.org/502236 | |
| 01:20:36 | openstackgerrit | jichenjc proposed openstack/nova master: propagate OSError to MigrationPreCheckError https://review.openstack.org/469779 | |
| 01:31:16 | openstackgerrit | jichenjc proposed openstack/nova master: check query param for server groups function https://review.openstack.org/500347 | |
| 01:44:35 | openstackgerrit | jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458 | |
| 01:57:37 | yufei | @alex_xu, could you please help take a look at this patch when you are free? a small patch which remove re-auth logic for service-users when nova call ironic. https://review.openstack.org/#/c/502382/ | |
| 02:25:59 | openstackgerrit | wanghao proposed openstack/nova master: Set min_disk in the image meta based on the root volume's size https://review.openstack.org/407397 | |
| 03:05:48 | openstackgerrit | Steven Webster proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853 | |
| 03:06:17 | openstackgerrit | Steven Webster proposed openstack/nova master: Race condition between audit and migrate/resize revert https://review.openstack.org/400995 | |
| 03:26:59 | gmann | alex_xu: do you remember why limit and marker are single_param in this - https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/hypervisors.py#L22 | |
| 03:29:05 | gmann | mriedem: alex_xu: because it was added in microversion ? because we agreed on keep accepting the multi param for 'limit' and other pagination query param and let controller code to fetch the one(which is second one as default due to dict) | |
| 03:39:15 | gmann | mriedem: alex_xu similar for additionalProperties, it is False. i think those are correct as those query are introduced with microversion. I will keep 2.33 schema also same way to avoid any inconsistency between 2.33 and 2.53 | |
| 03:56:26 | wlfightup | hello | |
| 03:57:52 | wlfightup | if resize same host, the schuduler don't filter delta, only filter new flavor. i think that it is don't right | |
| 04:11:39 | wlfightup | anyone there ? | |
| 04:32:57 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Implement query param schema for GET hypervisor(2.33) https://review.openstack.org/507344 | |
| 04:33:09 | gmann | mriedem: alex_xu this one for 2.33- https://review.openstack.org/507344 | |
| 05:02:53 | alex_xu | gmann: we should keep backward-compatible in 2.33? | |
| 05:15:19 | gmann | alex_xu: but there was no query param before 2.33 so making those as single value and no additional property should be fine? | |
| 05:15:44 | gmann | alex_xu: only issue was, we did not restrict and documented about that while doing 2.33 | |
| 05:16:53 | gmann | alex_xu: which can break people if someone using it with multi param and additional params >2.33 (<2.53 as it restrict those with schema) | |
| 05:19:31 | gmann | alex_xu: i think i agree to keep those same as it is behaving currently. | |
| 05:23:52 | alex_xu | gmann: yea, keep the same behaving currently | |
| 06:02:27 | wlfightup | can care me? | |
| 06:45:52 | openstackgerrit | Merged openstack/nova master: Live Migration sequence diagram https://review.openstack.org/506370 | |
| 07:19:55 | openstackgerrit | Alex Xu proposed openstack/nova-specs master: Add trait support in the allocation candidates API https://review.openstack.org/497713 | |
| 07:24:58 | ratailor_ | help | |
| 07:26:33 | ratailor | anybody expert in openstackclient, I want to know why the default version is taken as minimum api microversion 2.1, whereas in novaclient it is maximum the client can support. Is there any design consensus about it. | |
| 07:31:57 | ratailor | amotoki, ^^ | |