Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-25
21:26:55 mriedem we join those in the db query
21:27:01 mriedem same for services
21:27:13 dansmith mriedem: I shot you that link right?
21:27:18 mriedem https://github.com/openstack/nova/blob/29ef2474d9d6a59ae6859b5b01caad350252c23e/nova/api/openstack/compute/servers.py#L295 should probably be version constrained
21:27:24 mriedem dansmith: you send me something else
21:27:36 mriedem you sent me https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2229-L2229
21:27:48 mriedem but that's in a conditional block
21:28:19 dansmith oh I see, but actually I guess it's because they end up in columns_to_join_new
21:28:43 mriedem https://review.openstack.org/#/c/355558/
21:28:47 mriedem ^ doesn't go far enough
21:29:44 dansmith that's services
21:29:49 mriedem i know,
21:29:53 dansmith but yeah, that won't do anything I think,
21:30:01 dansmith because it's not in the _expected_cols() thing in instance
21:32:10 mriedem it worked because it was in _INSTANCE_OPTIONAL_JOINED_FIELDS
21:32:12 mriedem for services
21:32:19 mriedem fault wasn't before queens
21:32:30 dansmith services is in optional?
21:32:33 mriedem yup
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

Earlier   Later