Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-04-26
19:42:39 JPL aaah
19:43:38 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Backport connect_as fix from openstacksdk https://review.openstack.org/564604
19:44:15 mordred Shrews, JPL I'm pretty sure that will fix it - and it's no wonder it didn't get backported, it was hidden inside of a patch for using oslotest's base class
19:44:16 mordred sigh
19:45:26 JPL * highfive
19:45:26 mordred Shrews: with the release of sdk I put in apatch for today, we should be able to land the patch to make occ a shim and at least remove that layer of confusion - and then I think it'll be time to do the same thing with shade - the two-parallel-codebases thing is too much for my brain
19:46:10 Shrews too much for anyone's brain
19:54:23 harlowja mordred so how much of openstacksdk vs shade going forward?
19:54:29 harlowja merging? maybe/
19:54:30 harlowja ?
19:54:49 harlowja someday?, soon? never?
19:54:50 JPL I tried updating my local copy with that diff, got this nwo:AttributeError: 'OpenStackCloud' object has no attribute 'config'
19:55:55 mordred JPL: ok. darn. I'll have to work slighly harder :)
19:56:32 JPL Traceback (most recent call last): File "shade-test.py", line 22, in <module> project_cloud = default_cloud.connect_as_project(project=project_name) File "/usr/local/lib/python2.7/site-packages/shade/openstackcloud.py", line 411, in connect_as_project return self.connect_as(**auth) File "/usr/local/lib/python2.7/site-packages/shade/openstackcloud.py", line 328, in connect_as if self.config._openstack_config: Att
19:56:48 JPL That's really ugly copy/paste, womp
19:57:25 mordred harlowja: the shade and os-client-config code is already merged into sdk - the next step (which is the really tricky one) is to remove the shade code from shade and make it a very thin compat layer on top of the equiv code in sdk
19:57:34 harlowja mordred gotcha
19:57:39 harlowja TIL, lol
19:57:50 mordred harlowja: right now we're playing the 'keep both in sync with each other while we get that all done so that it doesn't break anybody' - which sucks
19:57:59 harlowja agreed
20:00:39 mordred JPL: change config = self.config._openstack_config to config = self.cloud_config._openstack_config
20:00:47 mordred JPL: copy-pasta error :)
20:01:18 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Backport connect_as fix from openstacksdk https://review.openstack.org/564604
20:02:50 mordred Shrews: ^ had to update because I'm a terrible person
20:02:52 JPL Also if self.config._openstack_config: ?
20:03:07 JPL same thing, otherwise I think this will do the trick
20:03:50 JPL Yay, working
20:03:52 JPL if self.cloud_config._openstack_config: config = self.cloud_config._openstack_config
20:05:16 mordred \o/
20:05:35 melwitt amotoki: hi, your review would be appreciated on this change we're trying to get in to make stable/queens novaclient work together with stable/queens openstackclient so that we may update the queens novaclient constraint to 10.x https://review.openstack.org/538070 so that users can use the multi-attach feature with queens out-of-the-box
20:05:45 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Backport connect_as fix from openstacksdk https://review.openstack.org/564604
20:06:05 melwitt amotoki: forgot to link the change, of course https://review.openstack.org/562840
20:06:16 JPL Thank you all for your help
20:06:33 mordred JPL: thanks for trying out the fixes! that first one really didn't fix anything did it? :)
20:06:54 JPL lol, any time
20:07:47 JPL Last question, assuming everything passes and makes it in, how long usually before a new release with that?
20:10:17 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Infer version from old versioned service type aliases https://review.openstack.org/564299
20:11:22 mordred JPL: should have one today or tomorrow - assuming I can get TheJulia to land https://review.openstack.org/#/c/564279/ :)
20:12:05 mordred (basically, as soon as that patch lands, we can land the fix for your issue and a patch pabelanger needs and cut a release
20:12:46 JPL awesomesauce
21:01:29 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Infer version from old versioned service type aliases https://review.openstack.org/564299
22:11:53 adriant mordred: Putting the code in the SDK layer was my first thought too. I think moving the advanced logic into the SDK layer makes sense if it's still part of that service, and shade just uses it as passthrough, but multi-project workflow type stuff should always stay in shade
22:11:58 adriant if that makes sense
22:12:12 adriant at least then we have some sort of consistency for where to put stuff
#openstack-sdks - 2018-04-27
03:53:25 openstackgerrit Mark Kirkwood proposed openstack/python-openstackclient master: Add an option to container create for storage policy. Also display some policy data for container show and object store account show commands. https://review.openstack.org/564665
05:12:00 openstackgerrit Mark Kirkwood proposed openstack/python-openstackclient master: Add an option to container create for storage policy. Also display some policy data for container show and object store account show commands. https://review.openstack.org/564665
10:45:06 mordred adriant: agree
10:49:42 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Remove DataCentred from list of vendors https://review.openstack.org/564727
10:49:43 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: pypy is not checked at gate https://review.openstack.org/564729
10:49:43 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add Limestone Networks vendor info https://review.openstack.org/564728
11:04:29 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Replace guts with openstack.config https://review.openstack.org/549307
11:13:17 mordred Shrews: ^^ assuming that passes, that should take care of occ
11:50:54 ttsiouts Hello! I have two patches up https://review.openstack.org/#/c/551811/ and https://review.openstack.org/#/c/562300/. I would really appreciate it if someone could help with reviewing. Thanks in advance!!
12:08:45 openstackgerrit Merged openstack-infra/shade master: Add get_volume_limits() support https://review.openstack.org/564247
12:54:28 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Adds Senlin support to shade https://review.openstack.org/542537
12:56:12 openstackgerrit Merged openstack-infra/shade master: Backport connect_as fix from openstacksdk https://review.openstack.org/564604
13:02:00 mordred Shrews: ^^ those two landed, I think it's time to cut a shade release
13:02:49 Shrews mordred: go for it
13:03:34 mordred Shrews: https://review.openstack.org/564752 Release shade 1.28.0
13:06:40 Shrews mordred: 1.28.0? I don't see a 1.27.0 tag
13:11:35 mordred Shrews: I believe 1.27.0. and 1.27.1 both got cut on stable/queens
13:12:58 Shrews mordred: oh, yeah. not used to this new release process
13:14:04 openstackgerrit Merged openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299
13:14:38 mordred Shrews: me either - I kind of failed at paying attention to freeze dates, thus why we woundup with queens releases :)
13:35:10 amotoki melwitt: sorry for late. I will take a look at the backport patch during this weekend at the latest. I need to refresh my memory on compute/network hybrid commands.
13:44:50 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Replace guts with openstack.config https://review.openstack.org/549307
13:53:44 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add Limestone Networks vendor info https://review.openstack.org/564728
13:56:49 melwitt thanks amotoki
13:57:10 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: pypy is not checked at gate https://review.openstack.org/564729
15:07:04 openstackgerrit Merged openstack/openstacksdk master: Remove DataCentred from list of vendors https://review.openstack.org/564727
15:57:54 openstackgerrit Daniel Speichert proposed openstack/openstacksdk master: Flavor: added is_public query parameter and description property https://review.openstack.org/564817
18:37:57 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Make name setting in connect_as more resilient https://review.openstack.org/564861
20:15:25 openstackgerrit Hongbin Lu proposed openstack/openstacksdk master: Add 'port_details' to Floating IP https://review.openstack.org/533811
20:29:36 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Format port_details field of Floating IP https://review.openstack.org/533809
20:50:41 openstackgerrit Hongbin Lu proposed openstack/openstacksdk master: Add 'port_details' to Floating IP https://review.openstack.org/533811
22:35:34 openstackgerrit Mark Kirkwood proposed openstack/python-openstackclient master: Add an option to container create for storage policy. Also display some policy data for container show and object store account show commands. https://review.openstack.org/564665
22:55:56 openstackgerrit Hongbin Lu proposed openstack/openstacksdk master: Add 'port_details' to Floating IP https://review.openstack.org/533811
#openstack-sdks - 2018-04-28
16:04:21 openstackgerrit Merged openstack/openstacksdk master: Flavor: added is_public query parameter and description property https://review.openstack.org/564817
#openstack-sdks - 2018-04-29
17:59:47 openstackgerrit Theodoros Tsioutsias proposed openstack/python-openstackclient master: Add project info to server list from all projects https://review.openstack.org/562300
18:00:34 openstackgerrit Theodoros Tsioutsias proposed openstack/python-openstackclient master: Add force parameter for quota set https://review.openstack.org/551811
#openstack-sdks - 2018-04-30
05:37:34 openstackgerrit Nobuto Murata proposed openstack/python-openstackclient master: Support --community in openstack image list https://review.openstack.org/565152
10:39:09 rcarrillocruz looking at openlab zuul dashboard, i see jobs for a terraform open-telekom specific provider. Does it mean open-telekom is a fork from openstack APIs ?
11:47:29 mordred rcarrillocruz: last I checked it was possible to use open-telekom with normal apis
11:49:28 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Adds Senlin support to shade https://review.openstack.org/542537
12:27:02 frickler slaweq: can you take a look at http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/network/v2/port.py#n108 please? to me it seems like these options are bogus ("option_name" and "option_value")
12:27:40 frickler looking at the api-ref, those seem to be used only within extra_dhcp_opt, not as top-level values https://developer.openstack.org/api-ref/network/v2/#extra-dhcp-option-extra-dhcp-opt-extension
12:29:28 frickler slaweq: actually there are even more strange parameters, e.g. "ip_address"
12:29:51 slaweq frickler: looking
12:31:22 frickler all introduced in this refactoring commit quite some time ago https://review.openstack.org/#/c/364052/4/openstack/network/v2/port.py
12:35:40 slaweq frickler: you're right, looking into https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/extra_dhcp_opt.py those are only options for extra_dhcp_opt
12:36:28 slaweq and according to ip_address comment, it's for https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/allowedaddresspairs.py
12:39:00 frickler slaweq: o.k., thanks for confirming, I'll do a cleanup patch
12:43:05 slaweq frickler: thx
12:59:52 openstackgerrit Doug Hellmann proposed openstack-infra/shade master: remove redundant information from release notes build https://review.openstack.org/565214
13:08:35 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk master: Drop bogus attributes from network port resource https://review.openstack.org/565217
13:08:45 frickler slaweq: FYI ^^
13:29:36 openstackgerrit Merged openstack/openstacksdk master: Add Limestone Networks vendor info https://review.openstack.org/564728
13:33:51 openstackgerrit Merged openstack-infra/shade master: Make name setting in connect_as more resilient https://review.openstack.org/564861
14:12:53 openstackgerrit Merged openstack-infra/shade master: remove redundant information from release notes build https://review.openstack.org/565214
20:05:03 gtmanfred mordred: you around to talk about https://storyboard.openstack.org/#!/story/2001794 ?

Earlier   Later