Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-03-09
17:50:53 cdent the PUT of multple limits, if getting rid of that is okay?
17:51:13 lbragstad well - it's a PUT, with a list of limits, and an ID in each limit
17:51:17 lbragstad which seems weird
17:51:28 cdent yes, that is weird
17:51:40 lbragstad i can see the batch update case
17:51:45 cdent and the reason it got in was because of the desire to do batch changes?
17:51:49 lbragstad but the batch create seems more critical
17:51:55 lbragstad right
17:51:57 cdent It seems to me that batch creates are much more common than batch updates, yeah?
17:52:05 lbragstad i would agree
17:52:06 cdent or rather not common
17:52:12 cdent but voluminous
17:52:20 cdent and upatess more granular
17:52:36 lbragstad right - if we associate an id to the thing, you can do a PATCH
17:52:42 lbragstad and that feels a little better
17:52:54 cdent also, in order for that PUT to be legit it would have to replace or update all of the limits
17:53:02 lbragstad true
17:53:41 lbragstad hmmm - i can see that being a big list if you just wanted to update a single limit
17:53:44 cdent So I would think removing PUT would be right, except for backward compat issues or whatever...
17:53:49 cdent yup
17:54:16 lbragstad ok - that makes sense
17:54:28 lbragstad the api is still marked as experimental and discoverable
17:55:09 cdent ah, good
17:55:33 cdent do you want me to respond to the message, or was above enough/useful?
17:56:18 lbragstad cdent: i can send a note linking to this conversation if that's easier, but certainly wouldn't want you to *not* weigh in on the conversation
17:56:46 cdent I'll go ahead and write something, for sake of completeness
17:57:01 lbragstad cdent: thanks for the input, we appreciate it
18:57:08 openstackgerrit eden nuriel proposed openstack-infra/shade master: added filters to inventory class and filters args to cmd/inventory.py - usefull for getting smaller set of cloud resources as the inventory when sharing a big project. https://review.openstack.org/530278
22:59:20 openstackgerrit Merged openstack/cliff master: Align parsed() call with cmd2 versions >= 0.7.3 https://review.openstack.org/548625
#openstack-sdks - 2018-03-10
00:09:19 openstackgerrit Merged openstack/cliff master: Fix cmd2 doc URL https://review.openstack.org/548063
08:25:38 frickler dtroyer: while testing https://review.openstack.org/550105 I was able to reproduce the SSL issue mentioned earlier. your patch introduces it for the "add fip" command
08:26:27 frickler dtroyer: for "openstack floating ip list" it is present in earlier versions and git bisect points to https://review.openstack.org/523938
08:28:09 frickler dtroyer: it might also be relevant that when I test this on a local devstack, running any OSC command from .tox/py27 only works if I add the "--insecure" option, which I don't need when running the devstack-installed global OSC version
08:29:12 frickler so the quintessence seems to be that 523938 seems to break the network client from being passed the insecure option properly
08:39:08 frickler it seems that there are in fact two issues with that, I need this double patch in order to resolve the issue for me http://paste.openstack.org/show/697635/
13:05:53 openstackgerrit OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/551449
13:09:06 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/548672
13:17:05 openstackgerrit OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/549536
13:39:34 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902
13:39:52 openstackgerrit OpenStack Proposal Bot proposed openstack/os-api-ref master: Updated from global requirements https://review.openstack.org/551592
13:40:17 openstackgerrit OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/551595
13:40:41 openstackgerrit OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/551598
13:41:37 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/551601
13:50:01 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/544222
13:50:13 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/551631
14:06:59 openstackgerrit Merged openstack-infra/shade master: Updated from global requirements https://review.openstack.org/551449
14:11:21 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Replace guts with openstack.config https://review.openstack.org/549307
14:43:44 openstackgerrit Merged openstack-infra/shade master: Adds toggle port security on network create https://review.openstack.org/550646
15:01:17 openstackgerrit Merged openstack/python-openstacksdk master: Redo role assignment list query filters https://review.openstack.org/550700
15:14:58 openstackgerrit Merged openstack/python-openstacksdk master: Prepare for os-client-config wrapper https://review.openstack.org/549306
18:05:56 openstackgerrit melissaml proposed openstack/keystoneauth master: Update links in README https://review.openstack.org/551694
#openstack-sdks - 2018-03-11
17:36:48 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstacksdk master: Fix octavia devstack tests https://review.openstack.org/551792
21:09:17 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstacksdk master: Fix octavia devstack tests https://review.openstack.org/551792
21:37:33 openstackgerrit Theodoros Tsioutsias proposed openstack/python-openstackclient master: Add force parameter for quota set https://review.openstack.org/551811
#openstack-sdks - 2018-03-12
01:24:52 adriant mordred, lbragstad: regarding the weird "user.id" vs "user_id" thing with role assignments: https://bugs.launchpad.net/keystone/+bug/1755056
01:24:53 openstack Launchpad bug 1755056 in OpenStack Identity (keystone) "role assignment query filter names inconsistent" [Undecided,New]
01:24:58 adriant bug opened as requested.
01:58:24 openstackgerrit Merged openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/551631
05:01:49 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstacksdk master: Fix devstack tests https://review.openstack.org/551792
06:58:17 openstackgerrit wangqi proposed openstack/python-openstacksdk master: Replace old http links with the newest https ones in docs https://review.openstack.org/551860
08:05:35 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstacksdk master: Fix devstack tests https://review.openstack.org/551792
12:55:56 openstackgerrit Merged openstack/python-openstacksdk master: Replace old http links with the newest https ones in docs https://review.openstack.org/551860
14:05:50 openstackgerrit Andreas Jaeger proposed openstack/keystoneauth master: Remove tox_install.sh and align with constraints consumption https://review.openstack.org/550837
16:33:56 openstackgerrit Dongcan Ye proposed openstack/python-openstackclient master: Remove duplicated network attributes https://review.openstack.org/547703
16:37:32 openstackgerrit Dongcan Ye proposed openstack/python-openstackclient master: Remove deprecated ip floating commands https://review.openstack.org/546924
17:52:38 openstackgerrit Chris Dent proposed openstack/api-wg master: Add guidance on needing cache-control headers https://review.openstack.org/550468
18:35:32 openstackgerrit Peter Balogh proposed openstack/python-openstacksdk master: Fix TypeError in case of FloatingIP add and remove https://review.openstack.org/552112
22:18:28 openstackgerrit Lingxian Kong proposed openstack/service-types-authority master: Add qinling project https://review.openstack.org/550787
#openstack-sdks - 2018-03-13
00:22:47 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
06:46:26 openstackgerrit OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/552305
06:49:16 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/548672
06:56:30 openstackgerrit OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/549536
07:05:01 openstackgerrit Peter Balogh proposed openstack/python-openstacksdk master: Fix TypeError in case of FloatingIP add and remove https://review.openstack.org/552112
07:18:53 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902
07:28:22 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/544222
07:28:32 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/552385
09:59:31 openstackgerrit Jens Harbott (frickler) proposed openstack-infra/shade master: Fix devstack jobs to use neutron-qos properly https://review.openstack.org/552459
10:28:05 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
10:28:06 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstackclient master: Adapt service names to new neutron-* scheme https://review.openstack.org/552469
10:31:27 frickler dtroyer: FYI, tried to fix your unittests a bit ^^ also hoping that devstack works better this way, though it might be we have to squash these two together, too
11:16:35 rcernin hi, who is the right person to answer diff in design between legacy and new openstack cli? I wonder why we got rid of --fields?
13:07:06 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
13:43:00 dtroyer frickler: thank you! I was working on a different approach, a bit simpler I think, but had not considered multiple inheritance for the test class, that simplifies it even more
13:43:23 dtroyer I'm going to let the tests run first, then squash in the zuul fix too
13:52:09 frickler dtroyer: we may not need the zuul fix immediately, I've agreed with andreaf to go back to using q-* services for now, see https://review.openstack.org/552503 .
13:53:15 dtroyer ah, ok. damn, it's almost 3 years since that change-over started
13:55:04 frickler yeah, would still be good to make the switch, just without blocking everyone else, that's why I proposed the dedicated job
13:56:45 frickler dtroyer: did you also look at the "--insecure vs. sdk_connection" issue? I'm wondering what would be the best way to set up a functional test exposing it
13:57:44 dtroyer frickler: I have not
14:06:16 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
14:09:41 frickler dtroyer: o.k., that is even nicer. now if you depend-on https://review.openstack.org/552503 it should hopefully even be passing all tests
14:10:40 dtroyer doh, thanks
14:11:23 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
16:27:26 openstackgerrit Harald Jensås proposed openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974
17:58:34 frickler dtroyer: if you are still there can you re-do your +W on https://review.openstack.org/550105 ? the dependency merged later, seems like zuul doesn't pick that up. would avoid a recheck and running the check queue again
18:13:09 dtroyer frickler: done, there it goes
18:28:29 openstackgerrit Hongbin Lu proposed openstack/cliff master: Remove the warning of getargspec removal https://review.openstack.org/552652
19:54:09 openstackgerrit Merged openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
19:54:35 frickler \o/
19:55:18 dtroyer \o/

Earlier   Later