Posted	Nick	Remark
#openstack-nova - 2018-08-13
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...
09:37:24	mdbooth	That seems user hostile
09:37:33	mriedem	gibi: we have a nova-status upgrade check for the ironic instance / flavor migration to resource classes
09:37:42	ratailor	mdbooth, mriedem in that case, should I abandon the patch ?
09:37:48	mriedem	gibi: i think we have something in the ptg etherpad about this, removing support for granular resources with ironic
09:38:39	gibi	mriedem: Is it L25 in the etherpad?
09:39:17	gibi	mriedem: it seems so. So I'm plugging my pain (https://review.openstack.org/#/c/583667/19/nova/tests/functional/test_servers.py@5135) into that item. thanks
09:40:04	mdbooth	ratailor: It's up to you, but personally I'd see if the bp and spec still have weight behind them, especially given that they're user hostile and your patch shows that fixing the underlying issue isn't hard.
09:40:45	mriedem	ratailor: mdbooth: right it's obviously still an issue,
09:40:53	mriedem	and the spec was merged at some point, so there was a plan with code,
09:41:03	mriedem	but then auggy was gone and sdague was gone and that whole HP team
09:41:11	mriedem	er IBM team or whatever
09:41:46	mriedem	gibi: yes that's the one
09:41:54	mriedem	definitely want to remove that in stein if we can
09:41:55	gibi	mriedem: thanks. I've added a note
09:43:03	sean-k-mooney	mriedem: are you in china right now or is this very early for you. im guessing its quite late but just wondering what time zone your in since your not normally on at this time
09:43:37	mriedem	sean-k-mooney: i'm in china this week
09:44:02	mdbooth	ratailor: Looks like the proposed code wasn't merged and was abandoned.
09:45:02	mdbooth	mriedem: Thanks for pointing that out. I'll follow up on the ML with a question around if this is still something we want to do.
09:45:33	mdbooth	ratailor: Unless you want to do that?
09:45:55	ratailor	mdbooth, no problem.
09:54:17	mriedem	Kevin_Zheng: i think resize to same host might have fixed the doubled allocation issue after all - we move the "source" allocations to the migration record and the target (new flavor) allocations to the instance
09:55:21	mriedem	Kevin_Zheng: although we'll still have allocations for both flavors against the same resource provider (compute node), so they are essentially doubled in placement...
09:57:09	Kevin_Zheng	you mean already fixed?
09:57:46	mriedem	well, depends on what is considered the issue; the instance, as a consumer, will only have allocations for the new_flavor during the resize,
09:58:02	mriedem	but the resource provider (compute node) will have allocations for both the old_flavor and new_flavor, but to different consumers
09:58:10	mriedem	so that is still an issue
09:58:38	mriedem	if i resize from 2 VCPU to 4 VCPU, during the resize to same host, the host RP will have 6 VCPU being used, rather than 4
09:58:46	Kevin_Zheng	Ok got it
09:59:09	Kevin_Zheng	But if it only has two more?
09:59:11	sean-k-mooney	mriedem: that is only while the resize is happening correct
09:59:23	Kevin_Zheng	Like only has 4
09:59:39	Kevin_Zheng	And we do 2->4
09:59:40	mriedem	sean-k-mooney: correct
09:59:55	mriedem	Kevin_Zheng: if the host only has 4 VCPU inventory, the resize to same host would fail
10:00:16	Kevin_Zheng	Yeah, but that’s our case
10:00:22	mriedem	yeah...
10:00:25	mriedem	alright, going to dinner
10:00:39	Kevin_Zheng	Coll
10:00:41	Kevin_Zheng	Cool
10:08:11	mdbooth	ratailor: I wonder if we can get zzzeek to comment on http://lists.openstack.org/pipermail/openstack-dev/2018-August/133202.html
10:11:20	ratailor	mdbooth, Thanks for putting it on ML nicely, I think we should get comment from him.
10:12:14	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] api-ref: Add missing aggregates example https://review.openstack.org/591332
10:25:46	openstackgerrit	Rajesh Tailor proposed openstack/nova master: Fix case-sensitivity for metadata keys https://review.openstack.org/504885
10:38:03	nalhat	Hi, Is any one aware about the filters used by nova: 'AggregateInstanceExtraSpecsFilter' while instance reservation(Blazar)
10:40:43	nalhat	I was referring this doc: https://docs.openstack.org/blazar/latest/cli/instance-reservation.html for instance reservation by Blazar, at the time of creating an instance in nova with reservation id this filter comes in picture
11:52:07	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4) https://review.openstack.org/568462
11:54:43	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: VMware: fix TypeError while get console log https://review.openstack.org/591365
12:00:12	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
12:05:15	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842
12:10:05	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6) https://review.openstack.org/571330
12:13:18	openstackgerrit	Chris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next https://review.openstack.org/591367
12:14:46	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
12:18:32	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
12:22:25	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
13:10:14	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] api-ref: Add missing aggregates example https://review.openstack.org/591332
13:47:46	openstackgerrit	Chris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next https://review.openstack.org/591367
13:50:22	efried	Scheduler meeting in 10 minutes in #openstack-meeting-alt
14:30:38	dansmith	sahid: we wanted to make sure you saw this: https://review.openstack.org/#/c/591275/
14:30:54	dansmith	sahid: and that the rationale was clear, because we discussed mostly on irc
14:31:28	dansmith	basically, I think that doing that in the libvirt driver will always be racy because of where the plug happens,
14:32:03	dansmith	but since mriedem added it generically in the compute manager (where it can be done properly) it has the same effect you were going for
14:32:22	dansmith	and, the two conflict of course because the wait in compute manager will "eat" the notification so that the libvirt wait will never see it

Earlier   Later