Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-10-29
18:05:45 dansmith yeah I know
18:11:49 mriedem ok, posting a fix for that bug
18:11:57 mriedem it's using the _wait_for_state_change i hate
18:12:02 mriedem but all of the libvirt functional tests use
18:33:01 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix race in test_vcpu_to_pcpu_reshape https://review.opendev.org/691941
18:33:06 mriedem dansmith: efried: ^ fixes that gate bug
18:48:28 dansmith mriedem: I haven't looked, but why does that fix it? because it might catch the ACTIVE before the transition or something?
18:51:46 efried dansmith: yes, that's how I understand it.
18:52:00 efried because it catches "not ACTIVE" immediately, rather.
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: Add functional regression test for bug 1849409 https://review.opendev.org/690720
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: Fix listing deleted servers with a marker https://review.opendev.org/690721
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: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656
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: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: libvirt: flatten rbd image during cross-cell move spawn at dest https://review.opendev.org/691991
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: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: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: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: ksa auth conf and client for Cyborg access https://review.opendev.org/631242
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:43 openstackgerrit Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244
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:44 openstackgerrit Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
02:41:44 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
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

Earlier   Later