Posted	Nick	Remark
#openstack-nova - 2018-02-08
22:41:55	openstackgerrit	melanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/540258
22:49:10	openstackgerrit	Arvind Nadendla proposed openstack/nova-specs master: Support traits in Glance https://review.openstack.org/541507
22:54:45	bkero-	Hey all, can anyone recall seeing a nova-libvirt bug in 2.5 about logfile chardev not being supported?
22:59:30	cfriesen	this is maybe a weird question, but is the ec2-id returned by the instance metadata server (in the format "i-00000004") expected to match the "instance-xxxx" value as returned in the OS-EXT-SRV-ATTR:instance_name field of "nova show"?
23:02:42	openstackgerrit	Jianghua Wang proposed openstack/nova master: VGPU: Modify the example of vgpu white_list set https://review.openstack.org/539183
23:22:22	smcginnis	mriedem: Hows that RC coming along? :)
23:23:47	melwitt	smcginnis: he's away atm, the release patch is up https://review.openstack.org/#/c/542459/ and depends-on the things we have left, which I'm going to finish reviewing in a few
23:23:58	melwitt	he'll be back later, he said
23:24:10	smcginnis	melwitt: OK, so just a couple things to push through yet?
23:24:34	melwitt	yeah, one is in the gate already, the other's need a +W and I'm gonna review those now
23:24:55	mriedem	i'm back
23:25:26	mriedem	bkero-: sounds familiar but not sure, might have been something markus zoeller worked on in ocata?
23:25:41	mriedem	bkero-: https://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/libvirt-virtlogd.html ?
23:25:45	smcginnis	melwitt: Thanks!
23:30:06	openstackgerrit	Matt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510
23:34:11	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
23:38:09	melwitt	mriedem: do you know why this test is removed instead of setting up the ComputeV4Proxy like the others? https://review.openstack.org/#/c/541005/6/nova/tests/unit/compute/test_compute_mgr.py@a6834
23:38:52	mriedem	because _rollback_live_migration doesn't do that thing anymore
23:39:00	mriedem	and _do_live_migration calls _rollback_live_migration
23:39:13	mriedem	and _do_live_migration already has the non-dict form of migrate_data by the time it calls _rollback_live_migration
23:39:17	mriedem	i think that's why anyway
23:39:48	mriedem	https://review.openstack.org/#/c/541005/5/nova/compute/manager.py@6394
23:41:50	melwitt	yeah, I mean, it looks like the other "handles_dict" tests were adapted to use the _ComputeV4Proxy because the V4Proxy version has to do the dict compat thing, right?
23:42:59	mriedem	yeah, but there isn't a compute v4 proxy for that rollback method
23:43:07	mriedem	the entry point / proxy method is live_migration
23:43:17	melwitt	oh, I see. okay
23:43:20	mriedem	which does the massageroo
23:52:19	gmann_	mriedem: yea it seems both running. it has been replaced with tempest-full in integrated job(and fix for stable branches etc)
23:52:46	ianw	in case anyone finds my comment in logs about aarch64 config-drive ... yes it does work, but ensure you've got all the right kernel cdrom drivers, which the default xenial kernel may not
23:53:02	mriedem	gmann_: https://review.openstack.org/542484
23:53:04	mriedem	think i figured it out
23:53:06	gmann_	mriedem: legacy-tempest-dsvm-neutron-ful should be running on stable branches only but it seem we have it running on master too in many place
23:53:19	mriedem	ianw: hrw is the aarch64 expert around these parts
23:53:35	mriedem	https://review.openstack.org/#/c/541728/
23:54:29	mriedem	ianw: maybe you can provide some details here? https://review.openstack.org/#/c/541728/9/doc/source/user/support-matrix.ini@872
23:54:39	mriedem	hrw is working on documenting aarch64+libvirt support in the feature matrix
23:54:46	mriedem	so that would be good info since we can put notes in these doc entries
23:54:49	ianw	mriedem: yes, wouldn't have got nearly so far without his help ;)
23:54:50	gmann_	mriedem: yea but that should not needed i guess as it is handled in integrated-gate template - http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/zuul.d/zuul-legacy-project-templates.yaml#n107
23:55:11	mriedem	gmann_: see the response from corvus in -infra,
23:55:23	mriedem	apparently b/c nova re-defines it in project-config w/o the branch specifier, it messes that up
23:55:57	gmann_	mriedem: yes, i think we should remove it all from project-config (nova, tempest, swift etc)
23:56:13	cfriesen	mriedem: that earlier issue with libvirt and specifying the disks to migrate....I think it was from before we rebased to 16.0.4. So likely has gone away now.
23:56:19	gmann_	mriedem: anyways i ll check that. thanks for this
23:57:18	ianw	mriedem: oh cool! thanks for pointing to that. yeah, we're working on bringing up dib/nodepool/infra etc on the linaro cloud. all things considered, it's going well i'd say! soon you should be able to easily add aarch64 testing (modulo cloud capacity ... it will be a soft start)
23:57:21	mriedem	cfriesen: ok
23:57:34	mriedem	ianw: that would be nice
23:57:41	mriedem	given the work he's been doing on fixing bugs in the libvirt driver
#openstack-nova - 2018-02-09
00:03:39	ianw	yeah, a couple of of things seems to get magically better when you use the arm64 hwe kernels (4.13) v the default (4.4) ... not surprising given the pace of change in the area
00:04:14	takashin	mriedem: Would you remove -2 in https://review.openstack.org/#/c/513721/ ?
00:04:26	takashin	mriedem: A patch for stable/pike has been merged. ( https://review.openstack.org/#/c/513717/ )
00:05:27	takashin	mriedem: Are you around?
00:06:30	mriedem	takashin: changed to a -1
00:07:43	takashin	mriedem: Thanks. I will fix it.
00:07:52	gmann_	mriedem: commented, due to irrelevant-files you want to keep that job explicitly in nove defined job?
00:08:56	openstackgerrit	Matt Riedemann proposed openstack/nova master: unquiesce instance on volume snapshot failure https://review.openstack.org/519464
00:08:57	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add the ability to get absolute limits from Cinder https://review.openstack.org/520122
00:08:57	openstackgerrit	Matt Riedemann proposed openstack/nova master: Check quota before creating volume snapshots https://review.openstack.org/520158
00:09:13	mriedem	gmann_: yes
00:10:52	gmann_	mriedem: +1.
00:10:58	gmann_	mriedem: we need to define the same for tempest-full too to avoid it running for those. also i still see neutron-grenade for doc/api-ref changes .i thought i fixed that. let me check again
00:18:00	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Fix the names on some specs https://review.openstack.org/542504
00:18:01	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Move queens implemented specs https://review.openstack.org/542505
00:24:26	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Fix the names on some specs https://review.openstack.org/542504
00:24:27	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Move queens implemented specs https://review.openstack.org/542505
00:30:54	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Move queens implemented specs https://review.openstack.org/542505
00:32:55	openstackgerrit	Matt Riedemann proposed openstack/nova master: doc: fix the link for the evacuate cli https://review.openstack.org/542508
00:44:11	openstackgerrit	Merged openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161
00:44:20	openstackgerrit	Matt Riedemann proposed openstack/nova master: doc: mention that --on-shared-storage is not needed with nova evacuate https://review.openstack.org/542512
01:17:57	openstackgerrit	peiyu proposed openstack/nova master: replace if by elif https://review.openstack.org/542520
01:24:31	jianghuaw	bauzas, are you around?
01:25:36	jianghuaw	bauzas, sorry. Too late for you:-)
01:29:41	openstackgerrit	Merged openstack/nova master: Bump compute RPC API to version 5.0 https://review.openstack.org/541005
01:31:42	mnaser	is there a list of topics being discussed at the ptg for nova?
01:34:45	takashin	mnaser: https://etherpad.openstack.org/p/nova-ptg-rocky
01:34:58	mnaser	takashin: thanks!
01:38:15	mnaser	added an item of something we just ran into :>
01:43:56	openstackgerrit	Merged openstack/nova master: Compute RPC client bump to 5.0 https://review.openstack.org/541035
01:44:05	openstackgerrit	Merged openstack/nova master: Clean up reservations in migrate_task call path https://review.openstack.org/542409
01:59:32	gmann_	alex_xu: mriedem johnthetubaguy sdague FYI, i added a API topic to discuss in PTG, granular policy : https://etherpad.openstack.org/p/nova-ptg-rocky L224
02:02:09	openstackgerrit	gaobin proposed openstack/nova master: Correction revised as Show https://review.openstack.org/542539
02:12:21	openstackgerrit	zhongshengping proposed openstack/nova master: Refine waiting for vif plug events during _hard_reboot https://review.openstack.org/541442
02:16:52	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Fix missing blueprint URLs https://review.openstack.org/542543
02:24:26	alex_xu	gmann_: thanks
02:58:05	openstackgerrit	peiyu proposed openstack/nova master: replace if by elif https://review.openstack.org/542520
03:11:38	openstackgerrit	Merged openstack/nova master: VGPU: Modify the example of vgpu white_list set https://review.openstack.org/539183
03:18:03	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Fix missing blueprint URLs https://review.openstack.org/542543
04:54:52	openstackgerrit	Merged openstack/nova stable/pike: Fix wrong link for "Manage Flavors" in CPU topologies doc https://review.openstack.org/542270
04:54:59	openstackgerrit	Merged openstack/nova stable/pike: Add release note for Aggregate[Core|Ram|Disk]Filter change https://review.openstack.org/542290
05:18:50	openstackgerrit	Merged openstack/nova master: Refine waiting for vif plug events during _hard_reboot https://review.openstack.org/541442
06:22:54	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/541561
07:04:35	openstackgerrit	sunjiazz proposed openstack/nova master: Snapshot the volume-backed instance without min_disk in image metadate. https://review.openstack.org/542646
07:15:29	openstackgerrit	OpenStack Release Bot proposed openstack/nova master: Update reno for stable/queens https://review.openstack.org/542658
07:36:19	openstackgerrit	jichenjc proposed openstack/nova master: Avoid live mgirate to same host https://review.openstack.org/542689
08:05:29	openstackgerrit	jichenjc proposed openstack/nova master: Remove MigrationPreCheckClientException https://review.openstack.org/542704
08:25:53	openstackgerrit	Ameed Ashour proposed openstack/osc-placement master: Change documentation theme https://review.openstack.org/542378
08:46:20	bauzas	good morning Novaers
08:47:05	gmann_	morning
08:47:32	gmann_	FYI. if you see neutron-grenade job running on api-ref change the this is fix - https://review.openstack.org/#/c/542705/
08:49:01	bauzas	gmann_: wrong chan? :p I can see it's for the neutron repo :)

Earlier   Later