Posted	Nick	Remark
#openstack-nova - 2018-08-10
18:31:41	SteelyDan	it's not a straight revert regardless, so I guess I'm not sure I see the point of putting things back like the bandwidth thing as that pattern then got copied in the params stuff afterwards in some places
18:31:57	SteelyDan	and like the virtapi thing makes no difference
18:32:04	SteelyDan	and the networkinfo
18:32:13	SteelyDan	but the things to collect the events are probably dead now
18:32:30	SteelyDan	not sure I'm going to get to this before I have to relocate for the day but I'll try
18:32:35	melwitt	okay, it just seems like test_live_migration_main and test_live_migration_ovs_vif are duplicates of one another, essentially
18:33:18	SteelyDan	probably yeah
18:33:46	melwitt	since the networkinfo part is taken out, that was the difference between them, I thought. ok, I can add a follow up to clean that part up since you've got to run soon
18:34:19	melwitt	since it's just extra duplicated unit test coverage
18:34:25	SteelyDan	I'll take a shot and let you know if I don't push something up
18:34:33	melwitt	ok
18:34:41	SteelyDan	yeah, the second test being a duplicate now makes sense
18:36:53	openstackgerrit	Chris Dent proposed openstack/nova stable/ocata: [placement] Retry allocation writes server side https://review.openstack.org/591042
18:38:34	SteelyDan	maybe that will all just come out easy, lemme stash and try
18:45:43	kosamara	efried: Hi! I've asked you some more questions on your spec. In the meantime I've published my take for review and expect to change it depending on your answers.
18:46:16	kosamara	efried: https://review.openstack.org/#/c/591037/
18:46:17	fried_rolls	kosamara: Neat, this one: https://review.openstack.org/591037 ?
18:46:20	fried_rolls	okay, cool.
18:48:06	openstackgerrit	Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538
19:18:46	openstackgerrit	Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538
19:25:30	openstackgerrit	Matthew Edmonds proposed openstack/nova master: Make Xen code py3-compatible https://review.openstack.org/591061
19:43:50	openstackgerrit	Eric Fried proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070
19:59:40	openstackgerrit	Merged openstack/nova master: Add additional info to resource provider aggregates update API https://review.openstack.org/590243
20:04:23	openstackgerrit	Matthew Edmonds proposed openstack/nova master: Make Xen code py3-compatible https://review.openstack.org/591061
20:11:15	openstackgerrit	Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
20:11:42	openstackgerrit	Eric Fried proposed openstack/nova master: DNM: demo https://review.openstack.org/591073
20:12:57	leakypipes	fried_rice: one sec. pulling your test patch onto the bottom.
20:13:26	fried_rice	leakypipes: Cool beans. I just posted my comments, as well as --^ another patch demoing some of them
20:13:30	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388
20:13:31	openstackgerrit	Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
20:13:32	openstackgerrit	Jay Pipes proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150
20:13:33	openstackgerrit	Jay Pipes proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070
20:13:36	leakypipes	fried_rice: et voila ^
20:14:36	fried_rice	leakypipes: You put it on top
20:15:19	fried_rice	leakypipes: You also reordered the other test patch to the top.
20:17:32	fried_rice	leakypipes: For ease of backportage, perhaps we should squash both test patches into Gigantor.
20:19:58	openstackgerrit	Jay Pipes proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150
20:19:59	openstackgerrit	Jay Pipes proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070
20:20:00	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388
20:20:01	openstackgerrit	Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
20:20:11	leakypipes	fried_rice: sorry bout that. fixed.
20:20:27	fried_rice	leakypipes: Okay, please see comments in PS6 - at least one blocker remaining.
20:20:44	fried_rice	s/at least//
20:35:59	openstackgerrit	melanie witt proposed openstack/nova master: Handle unicode characters in migration params https://review.openstack.org/591080
21:00:45	openstackgerrit	Merged openstack/nova master: Remove patching the mock lib https://review.openstack.org/590709
21:47:23	openstackgerrit	Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
21:48:05	leakypipes	fried_rice: ^^
21:48:22	fried_rice	ack
22:02:12	fried_rice	leakypipes: Just a few minor things.
22:31:00	openstackgerrit	Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
#openstack-nova - 2018-08-11
07:29:45	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/591129
#openstack-nova - 2018-08-12
07:33:06	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/591129
09:59:29	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Trivial fix to remove extra 'will' on microversion doc https://review.openstack.org/591235
14:41:10	tbachman	anyone here used heat templates with nova much? I was trying to use them to create instances in a project other than admin, but I didn’t see a parameter that I could use to pass in the project ID when creating the server
14:41:22	tbachman	hopefully I’m just missing something
14:44:20	tbachman	I guess I should say - it looks like it just uses the project ID that was used for the heat-template
14:45:40	tbachman	would have been nice if this could be a parameter
14:57:02	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667
22:27:58	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
22:28:27	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Making consistent used of GiB and MiB in API ref https://review.openstack.org/559985
22:48:10	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Update contributor guide for Stein https://review.openstack.org/591258
#openstack-nova - 2018-08-13
01:57:50	mriedem	muwahahaha
02:01:08	openstackgerrit	Matt Riedemann proposed openstack/nova master: add zvm into support matrix https://review.openstack.org/532720
03:58:28	openstackgerrit	Merged openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538
03:58:35	openstackgerrit	Merged openstack/nova master: Trivial fix to remove extra 'will' on microversion doc https://review.openstack.org/591235
03:58:42	openstackgerrit	Merged openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150
03:58:49	openstackgerrit	Merged openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070
04:05:20	openstackgerrit	Merged openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388
04:05:26	openstackgerrit	Merged openstack/nova master: Handle unicode characters in migration params https://review.openstack.org/591080
04:05:33	openstackgerrit	Merged openstack/nova master: improve migration script https://review.openstack.org/591030
06:20:06	openstackgerrit	Matt Riedemann proposed openstack/nova stable/rocky: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/591275
06:23:33	prometheanfire	:D
06:23:41	prometheanfire	one step at a time
07:34:35	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345
07:39:47	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629
07:44:12	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851
07:57:37	openstackgerrit	Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Placement model for PCI devices https://review.openstack.org/591037
08:34:16	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP scheduler: check request_spec.instance_group before updating https://review.openstack.org/590733
09:11:13	mriedem	gibi: oh hi
09:11:18	mriedem	we are timezone buddies!
09:12:07	gibi	mriedem: Ohh welcome in this side of the globe!
09:12:46	gibi	mriedem: are you in China?
09:18:06	openstackgerrit	garyk proposed openstack/nova master: Docs: remove invalid link for remote debugging https://review.openstack.org/591316
09:23:07	mriedem	mdbooth: http://lists.openstack.org/pipermail/openstack-dev/2018-August/133198.html - the functional test job runs mysql (and postgresql) "opportunistic" tests
09:23:11	mriedem	with mysql and pg backends
09:24:15	mdbooth	mriedem: So both are configured? IIRC the opportunistic tests run only if the specific DB is actually available.
09:24:34	mdbooth	mriedem: You're up early, btw. In Europe somewhere?
09:25:54	mriedem_	mdbooth: china
09:26:10	mdbooth	Ah, so you've been up for a while :)
09:26:26	mriedem_	https://github.com/openstack/nova/blob/master/tools/test-setup.sh sets up the mysql and pg dbs for the opportunistic tests
09:26:47	mdbooth	mriedem_: Thank you!
09:33:24	mriedem	mdbooth: btw is rajesh aware of the spec for that metadata case sensitivity issue?
09:33:42	mdbooth	mriedem: I was just putting a +1 on it and I noticed your comments. Reading...
09:34:02	mdbooth	ratailor: ^^^ ?
09:35:33	ratailor	mdbooth, I don't know about spec.
09:35:42	gibi	mriedem: do you know if we can remove this in Stein? https://github.com/openstack/nova/blob/16f89fd093217d22530570e8277b561ea79f46ff/nova/virt/ironic/driver.py#L147-L149
09:36:00	mdbooth	At first glance it doesn't sound like a very good idea to me, tbh.
09:36:58	ratailor	mdbooth, let me go through the comments on patch.
09:36:58	mdbooth	So.. we've allowed users to define their own metadata for however long, and now we want to change it just because of a mysql misconfiguration?
09:37:01	mriedem	gibi: maybe...

Earlier   Later