Posted	Nick	Remark
#openstack-nova - 2017-06-05
21:51:23	stvnoyes	so how would you like to resolve whether we want to go with old_attachment_id? talk with John Garbutt? I can update the rv in the meantime with that missing change.
21:53:05	mriedem	stvnoyes: yeah i'd say do what you're doing now, fix the db api thing, and then we talk to john to see if he had other idea
21:53:06	mriedem	*ideas
21:53:21	stvnoyes	ok cool
21:53:22	mriedem	i don't like it but it's probably the lesser of many evils
21:53:48	mriedem	unless of course dansmith has bright ideas he'd like to share about BDMs modeling and multi-attach
21:53:49	mriedem	...
21:54:12	dansmith	barf
21:54:32	jgriffith	You guys running some kinda vomitorium in here?
21:54:51	mriedem	vomiting on oneself is a common reaction to things in this channel
21:55:01	jgriffith	small world
21:55:14	jgriffith	seems were all more alike than we thought
21:57:21	stvnoyes	I found that libvirt driver needs to know what bdm is what and getting that info down in there was incredibly ugly. This way libvirt driver hardly needs to change at all.
21:58:57	mriedem	old_attachment_id vs attachment_id also gets weird,
21:59:05	mriedem	because we have to assume "old" means source host
21:59:15	mriedem	and ties it to migrations in the bdm object
21:59:26	mriedem	but, we have old/new flavors too for the same thing (resizes)
21:59:58	stvnoyes	using john Gr's attach code, I did get this working ok. I could migrate back and forth successfully
22:00:13	mriedem	actually, flavors gives me an idea here
22:00:31	mriedem	nova.objects.Migration has an old_instance_type_id and new_instance_type_id field
22:00:53	mriedem	i'm wondering if we can store the old/new attachment id information in the migration object
22:01:29	mriedem	however, an instance has 1 flavor,
22:01:36	mriedem	but possibly many volume attachments
22:01:39	mriedem	so it'd have to be list
22:01:44	stvnoyes	hmmm, ok i can take a look at that. is that object persisted and available to both hosts?
22:01:50	mriedem	yes
22:02:02	mriedem	that would be a cleaner interface if we can make it work
22:02:21	stvnoyes	agree, i'm not a big fan of putting new fields in the bdm.
22:02:40	stvnoyes	also, a migrate object seems more appropriate.
22:03:17	stvnoyes	i wasn't aware of that object. new things to learn.
22:04:35	openstackgerrit	Felipe Monteiro proposed openstack/nova-specs master: Patrole (RBAC) Keystone Gating https://review.openstack.org/471145
22:10:42	openstackgerrit	Eric Fried proposed openstack/nova master: Trim the fat from InstanceInfo https://review.openstack.org/471146
22:10:58	mriedem	cfriesen_: your release notes are fixed
22:11:09	efried	mriedem ^^
22:11:30	mriedem	efried: ok probably for tomorrow at this rate
22:11:32	cfriesen_	mriedem: sweet, thanks for the recheck too
22:11:43	efried	mriedem Oh, for sure.
22:25:40	openstackgerrit	Eric Fried proposed openstack/nova master: Trim the fat from InstanceInfo https://review.openstack.org/471146
22:49:33	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Uncap sphinx version https://review.openstack.org/471153
23:26:45	mriedem	cfriesen_: why does anything in https://review.openstack.org/#/c/435141/10/novaclient/tests/unit/test_shell.py need to change?
23:26:52	mriedem	i.e. what's up with all of the list -> flavor-list changes?
23:28:33	mriedem	i see in the commit message now,
23:28:46	mriedem	but still, seems we can set a default api_version in whatever fixture is used for those tests so you don't need to change them all
23:33:28	openstackgerrit	Merged openstack/nova master: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613
23:55:12	mriedem	artom: some questions inline before i approve https://review.openstack.org/#/c/465205/
23:55:32	artom	mriedem, man of your word. Much thanks, let me take a look
23:57:54	mriedem	i have to do something during girls gymnastics class (besides watch and support of course)
23:58:24	artom	Well crap, now I feel guilty for taking away father-daughter time
#openstack-nova - 2017-06-06
00:02:18	mriedem	no i'm up in a viewing area and she's down refusing to flip
00:02:26	mriedem	i'm with the screaming siblings
00:03:50	mriedem	preparing for the soul-crushing conversation on the car ride home about, "why didn't you do the thing we talked about you doing"
00:04:10	artom	Who's asking that question of whom?
00:05:08	mriedem	ha
01:03:25	mriedem	alex_xu: you should probably take a look at https://review.openstack.org/#/c/469578/
01:23:52	mriedem	jaypipes: i just reviewed the os-traits sync thing https://review.openstack.org/#/c/469578/ and there is a comment in there saying it's synced on start of the process, but i don't see where/how that happens
01:23:54	mriedem	do you know?
02:05:49	alex_xu	mriedem: I think it is only triggerred by get_by_name or get_all
02:07:39	alex_xu	mriedem: I guess cdent means that after "process started and get_by_name or get_all"
02:15:51	openstackgerrit	falseuser proposed openstack/nova master: [BugFix] Change the condition to delete a resizing instance https://review.openstack.org/463466
02:25:26	mriedem	alex_xu: maybe, but it's kind of weird to say, "none of this does anything w/o the code actually running" :)
02:25:29	mriedem	that's kind of assumed
02:26:05	mriedem	i was thinking maybe it was something people talked about in earlier patch sets but didn't actually do something
02:35:20	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923
02:47:29	openstackgerrit	Jianghua Wang proposed openstack/nova master: Fix the race condition with novnc https://review.openstack.org/470798
02:52:33	openstackgerrit	Merged openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540
02:53:02	openstackgerrit	Vu Cong Tuan proposed openstack/nova master: Fix html_last_updated_fmt for Python3 https://review.openstack.org/470676
03:38:17	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
05:08:19	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
05:30:01	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
06:05:02	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
06:07:33	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
06:07:40	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
06:12:14	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
06:12:20	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
06:16:42	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569
06:17:57	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
06:18:19	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923
06:39:15	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
07:13:50	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
07:14:19	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
07:51:23	openstackgerrit	Merged openstack/python-novaclient master: client.logger.warning wrongly used in migrations https://review.openstack.org/463508
08:09:18	bauzas	p!spy
09:13:41	openstackgerrit	jichenjc proposed openstack/nova master: Remove incorrect comments in multiple_create https://review.openstack.org/471273
09:16:26	openstackgerrit	jichenjc proposed openstack/nova master: Modify incorrect comment on return_reservation_id https://review.openstack.org/471278
09:44:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524
09:44:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625
09:46:12	sfinucan	bauzas: Can you take a look at https://review.openstack.org/#/c/438887/ today?
09:49:21	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
09:52:06	openstackgerrit	Stephen Finucane proposed openstack/nova-specs master: Revert "Cap sphinx<1.6.1" https://review.openstack.org/471298
09:52:47	openstackgerrit	Stephen Finucane proposed openstack/nova-specs master: Revert "Cap sphinx<1.6.1" https://review.openstack.org/471298
10:31:25	openstackgerrit	Chris Dent proposed openstack/nova master: Sync os-traits to Traits database table https://review.openstack.org/469578
10:48:05	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
10:58:36	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
10:58:37	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
11:07:49	sdague	anyone else want to bring this one in - https://review.openstack.org/#/c/467242/ - and get us one step closer?
12:07:40	openstackgerrit	Sean Dague proposed openstack/nova master: Remove Babel from requirements.txt https://review.openstack.org/468350
12:08:50	luisnho223	hey guys one quick question. I am doing a request to get stats from my tenant/project and everything works well but when i see get the hours for each instance/server or total GB/RAM/Hours from the tenants, the response give me numbers like 1,99999e^-9
12:09:09	luisnho223	btw, on Horizon and CLI everything is working fine
12:15:48	jaypipes	luisnho223: pls see /topic. best to ask usage questions on the openstack@ Mailing List or ask.openstack.org, for future reference...
12:16:16	jaypipes	luisnho223: when you say "doing a request to get stats", you mean you're doing a curl/http request directly to the Nova API?

Earlier   Later