Posted	Nick	Remark
#openstack-nova - 2017-03-02
22:12:44	mriedem	which is the microversion to list instance actions for deleted instances,
22:12:53	mriedem	and hard delete instance actions when archiving an instance
22:13:14	mriedem	cfriesen: if you wanted to push a change to add a comment about why we don't soft delete instance actions when deleting an instance, in the instance_destroy db api method, i'd +2 that
22:13:28	cfriesen	mriedem: okay
22:22:21	openstackgerrit	Anusha Unnam proposed openstack/nova master: Add description to policies to availability_zone https://review.openstack.org/440836
22:25:30	cfriesen	one final DB question (for today, anyway). What's the story around console_auth_tokens added in Newton? Is something going to delete them?
22:25:57	dansmith	my jar runneth over
22:28:58	mriedem	cfriesen: talk to melwitt about the spec for that
22:29:23	cfriesen	mriedem: I'm looking at https://blueprints.launchpad.net/nova/+spec/convert-consoles-to-objects ... looks like it got only partially implemented?
22:29:31	mriedem	cfriesen: yes
22:29:39	mriedem	melwitt is going to re-propose the spec for pike
22:30:25	cfriesen	okay, cool. so in Newton the table is there but unused?
22:31:24	melwitt	looks like it. you can see the patches that merged vs ones that ended up abandoned here https://review.openstack.org/#/q/topic:bp/convert-consoles-to-objects,n,z
22:32:18	melwitt	so my plan is to repropose the spec and then resurrect those old patches and fix them up if needed to finish the work
22:33:12	cfriesen	melwitt: thanks. I was looking for users of console_auth_token_create() but didn't see any.
22:33:31	melwitt	yeah
22:36:05	cfriesen	melwitt: okay, looks like https://review.openstack.org/#/c/325381/ introduces a periodic task to clean up expired tokens. seems maybe a bit odd to drive it from the compute nodes though.
22:38:36	melwitt	cfriesen: yeah, although that works out for cells, since the tokens will be kept in cell databases, and only n-cpu and n-cond run in a cell
22:44:33	jaypipes	dansmith: hey, splitting that big patch is prolly a good idea at this point, yeah. I'll do it when I get to my hotel this evening (currently sitting in SeaTac airport...)
22:44:48	dansmith	jaypipes: okay I think it'll be easy
22:45:51	cfriesen	just noticed something odd. at https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L1879 we soft-delete migrations when the instance is deleted, but then at https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L6474 we special-case the migrations table like we do for instance_actions. Is that no longer needed?
22:47:30	jaypipes	dansmith: ya, shouldn't be a big deal.
22:47:35	mriedem1	cfriesen: that's for dealing with migrations that were not soft deleted
22:47:51	mriedem1	so you'd have migrations in the db that weren't soft deleted and those would break the archive
22:48:07	mriedem1	anything after we started soft deleting migrations when the instance was deleted would be fine
22:48:51	mriedem	to drop that special case code in the archive function we'd probably need a schema migration that drops those old records
22:49:09	mriedem	or an online data migration actually
22:49:15	mriedem	then you don't have the downtime issue
22:54:38	cfriesen	is there anything stopping us from doing something similar to soft-delete instance_actions and instance_actions_events now in instance_destroy()? (Now that we've made the other changes from the mailing list summary, I mean.) Or is it not worth the effort?
22:55:14	mriedem	we don't want to soft delete those because their useful for operators to find out what actions were performed on an instance, which may be deleted now
22:55:19	mriedem	that was the reason for the microversion
22:55:24	mriedem	*they're
22:55:26	cfriesen	okay, got it
22:55:38	mriedem	i'd put that in the comment you add to instance_destroy
22:55:42	mriedem	wink wink
22:55:52	cfriesen	that's why I asked. :)
23:05:48	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Get instance availability_zone without hitting the api db https://review.openstack.org/439754
23:05:49	openstackgerrit	Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023
23:05:49	openstackgerrit	Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022
23:05:50	openstackgerrit	Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024
23:05:50	openstackgerrit	Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
23:05:51	openstackgerrit	Dan Smith proposed openstack/nova master: Enlighten server tags API about cells https://review.openstack.org/440856
23:05:51	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025
23:27:55	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
23:28:24	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
23:28:24	dansmith	dtp: still around?
23:28:30	dtp	yessir
23:29:00	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Fix formatting errors and warnings https://review.openstack.org/407364
23:29:14	dansmith	dtp: there are a few other tests that I think are depending on the hypervisor listing, like the aggregates one that uses hypervisors to get a list of hostnames
23:29:34	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
23:29:44	dansmith	so fixing that might get us a fair swipe of passing tests
23:29:47	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
23:31:56	dtp	dansmith that's cool. i haven't had a chance to start on that yet
23:32:03	dansmith	*gasp*
23:32:27	openstackgerrit	Sujitha proposed openstack/nova master: Add description to policies in admin_actions. https://review.openstack.org/440829
23:32:28	dansmith	dtp: okay well, let me know if you get overly delayed
23:32:28	dtp	yeah sorry...misjudged my week
23:32:36	dansmith	s'ok
23:33:16	dtp	don't let me slow you down - if it's better for you to do that one and me to come back again when i _really_ can focus, that's fine w/ me
23:33:39	dansmith	okay I'll ping you if I break your lock
23:34:48	dtp	okey dokey
23:53:41	openstackgerrit	Giridhar Jayavelu proposed openstack/nova-specs master: VMware: add SR-IOV support https://review.openstack.org/317126
#openstack-nova - 2017-03-03
00:07:44	openstackgerrit	Giridhar Jayavelu proposed openstack/nova-specs master: VMware: add SR-IOV support https://review.openstack.org/317126
00:41:49	mriedem	feel free to take them to the bridge, james brown
01:04:37	mriedem	alex_xu: are you ok with this? https://review.openstack.org/#/c/440654/
01:23:59	mriedem	lyarwood: oh so close https://review.openstack.org/#/c/242602/
01:45:11	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: [WIP] Live migration of rescued instances https://review.openstack.org/440893
01:55:52	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.api.openstack.compute.test_block_device_mapping https://review.openstack.org/440475
02:38:02	openstackgerrit	Matt Riedemann proposed openstack/nova master: Reno for additional-notification-fields-for-searchlight https://review.openstack.org/428835
02:38:05	openstackgerrit	Zhenyu Zheng proposed openstack/nova-specs master: Send out notifications when instance tags chaned https://review.openstack.org/429569
02:43:50	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: [WIP] Live migration of rescued instances https://review.openstack.org/440904
04:21:11	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add api-ref for filter/sort whitelist https://review.openstack.org/421760
04:32:46	gaoyuan	/SET use_status_window OFF
05:00:18	openstackgerrit	Huan Xie proposed openstack/nova master: Make xenapi driver compatible with assert_can_migrate https://review.openstack.org/439960
05:04:42	openstackgerrit	Rikimaru Honjo proposed openstack/nova master: Consolidate unit tests for shelve API https://review.openstack.org/421220
05:35:48	Herr-Herner	The first time :-) Can someone please help me... I am setting up OpenStack Ocata on UBuntu 16.04. and facing a blocker regarding Libvirt/KVM. I think it is just a small issue, but with a big impact.
05:50:25	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738
05:50:50	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436
06:32:48	ratailor	johnthetubaguy, Hi
06:32:53	ratailor	johnthetubaguy, you around ?
07:00:59	openstackgerrit	Zhenyu Zheng proposed openstack/python-novaclient master: Tags and Metadata fields with unicode cannot be correctly displayed https://review.openstack.org/440949
07:24:50	migi	edleafe: hi
08:29:19	bauzas	good morning Nova
08:32:40	openstackgerrit	Feodor Tersin proposed openstack/nova-specs master: Add ScaleIO ephemeral storage backend https://review.openstack.org/304715
09:13:27	openstackgerrit	Huan Xie proposed openstack/nova master: Make xenapi driver compatible with assert_can_migrate https://review.openstack.org/439960
09:14:32	denaitre	hi everyone
09:14:47	denaitre	I was looking for a piece of information
09:15:45	denaitre	I noticed that one can set different api adresses in nova.conf (e.g. glance, neutron)
09:16:14	denaitre	I thought nova would contact keystone for getting such information
09:17:46	denaitre	I feel like I missed something
09:18:36	bauwser	denaitre: which specific conf opts are you looking at ?
09:19:43	denaitre	for instance, `api_servers` under [glance]
09:20:19	denaitre	i think it is called `url` under [neutron]
09:27:08	bauwser	ah that
09:27:11	bauwser	well
09:28:25	bauwser	denaitre: the api_servers conf opt is a legacy feature helping glance client to do round-robin calls between all the glance API services
09:28:47	bauwser	from my personal PoV, that's very unfortunate we accepted that
09:34:32	adisky_	hi , can anybody help on some query related to nova volume-update
09:34:34	adisky_	?
09:41:07	denaitre	bauwser: thanks for the answer! from my PoV you should declare your different glance api endpoints to a service like haproxy to manage the round-robin, and register the haproxy endpoint in keystone, does it make sense?

Earlier   Later