Posted	Nick	Remark
#openstack-nova - 2019-08-20
21:12:07	my_nd	thanks mriedem
21:13:13	mriedem	efried: postgresql shouldn't even be running
21:13:35	mriedem	i don't see it in https://a6b55b727208d469fce9-a76e08006e67cd388b33580b35bd0960.ssl.cf2.rackcdn.com/676684/7/check/nova-grenade-live-migration/8626716/logs/screen-peakmem_tracker.txt.gz anyway
21:13:44	mriedem	mysqld is in there
21:16:23	mriedem	unfortunately peakmemtracker seems to die before we start seeing segfaults
21:16:40	mriedem	top memory usage is unsurprisingly mysqld, neutron, and nova (api and compute)
21:18:01	efried	I can't make that logstash query work, because it split lines on me. Can you paste just the text from the query box?
21:18:23	mriedem	message:"Unexpected exception in API method: ConnectFailure: Unable to establish connection to" AND message:"v2.0/ports?device_id" AND message:"('Connection aborted.', BadStatusLine('No status line received - the server has closed the connection',))" AND tags:"screen-n-api.txt"
21:18:29	efried	or you can just tell me: are there hits from the earlier patch sets?
21:18:55	mriedem	6 and 7 yes
21:19:18	efried	mm
21:19:37	mriedem	nothing really changed much since PS5 though...
21:19:46	efried	those are all from nova-grenade-live-migration.
21:20:03	efried	how did the other grenades fail?
21:23:13	mriedem	similar, but in one case the grenade script calling neutron api directly rather than proxying through nova
21:23:14	mriedem	2019-08-20 15:42:11.965 | + /opt/stack/new/grenade/projects/70_cinder/resources.sh:destroy:265 : openstack floating ip unset --port 172.24.5.75 2019-08-20 15:42:15.779 | Unable to establish connection to http://104.239.175.129:9696/v2.0/floatingips/172.24.5.75: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))
21:23:36	mriedem	Aug 20 15:42:15.153161 ubuntu-bionic-rax-iad-0010301032 neutron-server[11166]: INFO oslo_service.service [-] Child 24291 killed by signal 11
21:24:28	mriedem	Aug 20 15:42:15 ubuntu-bionic-rax-iad-0010301032 kernel: /usr/bin/python[22305]: segfault at a9 ip 0000000000585794 sp 00007ffd8b90c5b8 error 4 in python3.6[400000+3bf000]
21:32:39	efried	Is there a unifying logstash search to rule them all?
21:35:44	efried	message:"Unable to establish connection to" AND tags:"screen-n-api.txt"
21:35:44	efried	with 7d
21:35:44	efried	gives me 64 hits on three different grenade jobs on both of those patches and no others
21:36:00	efried	(that one and its now-abandoned successor)
21:36:24	efried	That would be enough for me to say, "sorry, this -binary thing is no good".
21:47:25	mriedem	yeah there were hits on the now abandoned patch that was on top of the -binary one too
21:47:26	mriedem	super weird
21:52:02	openstackgerrit	Matt Riedemann proposed openstack/nova master: Delete InstanceMapping in conductor if BuildRequest is already deleted https://review.opendev.org/666438
21:57:36	mriedem	dansmith: would be nice to get this in https://review.opendev.org/#/c/673620/ so we can drop the service uuid online data migration in U
21:59:38	dansmith	mriedem: hmm, we've had other blocker migrations since havana that didn't get added to that special list
22:00:41	mriedem	i know, and they don't do anything
22:01:02	dansmith	why do you need to add yours there then?
22:01:22	mriedem	https://review.opendev.org/#/c/673620/1/nova/tests/unit/db/test_migrations.py@730
22:01:27	mriedem	because it's cleaner?
22:01:46	mriedem	my guess is that skip list came after the older blocker migrations
22:01:55	dansmith	I don't think so,
22:02:00	dansmith	I think I added that a long time ago
22:03:31	mriedem	ok yeah https://review.opendev.org/#/c/122162/ vs https://review.opendev.org/#/c/174480/
22:03:36	mriedem	but having said that, i'm not sure why it matters
22:03:49	mriedem	it's just making sure there is a method for the migration, but the method just passes so...
22:03:53	mriedem	the actual test is in another module
22:04:09	dansmith	well, it's more just the principle of the case where nothing should be added to those exclusion lists in the future, and I don't think you need to
22:04:36	mriedem	if you're +2 besides that will you fast approve if i make that change?
22:04:52	dansmith	yeah
22:05:11	dansmith	sorry
22:10:33	mriedem	np, i have to rebase anyway
22:14:23	dansmith	mriedem: you meant soon right? :)
22:15:40	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add blocker migration for completing services.uuid migration https://review.opendev.org/673620
22:15:42	mriedem	thar she blar
22:18:06	dansmith	got it
23:04:34	mriedem	thanks
23:05:29	sean-k-mooney	mriedem: ack, thanks for updating the deprecation patch. i had planned to throw up a [WIP:U] patch to actully delete the thing in u but that is still future
23:05:42	sean-k-mooney	just missed him hehe
23:41:12	openstackgerrit	Dustin Cowles proposed openstack/nova master: Public method to retrieve custom resource providers https://review.opendev.org/676029
23:41:13	openstackgerrit	Dustin Cowles proposed openstack/nova master: WIP: Load the custom resource providers to resource tracker https://review.opendev.org/676522
#openstack-nova - 2019-08-21
01:15:27	openstackgerrit	sean mooney proposed openstack/nova stable/pike: lxc: make use of filter python3 compatible https://review.opendev.org/676502
01:43:30	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix missing rule description in HACKING.rst https://review.opendev.org/677589
02:21:05	openstackgerrit	Merged openstack/nova master: [Trivial]Remove unused helper filter_and_format_resource_metadata https://review.opendev.org/676038
03:00:45	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add a hacking rule for non-existent assertions https://review.opendev.org/677596
03:26:31	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969
04:44:01	openstackgerrit	melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
04:44:51	openstackgerrit	melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
05:23:07	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add a hacking rule for non-existent assertions https://review.opendev.org/677596
05:46:17	openstackgerrit	Merged openstack/nova master: doc: Trivial fixes to API version history https://review.opendev.org/649561
05:46:49	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add a hacking rule for useless assertions https://review.opendev.org/677610
05:50:17	openstackgerrit	Merged openstack/nova master: Add blocker migration for completing services.uuid migration https://review.opendev.org/673620
07:02:52	alex_xu	efried: sean-k-mooney dansmith, here is the refined version about using db persistent https://etherpad.openstack.org/p/vpmems-non-virt-driver-specific-new. I also sent email to summary the pros/cons of two ways http://lists.openstack.org/pipermail/openstack-discuss/2019-August/008625.html
08:07:11	openstackgerrit	Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
09:00:15	openstackgerrit	Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.77: Support Specifying AZ to unshelve https://review.opendev.org/665136
09:56:27	openstackgerrit	Merged openstack/nova master: [Trivial]Remove unused helper _get_instances_by_filters https://review.opendev.org/676036
10:34:32	aspiers	alex_xu: I will have another PS for you very soon
10:34:41	aspiers	alex_xu: I'm adding the missing tests
11:15:31	openstackgerrit	Merged openstack/nova master: Libvirt: add support for vPMU configuration. https://review.opendev.org/671338
11:51:55	alex_xu	aspiers: thanks a lot
11:52:10	aspiers	alex_xu: just finished now, submitting ...
11:52:58	openstackgerrit	Adam Spiers proposed openstack/nova master: Split fake host capabilities into reusable variables https://review.opendev.org/677710
11:53:38	openstackgerrit	Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151
11:53:50	mdbooth	Heads up: the libvirt tpool proxy doesn't work reliably: https://bugs.launchpad.net/nova/+bug/1840912 Potentially the cause of a bunch of interesting behaviours
11:53:51	openstack	Launchpad bug 1840912 in OpenStack Compute (nova) "libvirt calls aren't reliably using tpool.Proxy" [Undecided,New]
11:53:52	openstackgerrit	Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
11:55:49	alex_xu	aspiers: I will try another around tomorrow
11:56:16	aspiers	alex_xu: thanks. I just updated the test comment and added some simple tests to test_host.py
11:56:22	aspiers	alex_xu: so it should be easy to review
11:58:20	alex_xu	aspiers: cool
12:27:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Rewrite host aggregate, availability zone docs https://review.opendev.org/667133
12:28:45	stephenfin	alex_xu: Think you could send this on its way? https://review.opendev.org/#/c/675570/
12:36:38	openstackgerrit	Adam Spiers proposed openstack/nova master: Split fake host capabilities into reusable variables https://review.opendev.org/677710
12:36:53	openstackgerrit	Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151
12:37:05	openstackgerrit	Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964
12:37:06	openstackgerrit	Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
12:37:22	openstackgerrit	Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
12:37:40	openstackgerrit	Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
12:37:51	openstackgerrit	Adam Spiers proposed openstack/nova master: Add and to config.py https://review.opendev.org/636318
12:38:03	openstackgerrit	Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
12:38:21	openstackgerrit	Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
12:40:54	shilpasd	stephenfin: Hi, small doubt, at https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/versions.py#L43
12:41:12	shilpasd	for multiple choices reply, links getting printed
12:41:30	shilpasd	and if we given wrong request path, that getting added to response
12:41:41	stephenfin	shilpasd: Could you give me an example?
12:41:46	stephenfin	Expected vs. actual
12:42:00	shilpasd	actual: "links": [{"href": "http://10.232.48.206/compute/v2.1/compute/v4.1/servers/detail", "rel": "self"}]

Earlier   Later