Posted	Nick	Remark
#openstack-nova - 2018-09-28
20:46:30	openstackgerrit	Merged openstack/nova stable/queens: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/603737
21:14:39	openstackgerrit	Merged openstack/nova master: nova-manage - fix online_data_migrations counts https://review.openstack.org/605329
21:15:24	openstackgerrit	Merged openstack/nova stable/queens: cover migration cases with functional tests https://review.openstack.org/602408
22:10:29	openstackgerrit	melanie witt proposed openstack/nova master: Replace usage of get_legacy_facade() with get_engine() https://review.openstack.org/606213
22:38:26	openstackgerrit	melanie witt proposed openstack/nova master: Use an alternative class 'MoxStubout' https://review.openstack.org/579482
#openstack-nova - 2018-09-29
00:15:42	openstackgerrit	Merged openstack/nova master: Follow up for Iba230201803ef3d33bccaaf83eb10453eea43f20 https://review.openstack.org/605653
01:26:38	openstackgerrit	Merged openstack/nova master: Follow up for Ie991d4b53e9bb5e7ec26da99219178ab7695abf6 https://review.openstack.org/605658
01:51:33	openstackgerrit	Merged openstack/nova stable/rocky: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/605260
02:13:40	openstackgerrit	Merged openstack/nova stable/rocky: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/605261
03:20:24	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
06:49:45	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
07:54:22	openstackgerrit	Merged openstack/nova stable/queens: Update RequestSpec.flavor on resize_revert https://review.openstack.org/602409
08:47:45	openstackgerrit	Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support ``volume_type`` https://review.openstack.org/605573
09:52:02	openstackgerrit	Filippo Inzaghi proposed openstack/os-traits master: Removed older version of python added 3.5 https://review.openstack.org/606370
11:43:29	openstackgerrit	Brin Zhang proposed openstack/nova master: Add compute API version for when a ``volume_type`` is requested https://review.openstack.org/605573
12:08:20	openstackgerrit	Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
12:17:38	openstackgerrit	Brin Zhang proposed openstack/nova master: Add compute API version for when a ``volume_type`` is requested https://review.openstack.org/605573
12:27:41	openstackgerrit	Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
20:23:27	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix neutron-tempest-linuxbridge irrelevant-files https://review.openstack.org/606616
#openstack-nova - 2018-09-30
00:12:00	openstackgerrit	Merged openstack/nova master: Fix stacktraces with redis caching backend https://review.openstack.org/605748
02:48:13	openstackgerrit	Brin Zhang proposed openstack/nova master: Add API samples test for volume_type https://review.openstack.org/606758
02:54:53	openstackgerrit	Brin Zhang proposed openstack/nova master: Add API samples test for ``volume_type`` https://review.openstack.org/606758
05:55:13	openstackgerrit	Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
06:04:32	openstackgerrit	Brin Zhang proposed openstack/nova master: Add API samples test for ``volume_type`` https://review.openstack.org/606758
07:54:08	openstackgerrit	Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
11:48:37	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge used_limits extension response into limit view builder https://review.openstack.org/606031
12:00:59	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge used_limits extension response into limit view builder https://review.openstack.org/606031
13:38:26	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge image_size extension response into image view builder https://review.openstack.org/606845
15:25:49	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add hide server address tests in test_serversV21.py https://review.openstack.org/605435
20:50:04	openstackgerrit	Merged openstack/nova master: Use INFO for logging no allocation candidates https://review.openstack.org/605765
23:08:06	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Update the CLI reference https://review.openstack.org/606871
#openstack-nova - 2018-10-01
00:43:49	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628
06:24:07	openstackgerrit	Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
06:45:54	openstackgerrit	Sam Morrison proposed openstack/python-novaclient master: Fix up userdata argument to rebuild. https://review.openstack.org/605341
07:01:30	bauzas	good morning Nova
07:50:54	openstackgerrit	Christoph Manns proposed openstack/nova stable/rocky: Fix stacktraces with redis caching backend https://review.openstack.org/606895
07:52:09	openstackgerrit	Christoph Manns proposed openstack/nova stable/queens: Fix stacktraces with redis caching backend https://review.openstack.org/606896
07:53:46	openstackgerrit	Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
09:07:47	openstackgerrit	Matthew Booth proposed openstack/nova master: Raise error on timeout in wait_for_versioned_notifications https://review.openstack.org/604859
09:08:06	openstackgerrit	Matthew Booth proposed openstack/nova master: Add regression test for bug 1550919 https://review.openstack.org/591733
09:08:06	openstack	bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)
09:08:06	openstackgerrit	Matthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate https://review.openstack.org/578846
09:11:58	kashyap	testone nova.tests.unit.virt.libvirt.test_driver.LibvirtConnTestCase.test_next_min_qemu_version_ok
09:12:47	kashyap	Err, disregard that
09:13:14	openstackgerrit	Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212
09:13:47	ttsiouts	johnthetubaguy: I tried to refresh the spec ^
10:03:21	sean-k-mooney	o/
10:04:17	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Update the contributor guide https://review.openstack.org/606926
10:05:40	sean-k-mooney	stephenfin: when you get a chance can you run your eyes over https://review.openstack.org/#/c/604992/3. looks like an easy addtion to me.
10:05:48	stephenfin	ack
10:06:23	sean-k-mooney	stephenfin: i think we need to port other vif types from kuyr-kubernetes too but they can be in different patches
10:35:08	sean-k-mooney	ralonsoh: hi are you around today?
10:35:47	ralonsoh	sean-k-mooney: very well, fighting with DVR and FIP (and SRIOV)
10:36:52	stephenfin	sean-k-mooney: Also, "i prefer explcit else clauses"
10:37:00	stephenfin	sean-k-mooney: Just wanted you to know you're wrong
10:37:04	stephenfin	sean-k-mooney: That is all.
10:37:05	stephenfin	:)
10:37:17	sean-k-mooney	ralonsoh: if you bussy i can respin https://review.openstack.org/#/c/605422/2
10:38:05	ralonsoh	sean-k-mooney: sure, I'll do this now
10:38:40	sean-k-mooney	stephenfin: hehe well we can agree to disagree on that. it can help prevent some types of errors when you are maintaing the fucntion in the future but also consumes indentation space so is not allways desireable
10:48:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Rewrite the console doc https://review.openstack.org/606148
10:48:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Allow 'nova-xvpvncproxy' to be called with CLI args https://review.openstack.org/606929
10:51:27	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Remove IPTools deprecated implementation https://review.openstack.org/605422
10:51:45	ralonsoh	sean-k-mooney: done
10:55:27	sean-k-mooney	ralonsoh: thanks :)
11:11:38	openstackgerrit	Theodoros Tsioutsias proposed openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.openstack.org/554218
11:15:21	openstackgerrit	Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212
11:28:36	openstackgerrit	Merged openstack/nova stable/rocky: Use INFO for logging no allocation candidates https://review.openstack.org/605771
11:48:27	openstackgerrit	Merged openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405
12:05:59	kashyap	stephenfin: On that version bump thing for libvirt, I just looked at "what's involved" to clean up
12:06:15	kashyap	stephenfin: I need to address at least 10 unit tests. Will get to it this week. Currently buried in something a bit more urgent
12:30:43	stephenfin	kashyap: Yup, it's low priority and can be done anytime this cycle. No rush!
12:31:56	kashyap	Yep.
12:32:40	kashyap	Then, I might even wait until I'm through all the CPU modelling stuff that I'm looking into (until 27 Oct)
12:40:36	mdbooth	bauzas: LM failure on https://review.openstack.org/#/c/604400/
12:41:05	mdbooth	So, the exact failure is unlikely to be relevant. Chances are that this is just the first thing which touched that host, and the host was broken by the evacuate test we just ran.
12:41:35	bauzas	mdbooth: so, you got a failure for req-dd88c142-4710-49d0-b6e9-594c6a1e89f6 (live-migrate action) because of http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/screen-n-sch.txt.gz#_Sep_28_20_39_55_712058
12:41:52	mdbooth	bauzas: Yes
12:42:01	bauzas	rather http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/screen-n-sch.txt.gz#_Sep_28_20_39_55_711740
12:42:07	mdbooth	So, the question is: why does the computefilter think the host is down?
12:42:26	bauzas	mdbooth: because the db record is old
12:42:45	bauzas	mdbooth: I mean the updated_at filed
12:42:47	bauzas	field*
12:42:52	mdbooth	bauzas: Right, but in the compute logs I see the host joining the service group within the last 60 secs
12:42:58	bauzas	https://github.com/openstack/nova/blob/master/nova/scheduler/filters/compute_filter.py#L44
12:43:12	mdbooth	However, I note that it doesn't update the record at that point, just schedules it to be updated in 5 seconds
12:43:40	mdbooth	So I then got to wondering why it doesn't seem to have been updated 5 seconds later.
12:44:00	mdbooth	I was wondering if perhaps a long running init_host might be related?
12:44:16	mdbooth	Because that's feasible following the evacuate test
12:45:01	bauzas	https://github.com/openstack/nova/blob/master/nova/servicegroup/drivers/db.py#L73
12:45:09	mdbooth	However, threadgroup.add_timer seems to be unconditional
12:45:21	bauzas	and https://github.com/openstack/nova/blob/master/nova/servicegroup/drivers/db.py#L60
12:45:59	mdbooth	bauzas: Right, so it hasn't updated its record within the last 60 seconds
12:46:20	bauzas	just checking, it's 60sec http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/screen-n-sch.txt.gz#_Sep_28_20_25_02_439642
12:46:27	bauzas	yup
12:46:32	mdbooth	But given that it joined the servicegroup http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/subnode-2/screen-n-cpu.txt.gz?#_Sep_28_20_39_12_277355
12:46:38	bauzas	now, I'm checking the compute logs
12:46:53	mdbooth	I can't see why not.
12:47:08	bauzas	it's a periodic task IIRC
12:47:41	mdbooth	Looks like it's a threadgroup timer

Earlier   Later