Posted	Nick	Remark
#openstack-nova - 2017-07-07
22:06:24	cburgess	So I feel for them.. but there is no solution. They could comment the checks out of nova if they wanted to but even that is a just rolling the device on the VMs crashing on migration.
22:06:30	cburgess	Or libvirt just refusing to do it.
22:07:28	sgordon	i dont think that helps them anyway
22:07:36	sgordon	because it's passed the nova checks
22:07:41	sgordon	it's the libvirt checks kicking it out
22:07:50	cburgess	Oh yeah.. well.. like I said...
22:08:16	sgordon	thanks for being a sounding board, clearly i have friday afternoon brain
22:08:21	sgordon	i will try and frame this in an email
22:10:13	cburgess	sgordon No worries, we dealt with this very issue a lot in the essex/folsom days. Trying to figure out exacly what CPU type and model info to use to guarnatee the best mix of performance and portability.
22:12:23	sgordon	cburgess, yeah - it's basically impossible to set defaults this because any given selection is a tradeoff
22:12:34	sgordon	cburgess, depends on the characteristics and requirements of each environment
22:12:45	cburgess	sgordon I think the key bit to explains is that all the config values are basically useless once you get down to the libvirt internal code. It will *always* use whats defined in the running VMs XML.
22:12:58	sgordon	(i mean obviously we *do* set defaults for this, it's just that they arent going to fit for everyone)
22:13:17	cburgess	sgordon Exactly. So we have to play around with a sane "default" for our product and then enable customers to change it for their deployment based upon their needs.
22:13:53	cburgess	Though to be fair most use our default since its designed around maximum portability and most of customers can't have downtime for their pets. :(
22:16:45	sgordon	yeah the other aspect of course is someone may not notice when doing a greenfield deployment
22:16:54	sgordon	the problem comes when they expand later and add a new generation of hosts
22:18:00	cburgess	Yup. we didn't notice it until our first client did a server refresh after about a year.
23:12:23	openstackgerrit	Matt Riedemann proposed openstack/nova master: Handle uuids in os-hypervisors API https://review.openstack.org/471899
23:12:24	openstackgerrit	Matt Riedemann proposed openstack/nova master: Use uuid for id in os-services API https://review.openstack.org/464280
23:19:43	openstackgerrit	Matt Riedemann proposed openstack/nova master: Handle uuids in os-hypervisors API https://review.openstack.org/471899
23:24:25	mriedem	mgagne: you overwrote my changes and re-introduced a bug https://review.openstack.org/#/c/454322/29..31 :)
23:24:31	mriedem	i'll just push my version again
23:26:21	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add ability to signal and perform online volume size change https://review.openstack.org/454322
23:35:13	mriedem	fixed, now i just need to flesh out the tests and docs for the os-instance-actions details changes in there, probably over the weekend
23:35:27	mriedem	the tempest/devstack test stack had a clean run last night though, which is awesome
#openstack-nova - 2017-07-08
00:22:19	openstackgerrit	John Griffith proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
11:34:49	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091
15:09:37	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach https://review.openstack.org/480392
15:17:26	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.50 - List/Show all server migration types https://review.openstack.org/430839
15:17:35	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.50 - Enable cold migration with target host https://review.openstack.org/406707
16:15:15	openstackgerrit	John Griffith proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
#openstack-nova - 2017-07-09
23:42:48	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
23:52:48	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
23:59:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
#openstack-nova - 2017-07-10
04:42:09	openstackgerrit	jichenjc proposed openstack/nova master: WIP: remove extension pop https://review.openstack.org/481491
05:07:16	masber	good afternoon, I am trying to create a server but apparently I am exceeding the quota? how is it possible? http://paste.openstack.org/raw/614863/
06:06:05	openstackgerrit	Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023
06:22:11	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
06:26:16	openstackgerrit	jichenjc proposed openstack/nova master: remove extension param and usage https://review.openstack.org/481491
06:32:09	openstackgerrit	jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482013
06:34:52	openstackgerrit	bhavani proposed openstack/nova master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/482014
06:44:52	openstackgerrit	jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482020
06:50:03	openstackgerrit	jichenjc proposed openstack/nova master: Modify some comments for virt driver https://review.openstack.org/482022
06:51:56	openstackgerrit	jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482020
07:04:42	openstackgerrit	bhavani proposed openstack/nova master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/482014
07:31:12	kiennt	hi, i saw Nova already had assert:supports-rolling-upgrade tag. How you test rolling upgrade in gate job? I am woking on Heat rolling upgrade and have to setup the CI to test this.
07:51:26	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix quota class set APIs https://review.openstack.org/467999
08:49:01	openstackgerrit	Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250
08:50:34	moshele	sean-k-mooney: hi
08:50:45	openstackgerrit	sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
08:50:54	moshele	sean-k-mooney: can you review my respond on https://review.openstack.org/#/c/398265/ ?
08:53:35	openstackgerrit	Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250
09:14:24	openstackgerrit	bhavani proposed openstack/nova master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/482014
09:19:29	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform instance-live_migration_pre notification https://review.openstack.org/480119
09:20:17	ildikov	stephenfin: hi :)
09:20:26	stephenfin	ildikov: Morning :)
09:21:26	ildikov	stephenfin: just wanted to give you a heads up that there are a few positive review comments on the swap_volume change: https://review.openstack.org/#/c/456971/
09:21:40	ildikov	stephenfin: in case that would help with reviewing it :)
09:24:38	stephenfin	ildikov: Can't do any harm :) I'll take a shot again today, but I can't make any promises
09:24:46	stephenfin	(just setting expectations)
09:25:53	ildikov	stephenfin: thanks
09:27:02	ildikov	stephenfin: Matt is basically fine with the patch; and it's also not that easy to find people in the core team who're familiar with this part of the code, so I thought to ping people first who reviewed related code changes earlier
09:27:26	ildikov	stephenfin: so no pressure, but it would be a big help :)
09:27:59	stephenfin	ildikov: I must be missing something, but why didn't he +2 it himself?
09:28:34	ildikov	stephenfin: he uploaded the last patch set with some more tests
09:28:50	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform instance-live_migration_pre notification https://review.openstack.org/482070
09:32:13	gmann	alex_xu, can you check this -https://review.openstack.org/#/c/467999/9
09:32:40	alex_xu	gmann: yea, I will review before the end of day
09:33:19	gmann	alex_xu, thanks
09:33:29	alex_xu	gmann: np, thanks for the patch
09:42:36	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform instance-live_migration_post notification https://review.openstack.org/480119
09:55:18	openstackgerrit	bhavani proposed openstack/nova master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/482014
11:18:54	andreykurilin	sdague: hi! Have a minute? Can you check one small patch to novaclient which fixes discoverign versionsand unblocks gates? (It is already have one +2)
11:19:07	sdague	sure, url?
11:19:29	andreykurilin	https://review.openstack.org/#/c/479818/
11:35:18	sean-k-mooney	moshele: hi moshele i just responded in the review but yes using something like openstack port create --binding-profile
11:35:20	sean-k-mooney	'{"capabilities": ["hw-tc-offload"]}' --vnic_type direct --network private port1.
11:35:27	sean-k-mooney	works for me
11:37:14	moshele	sean-k-mooney: you want me to add a TODO comment in the code for now?
11:41:48	sean-k-mooney	moshele: https://review.openstack.org/#/c/398265/ should probably be rebased ontop of https://review.openstack.org/#/c/466051
11:42:43	sean-k-mooney	moshele: if we open a todo comment we should also log a bug to track its completeion
11:44:00	sean-k-mooney	moshele: does libvirt/ethtool currently expose the hw-tc-offload or switchdev feature flag
11:57:38	moshele	sean-k-mooney: ok bye the way we have CI ready http://52.169.200.208/13/480213/3/check-os-vif/OVS_HW_offload/08d10ee/
11:57:52	moshele	s/bye/by
11:58:21	sean-k-mooney	moshele: oh cool and you started with the 2 most value test sets +1
11:59:08	sean-k-mooney	moshele: will you be adding a full temest compute or networking run or leaving it at the scenario tests?
12:00:03	moshele	sean-k-mooney: I think the scenario covers all we need for now
12:01:20	sean-k-mooney	moshele: yep they should we only ran the other tests as nova wanted that before giveing voteing rights
12:03:27	bhagyashris	cdent: Hi, Thanks for new release of gabbi. Actually I am just trying to chcek response object with sample.json (expected response) for the placement functional gabbi test "confirm the correct post" but i am getting the error as "Unable to match $ as <@sample.json, got" after running the test.
12:04:05	bhagyashris	cdent: I am checking like: response_json_paths:
12:04:06	bhagyashris	$: <@sample.json
12:44:19	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
12:44:20	openstackgerrit	Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764
12:44:20	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
12:57:59	bhagyashris	cdent: Hi, Thanks for new release of gabbi. Actually I am just trying to chcek response object with sample.json (expected response) for the placement functional gabbi test "confirm the correct post" but i am getting the error as "Unable to match $ as <@sample.json, got" after running the test.
12:58:05	bhagyashris	cdent: I am checking like: response_json_paths:
12:58:05	bhagyashris	<bhagyashris> $: <@sample.json
12:59:06	cdent	the file sample.json needs to be in the same directory as the yaml file, and it of course needs to have the correct content in it
12:59:19	cdent	can you paste your yaml file and sample.json somewhere?

Earlier   Later