Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-27
06:02:06 gmann just checking if someone here so we can start the API things and from next week we can have office hour with more people
06:09:26 gmann ok seems no one available today so we can skip today. i will send the update i collected from bugs etc over email.
06:09:38 gmann #endmeeting
06:09:40 openstack Meeting ended Wed Jun 27 06:09:38 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
06:09:41 openstack Minutes: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-06-27-06.00.html
06:09:42 openstack Minutes (text): http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-06-27-06.00.txt
06:09:43 openstack Log: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-06-27-06.00.log.html
06:28:09 openstackgerrit jiang wei proposed openstack/nova master: Add action initiator attribute to the instance payload https://review.openstack.org/536243
06:50:54 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/578019
06:53:27 openstackgerrit Merged openstack/nova master: Remove mox in test_conductor.py (2) https://review.openstack.org/564659
07:20:57 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
08:03:53 tssurya good morning gibi: regarding the handling the cell down spec, I wanted to know your take on these concerns you specified (https://review.openstack.org/#/c/557369/4/specs/rocky/approved/handling-down-cell.rst@85): since we are anyways going for a new microversion, I was thinking if its better to just skip all these keys totally or put up 'UNKNOWN' values for those keys. do you have a strong take?
08:44:08 gibi tssurya: I prefer not including the keys we dont now the value at all as that is a bit harder to missunderstood than a field with some unknown value. I don't have this as a strong take as the microversion bump can be used to precisely define the behavior anyhow.
08:49:44 tssurya gibi: yea thanks,
08:50:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for config drive extension https://review.openstack.org/578329
09:03:02 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for keypair extension https://review.openstack.org/578332
09:16:32 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
09:19:53 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for multiple create extension https://review.openstack.org/578338
09:25:30 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261
09:34:55 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for scheduler hint extension https://review.openstack.org/578343
09:41:19 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for security group extension https://review.openstack.org/578345
10:04:56 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for BDM v2 extension https://review.openstack.org/577185
10:05:04 openstackgerrit Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262
10:08:55 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Fix capacity tracking in POST /allocations https://review.openstack.org/578133
10:12:16 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Fix capacity tracking in POST /allocations https://review.openstack.org/578133
10:15:05 openstackgerrit Chris Dent proposed openstack/nova master: [placement] demonstrate part of bug 1778591 with a gabbi test https://review.openstack.org/578139
10:15:05 openstack bug 1778591 in OpenStack Compute (nova) "GET /allocations/{uuid} on a consumer with no allocations provides no generation" [Medium,Triaged] https://launchpad.net/bugs/1778591
10:15:33 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for user data extension https://review.openstack.org/578349
10:24:01 openstackgerrit Merged openstack/nova master: placement: Make API history doc more consistent https://review.openstack.org/477478
10:24:08 openstackgerrit Merged openstack/nova master: Fix API reference for os-floating-ips-bulk https://review.openstack.org/578106
10:24:16 openstackgerrit Merged openstack/nova master: Fix API reference for os-floating-ip-dns https://review.openstack.org/578107
10:24:25 openstackgerrit Merged openstack/nova master: Update API reference for os-floating-ip-pools https://review.openstack.org/578108
11:09:32 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table. https://review.openstack.org/560832
11:09:33 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object https://review.openstack.org/573628
11:09:34 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375
11:09:35 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401
11:09:36 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166
11:09:37 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model https://review.openstack.org/571465
11:09:38 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534
11:34:57 melen Guys, anyone knows what service calls the procedure of creating ssh-keys, distribute them among compute nodes - for migration use?
13:05:42 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
13:28:52 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
13:41:37 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix unit test modifying global state https://review.openstack.org/578387
13:46:14 gibi mriedem, stephenfin: this ^^ was the problem casing my RequestSpec instances to behave strangely
13:46:24 mriedem already approved
13:46:28 gibi mriedem: thanks
13:48:07 gibi it was nicely lost two days to tracking it down
13:49:02 openstackgerrit sahid proposed openstack/nova stable/queens: add mtu to libvirt xml for ethernet and bridge types https://review.openstack.org/578010
13:57:42 dansmith gibi: nice
13:58:02 openstackgerrit Chris Dent proposed openstack/nova master: [placement] demonstrate part of bug 1778591 with a gabbi test https://review.openstack.org/578139
13:58:02 openstack bug 1778591 in OpenStack Compute (nova) "GET /allocations/{uuid} on a consumer with no allocations provides no generation" [Medium,Triaged] https://launchpad.net/bugs/1778591
14:03:42 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for config drive extension https://review.openstack.org/578329
14:05:44 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for config drive extension https://review.openstack.org/578329
14:06:11 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for keypair extension https://review.openstack.org/578332
14:07:21 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for multiple create extension https://review.openstack.org/578338
14:07:55 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for multiple create extension https://review.openstack.org/578338
14:08:27 mriedem melwitt: fyi, i had removed the bp for ^ from rocky since there weren't any changes posted as of june 13th, but now there are several https://review.openstack.org/#/q/topic:bp/api-extensions-merge-rocky+(status:open+OR+status:merged) so i guess throw it back into the rocky mix if you want
14:08:43 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for scheduler hint extension https://review.openstack.org/578343
14:09:12 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for scheduler hint extension https://review.openstack.org/578343
14:09:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for security group extension https://review.openstack.org/578345
14:13:59 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
14:14:00 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
14:14:01 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
14:14:02 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
14:14:03 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
14:14:04 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
14:14:05 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
14:24:03 openstackgerrit Merged openstack/nova master: Drop API compat handling for old compute error cases https://review.openstack.org/546256
14:38:09 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for security group extension https://review.openstack.org/578345
14:47:14 mriedem dansmith: you should take a look at this https://review.openstack.org/#/c/563375/ - it's attempting to deprecate InstanceGroup.policies (which is a list of policy names - even though a group can only ever have 1 policy associated with it), and add InstanceGroup.policy which is a new InstanceGroupPolicy object - but it straddles both fields during the deprecation. i'm not sure if we can just drop the InstanceGroup.policies fie
14:47:14 mriedem like metadetails in there) and just handle the backport compat if necessary?
14:47:24 mriedem but the code handling both is overly complicated imo
14:47:33 mriedem yikun: ^
14:48:00 dansmith mriedem: okay in a sec
14:50:34 mriedem and gerrit is dead
14:52:21 mriedem nvm
14:55:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove compatibility code for instance groups https://review.openstack.org/578192
14:55:55 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove unused DB API instance_group_delete method https://review.openstack.org/578223
15:03:37 melwitt mriedem_afk: copy that
15:08:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create schema for user data extension https://review.openstack.org/578349
15:15:42 dansmith mriedem_afk: commented on that policy thing, let me know if that all makes sense
15:19:25 openstackgerrit qingszhao proposed openstack/os-traits master: Add release note link in README https://review.openstack.org/578444
15:20:08 fungi mriedem_afk: when you have a sec, i'm curious whether https://review.openstack.org/572195 (or more specifically its stable backport) is considered sufficient to mitigate https://launchpad.net/bugs/1742102 on upgrade without needing any configuration adjustments
15:20:08 openstack Launchpad bug 1742102 in OpenStack Compute (nova) "Simple user can disable compute" [High,In progress]
15:20:42 fungi just trying to decide how to move forward on the report now that those related backports have all merged
15:20:47 dansmith fungi: that's the point
15:21:48 fungi cool, i wonder why they weren't flagged as fixing that bug. can we consider those bugtasks fix-committed now?
15:21:53 dansmith fungi: second paragraph of the commit message
15:22:40 dansmith fungi: because that bug is also about the fact that we consider overquota for volumes as a retryable event
15:23:23 fungi ahh. and someone retrying overquota volume requests can still be used as a denial of service attack against compute nodes?
15:23:52 dansmith no, meaning compute auto-retrying something we know will fail is just a waste of resources
15:24:09 dansmith the user can retry of course, but we shouldn't automatically retry something we know is going to fail again
15:24:22 dansmith we have a list of things that we know are terminal and just cause the build to go to error,
15:24:58 dansmith and this bug is also indicating we should do that for this situation, aside from the fact that it shouldn't have counted against the build failures for the disabler thing
15:25:14 fungi okay, so the bug report remains open, but the security vulnerability described in the bug is no longer relevant on upgrade to the related patches
15:25:27 dansmith aye
15:25:55 fungi perfect, thanks! i'll move forward on drafting the advisory and get some input on the wording once i have it fleshed out
15:26:08 dansmith okay
15:26:53 openstackgerrit qingszhao proposed openstack/os-vif master: Add release note link in README https://review.openstack.org/578446
15:52:30 openstackgerrit Matt Riedemann proposed openstack/os-vif master: Add release note link in README https://review.openstack.org/578446

Earlier   Later