Posted	Nick	Remark
#openstack-nova - 2019-10-29
18:52:43	efried	mriedem: +2
18:56:58	dansmith	yeah okay
18:57:11	dansmith	oh I should read the damn commit message :P
19:00:27	mriedem	always start with the commit message...
19:00:30	mriedem	at least on my patches
19:00:53	dansmith	well, I like to look at code first, but sometimes forget to go back
19:01:00	dansmith	mriedem: thanks for recheckin' mah thing
19:01:48	mriedem	hey, it's my pleasure
19:01:54	mriedem	;P
19:05:01	mriedem	brinzhang keeps pushing this spec since berlin and i just don't see the justification for it in case others want to pile on https://review.opendev.org/#/c/580336/
19:05:08	mriedem	it also requires changing what is now the swap volume only api
19:05:15	mriedem	which could get a bit weird
19:25:48	openstackgerrit	Matt Riedemann proposed openstack/nova stable/rocky: Fix listing deleted servers with a marker https://review.opendev.org/690721
19:25:48	openstack	bug 1849409 in OpenStack Compute (nova) rocky "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem)
19:25:48	openstackgerrit	Matt Riedemann proposed openstack/nova stable/rocky: Add functional regression test for bug 1849409 https://review.opendev.org/690720
19:27:11	mriedem	melwitt: i worked around the rocky-only intermittent failure you noticed in ^
19:35:54	artom	mriedem, re: brinzhang's spec - isn't that heat's job?
19:38:08	artom	(Not updating volume attachments, deleting related things as a unit)
19:38:15	artom	(Which is what they want... right?)
19:39:28	artom	Oh, just read your last comment
19:39:38	dansmith	artom: this is more about instances that were not created with the flag set the way they want it, right/
19:40:04	artom	I was thinking "make this volume go away when its instance does"
19:40:16	dansmith	right but you can already do that today
19:40:24	dansmith	if you get the flag set when you create the instance
19:40:28	artom	Not "oops, I messed up, please keep this volume when its instance goes away"
19:40:34	dansmith	if you don't, then you might want to change it later, which I thought is the point of this spec
19:40:35	artom	dansmith, right, they want to set it after
19:42:08	artom	... besides the wrong API stuff that mriedem raised, seems reasonable to me...
20:08:34	mriedem	the use case is you boot from volume with delete_on_termination=True, do some awesome stuff in the volume and then realize you don't want it deleted when you delete the server so you want to change delete_on_termination=False,
20:08:59	mriedem	which you can do since 2.79 with non-root volumes,
20:09:10	mriedem	and you can create a snapshot of the server which will create volume snapshots since forever
20:09:36	mriedem	so it seems like a waste of effort to bake that all into an API that is already solely just dealing with swap volume semantics
20:10:39	mriedem	obviously we *can* do this but i'm saying it doesn't seem justifiable that we *should* do it
20:11:15	gmann	efried: can you remove -2 on this, spec is merged and I am rebasing the series now - https://review.opendev.org/#/c/674038/
20:12:34	mriedem	another spec that is a good example of things we can do but i'm not sure we should https://review.opendev.org/#/c/682302/
20:12:48	mriedem	but we kind of opened pandora's box when adding changePassword in the first place
20:13:25	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles and mapping in policy base class https://review.opendev.org/645452
20:15:08	mriedem	heh http://lists.openstack.org/pipermail/openstack-discuss/2019-October/010429.html
20:15:38	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add policy deprecation fixture and Suppress warnings in tests https://review.opendev.org/676670
20:15:56	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-services API policies https://review.opendev.org/648480
20:16:08	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the target in os-services APIs policy https://review.opendev.org/676688
20:20:05	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing admin_actions policies https://review.opendev.org/657698
20:20:30	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in Admin Actions https://review.opendev.org/657823
20:20:57	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in Admin Action API policies https://review.opendev.org/676682
20:21:14	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095
20:23:52	efried	gmann: done
20:24:13	gmann	johnthetubaguy: melwitt ^^ rebased the policy patches. this and all its base patches are one set of os-services API policy change - https://review.opendev.org/#/c/676688/2
20:24:15	gmann	efried: thanks
20:27:50	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing admin_actions policies https://review.opendev.org/657698
20:28:05	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in Admin Actions https://review.opendev.org/657823
20:28:15	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in Admin Action API policies https://review.opendev.org/676682
20:28:27	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095
21:21:24	openstackgerrit	sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775
21:40:07	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add cross-cell resize policy rule and enable in API https://review.opendev.org/638269
21:40:08	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832
21:40:08	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656
21:40:09	openstackgerrit	Matt Riedemann proposed openstack/nova master: libvirt: flatten rbd image during cross-cell move spawn at dest https://review.opendev.org/691991
21:42:00	mriedem	gibi: when you get a chance, can you come back on these cross-cell resize patches that you were +2 on before but have been rebased https://review.opendev.org/#/c/635668/ https://review.opendev.org/#/c/635684/
21:45:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add cross-cell resize policy rule and enable in API https://review.opendev.org/638269
21:45:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: libvirt: flatten rbd image during cross-cell move spawn at dest https://review.opendev.org/691991
21:45:23	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832
21:45:23	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656
21:53:27	mriedem	smcginnis: btw, still waiting on https://review.opendev.org/#/c/687058/ for queens EM, it's been getting rechecked for a couple of weeks now - random unrelated failures and a long queue for nova to get ci run
21:53:54	mriedem	i think it if continues to fail into tomorrow i say we just cut it and let that not be part of the last release, it's a rando 3rd party volume driver fix that is low priority
21:54:08	smcginnis	mriedem: Thanks, I had meant to check up on things today.
21:54:59	smcginnis	Dang, (10 rechecks)
21:57:32	mriedem	dansmith: http://status.openstack.org/elastic-recheck/#1844929 continues to hurt the gate pretty badly - any ideas on ways that we could maybe punch/tickle the scheduler/cell db post-upgrade in grenade before running smoke tests to see if we can flush that out somehow?
22:06:52	openstackgerrit	Merged openstack/nova master: Switch to opensuse-15 nodeset https://review.opendev.org/689020
22:39:23	openstackgerrit	Sundar Nadathur proposed openstack/nova-specs master: Fix a bug in the sequence diagram. https://review.opendev.org/692001
23:18:41	openstackgerrit	Merged openstack/nova stable/queens: Fixes multi-registry config in Quobyte driver https://review.opendev.org/687058
23:18:53	openstackgerrit	Merged openstack/nova master: Fix race in test_vcpu_to_pcpu_reshape https://review.opendev.org/691941
#openstack-nova - 2019-10-30
01:16:06	openstackgerrit	Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772
02:41:42	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243
02:41:42	openstackgerrit	Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242
02:41:43	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245
02:41:43	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244
02:41:44	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
02:41:44	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
03:14:26	openstackgerrit	Merged openstack/nova master: Make nova-next multinode and drop tempest-slow-py3 https://review.opendev.org/683988
03:24:43	openstackgerrit	Qiu Fossen proposed openstack/nova-specs master: Support fuzzy querying instance by tag https://review.opendev.org/691651
03:25:21	openstackgerrit	Sundar Nadathur proposed openstack/nova-specs master: Updated Nova-Cyborg interaction spec. https://review.opendev.org/684151
06:04:03	openstackgerrit	Andreas Jaeger proposed openstack/nova stable/train: Switch to opensuse-15 nodeset https://review.opendev.org/692032
06:07:31	openstackgerrit	Andreas Jaeger proposed openstack/nova stable/stein: Switch to opensuse-15 nodeset https://review.opendev.org/692033
07:39:47	openstackgerrit	Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772
08:50:25	alex_xu	gibi: good morning
08:50:58	gibi	alex_xu: good morning
08:51:11	alex_xu	gibi: just want to check the plan of project update
08:51:31	gibi	alex_xu: for me the slides looks OK
08:51:39	alex_xu	yes, I think it is ok also
08:51:57	alex_xu	do we need to expand the U release, I guess there are more specs
08:52:23	alex_xu	but can't sure which is for sure can be accepted
08:52:38	alex_xu	gibi: so who responsible for which pages?
08:53:01	gibi	alex_xu: let me check what are the freshly approved spec...
08:53:19	gibi	alex_xu: I think we should only mention already approved specs
08:53:31	alex_xu	cool
08:55:45	gibi	alex_xu: regarding spliting the work. I'm OK if you or stephenfin talks about the generic part of the beginning. I can talk about couple of features on slides 5 - 9. I think we can note who talk about what in the speaker notes
08:56:36	alex_xu	gibi: ok, cool, I guess all we familar with different features
08:56:48	gibi	alex_xu: yes definitly
08:57:29	alex_xu	gibi: looks like stephenfin is on vacation already, probably we need to catch him at Shanghai :)
08:58:03	gibi	alex_xu: I've just checked we can less specs approved than what is already mentioned on the Ussuri slides so I think we dont need to extend that

Earlier   Later