Posted	Nick	Remark
#openstack-nova - 2018-11-01
23:07:25	efried	I should link today's IRC discussion in there. But I gotta run riiight now.
#openstack-nova - 2018-11-02
00:16:12	openstackgerrit	Vladyslav Drok proposed openstack/nova master: Do not remove baremetal ports in deallocate_for_instance https://review.openstack.org/473873
00:38:24	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
00:38:38	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
00:39:24	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
00:39:43	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629
00:40:26	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Use oslo_db.sqlalchemy.test_fixtures https://review.openstack.org/609352
01:14:54	openstackgerrit	gaobin proposed openstack/nova master: Improve the properties of the api https://review.openstack.org/614730
01:43:35	openstackgerrit	Nguyen Hai Truong proposed openstack/nova master: [Trivial Fix] Correct spelling error of "should" https://review.openstack.org/614899
02:03:49	gmann	artom: johnthetubaguy yeah, we keep tempest config as close as with latest defaults on service side. if that cause to fail on stable branch testing then we can modify them via devstack which is branched
02:32:06	openstackgerrit	Nguyen Hai Truong proposed openstack/nova master: [Trivial Fix] Correct spelling error of "should" and "resource" https://review.openstack.org/614899
02:38:32	openstackgerrit	Merged openstack/nova master: Modify get_by_cell_and_project() to get_not_deleted_by_cell_and_project() https://review.openstack.org/607663
02:38:41	openstackgerrit	Merged openstack/nova master: Minimal construct plumbing for nova list when a cell is down https://review.openstack.org/567785
03:06:05	openstackgerrit	Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
03:06:47	openstackgerrit	huanhongda proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/611833
03:41:55	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: WIP support attach/detach root volume 2 https://review.openstack.org/614750
05:55:31	openstackgerrit	Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
06:06:40	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/614757
07:00:24	openstackgerrit	Vieri proposed openstack/os-vif master: Update min tox version to 2.0 https://review.openstack.org/615046
07:08:33	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Bump compute service to indicate attach/detach root volume is supported https://review.openstack.org/614750
07:58:27	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for vol-backed snapshots https://review.openstack.org/615084
08:08:32	openstackgerrit	Silvan Kaiser proposed openstack/nova master: [WIP] Added Qemu libquobyte Support to the Quobyte Driver https://review.openstack.org/546500
08:23:42	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Support delete_on_termination in volume attach api https://review.openstack.org/612949
08:52:46	bauwser	good Friday nova
09:26:21	openstackgerrit	Jie Li proposed openstack/nova-specs master: Support volume-backed server rebuild https://review.openstack.org/532407
09:38:14	sri_	Hello, folks!, Does Ceph monitor IPs are still hardcoded in the database ? did anybody tried replacing ceph monitors(IP) recently ?
09:40:15	openstackgerrit	Yikun Jiang proposed openstack/nova master: Change the default values of XXX_allocation_ratio https://review.openstack.org/602803
09:40:15	openstackgerrit	Yikun Jiang proposed openstack/nova master: Use new ``initial_xxx_allocation_ratio`` CONF https://review.openstack.org/602804
09:40:16	openstackgerrit	Yikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499
09:40:16	openstackgerrit	Yikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration script https://review.openstack.org/609995
09:53:22	openstackgerrit	Nguyen Hai Truong proposed openstack/nova master: Add tox python3 https://review.openstack.org/615118
10:09:36	openstackgerrit	gaobin proposed openstack/nova master: Improve the properties of the api https://review.openstack.org/614730
10:20:44	openstackgerrit	Jie Li proposed openstack/nova-specs master: Support volume-backed server rebuild https://review.openstack.org/532407
10:25:23	mnaser	hey -- trying to fix https://bugs.launchpad.net/nova/+bug/1789423 locally, got it working but i need to get my pep8 to pass..
10:25:23	openstack	Launchpad bug 1789423 in OpenStack Compute (nova) "Server operations fail to complete with versioned notifications if payload contains unset is_public field" [Medium,Triaged] - Assigned to Mohammed Naser (mnaser)
10:25:27	mnaser	http://paste.openstack.org/show/733952/
10:25:36	mnaser	any ideas how to ideally clean that up to fit 80 chars?
11:23:12	openstackgerrit	Mohammed Naser proposed openstack/nova master: Default embedded instance.flavor.is_public attribute https://review.openstack.org/615134
11:24:59	mnaser	^ if someone can help with that briefly
11:42:43	jroll	mnaser: I'd do Flavor(\n and indent the rest by whatever looks most readable
11:43:39	jroll	mnaser: probably this if it fits, I didn't count: http://paste.openstack.org/show/734030/
11:44:18	jroll	or maybe http://paste.openstack.org/show/734031/
11:44:50	jroll	of course you could always assign those to a variable above where you define fake_flavor, too
11:45:59	openstackgerrit	huanhongda proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/611833
11:50:09	openstackgerrit	Jose Castro Leon proposed openstack/nova master: Fixes bug on attaching volume scsi on an instance boot from iso https://review.openstack.org/615141
12:11:31	openstackgerrit	Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363
12:13:41	openstackgerrit	Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328
12:14:12	openstackgerrit	huanhongda proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/611833
12:23:42	openstackgerrit	Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329
12:32:21	openstackgerrit	Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786
12:34:52	openstackgerrit	Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787
12:39:41	johnthetubaguy	jaypipes: melwitt: suddenly noticed we can do per-project instance count checks from the CellMapping table. I worked this out before and keep forgetting about it.
12:46:08	openstackgerrit	John Garbutt proposed openstack/nova master: WIP: Unified limits ideas https://review.openstack.org/615180
12:49:55	openstackgerrit	Mohammed Naser proposed openstack/nova master: Default embedded instance.flavor.is_public attribute https://review.openstack.org/615134
12:50:07	mnaser	thanks for the input jroll ! %
12:50:15	jroll	:)
12:56:06	jaypipes	johnthetubaguy: yeah, that's what melwitt added the queue_for_delete column I think in the API DB's instance_mappings table, right?
13:06:21	openstackgerrit	John Garbutt proposed openstack/nova master: WIP: Unified limits ideas https://review.openstack.org/615180
13:34:01	sri_	Hello folks, quick quastion Does Ceph monitor IPs are hardcoded in the database <block_device_mapping> ? did anybody tried replacing ceph monitors(IP) recently ?
13:39:22	johnthetubaguy	leakypipes: yeah, mind like a sieve
13:40:04	mriedem	sri_: you might be looking for https://review.openstack.org/#/c/579004/
13:40:23	mriedem	https://bugs.launchpad.net/nova/+bug/1452641
13:40:23	openstack	Launchpad bug 1452641 in nova (Ubuntu) "Static Ceph mon IP addresses in connection_info can prevent VM startup" [Medium,In progress] - Assigned to Corey Bryant (corey.bryant)
13:41:36	openstackgerrit	Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380
13:54:03	openstackgerrit	Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328
13:55:53	openstackgerrit	Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329
13:59:36	openstackgerrit	Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786
13:59:36	openstackgerrit	Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787
14:00:10	openstackgerrit	Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347
14:00:10	openstackgerrit	Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029
14:00:11	openstackgerrit	Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363
14:00:11	openstackgerrit	Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380
14:07:52	mnaser	is there any spec that proposes merging image backend + image cache (the thing that i kept talking about but never did?)
14:08:28	fried_rice	tssurya, mriedem, leakypipes: Was the CERN workaround for refresh interval based on setting CONF.compute.resource_provider_association_refresh to a high value, or was there some component of out-of-tree hackage as well?
14:09:18	leakypipes	fried_rice: UIAM, they disabled it entirely.
14:09:31	fried_rice	leakypipes: By hacking code?
14:09:42	dansmith	yup
14:10:05	fried_rice	okay. I would like to know exactly what that patch looked like at some point soon if possible.
14:10:23	mnaser	im gonna assume
14:10:23	mnaser	https://github.com/openstack/nova/blob/a0563e754ce5228df73252ea9e18821781c06d80/nova/scheduler/client/report.py#L843-L853
14:10:24	mnaser	return True
14:10:28	mnaser	:p
14:12:37	fried_rice	It would be `return False`. That's probably it, but it's going to make a difference if it was something subtly different.
14:13:03	mriedem	dansmith: so i think i have a lead on https://bugs.launchpad.net/nova/+bug/1800472
14:13:03	openstack	Launchpad bug 1800472 in OpenStack Compute (nova) "nova.tests.functional.test_server_group.ServerGroupTestV264.test_boot_servers_with_affinity_no_valid_host intermittently failing with "OpenStackApiNotFoundException: Item not found"" [Medium,Triaged]
14:13:28	mriedem	it appears that we're getting the 404 because we're looking up the nova-api service version when fetching the instance and thinking it's version is 0
14:13:44	mriedem	so it goes down the old pre-cellsv2 path and tries to get the instance out of the local-to-api db
14:13:50	mriedem	which with the cells fixture should still default to cell1
14:13:59	mriedem	but the instance is buried in cell0 which is why the lookup would fail
14:14:06	SteelyDan	orly
14:14:18	SteelyDan	surely we can remove the old paths now yeah?
14:16:04	mriedem	i think so,
14:16:26	mriedem	it's also weird since the api fixture does create a service record, and that should be in the default cell1 db
14:16:34	mriedem	so i'm not sure why we're failing to find it, or with the correct version
14:22:44	SteelyDan	ack
14:23:40	sean-k-mooney	fried_rice: did my reivew of https://review.openstack.org/#/c/614886/ make sense
14:24:06	fried_rice	sean-k-mooney: Hadn't gotten to it yet. Thank you for looking at it.
14:26:21	sean-k-mooney	no worries. tl;dr ya i like the direction some nits inline but nothing major
14:27:01	sean-k-mooney	fried_rice: we proably can do more then you do in the patch but it a solid first step if we decide to go that way
14:27:31	fried_rice	sean-k-mooney: The random jitter thing is a fine idea, but would be a totally independent piece of work. (I will respond accordingly.)

Earlier   Later