Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-27
03:08:36 deepak_mourya efried: Hi, what should be the good string to use here according to you https://review.openstack.org/#/c/577726/6/nova/api/openstack/placement/handlers/resource_provider.py
03:09:11 efried deepak_mourya: Yeah, sorry, I didn't leave a lot of detail on that. Are you familiar with the framework for defining the error codes?
03:10:23 deepak_mourya efried: It's ok, any example for the same in the nova before ?
03:10:32 efried deepak_mourya: You want to define it in nova/api/openstack/placement/errors.py, and then look at how the other consts therein are being used.
03:10:46 efried deepak_mourya: But you can do that in a followon patch. The -1 is just for the failing functional test.
03:16:43 deepak_mourya efried: ok, i didn't understand much from here nova/api/openstack/placement/errors.py
03:16:43 deepak_mourya ok so what exactly these things are
03:16:43 deepak_mourya DEFAULT = 'placement.undefined_code'
03:16:43 deepak_mourya INVENTORY_INUSE = 'placement.inventory.inuse'
03:16:43 deepak_mourya CONCURRENT_UPDATE = 'placement.concurrent_update'
03:17:02 deepak_mourya Do i need to create another consts for the same?
03:17:40 efried deepak_mourya: Yes. It's explained in this spec: http://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/placement-api-error-handling.html
03:18:01 efried deepak_mourya: But again, don't try to do it in this same patch. Submit a separate one on top of this one.
03:19:17 deepak_mourya efried: ok i will go through with it first, thanks for the reply and help
03:19:54 efried deepak_mourya: Thanks for the fix!
03:29:32 openstackgerrit xulei proposed openstack/nova master: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/576693
03:52:28 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: WIP: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261
04:27:07 gmann melwitt: can you approve this BP (queens continue work). I have started the coding on this and should be able to finish soon- https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-rocky
04:27:48 gmann melwitt: do not want to bring this for reviewer untill BP is approved.
04:59:37 openstackgerrit Merged openstack/nova master: [placement] Demonstrate bug in consumer generation handling https://review.openstack.org/577914
05:59:47 gmann nova api office hour time
06:00:22 gmann #startmeeting nova api
06:00:23 openstack Meeting started Wed Jun 27 06:00:22 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot.
06:00:24 openstack Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
06:00:27 openstack The meeting name has been set to 'nova_api'
06:00:31 gmann who all here?
06:01:03 gmann might be many china folks are in linuxcon as alex_xu mentioned
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: ^

Earlier   Later