| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-12-01 | |||
| 21:21:15 | slaweq | dtroyer: thx for Your comment in https://review.openstack.org/#/c/515401 | |
| 21:21:23 | slaweq | I have one more question to You about this patch | |
| 21:21:26 | dtroyer | ok | |
| 21:21:34 | slaweq | can You see my last comment on it maybe? | |
| 21:21:41 | slaweq | thx in advance | |
| 21:22:30 | dtroyer | I would not add the —details option to the show command | |
| 21:22:47 | dtroyer | show commands already return everything they receive from the API | |
| 21:23:10 | slaweq | yes, I know | |
| 21:23:22 | dtroyer | it would only be doing additional queries for things in the resource fields (looking up image names for example) that we add to show, normally | |
| 21:23:24 | slaweq | but I was asking about list command | |
| 21:23:54 | dtroyer | the comment asks about quota show, did you mean list there? | |
| 21:23:56 | slaweq | but list command currently is IMHO returning quota per tenant | |
| 21:24:17 | slaweq | sorry, maybe I don't understand something still | |
| 21:24:18 | slaweq | so | |
| 21:24:32 | dtroyer | ok I may have misread that then | |
| 21:24:46 | dtroyer | in "change its behaviour" you man list? | |
| 21:24:50 | slaweq | I understood that You want to add "--detail" flag to list command instead of show (like it is currently proposed), right? | |
| 21:24:52 | dtroyer | s/man/mean/ | |
| 21:24:59 | dtroyer | yes | |
| 21:24:59 | slaweq | yes, I mean list | |
| 21:25:38 | slaweq | so, AFAIU list command currently lists "non default" quota for all tenants | |
| 21:26:05 | slaweq | I want to add call which will return quota values for one tenant only | |
| 21:26:17 | dtroyer | ok, then I get it. and yes, sometimes we do change things about how list commands based on options. I should probably think about it a bit deeper to be sure, but we may want to add some options to do those sorts of selections, and leave the defaults to get what we have today | |
| 21:26:22 | dtroyer | does that make sense? | |
| 21:26:53 | dtroyer | normally, for one tenant, we would add —project for the filter, or something (I can't rememner exactly what) to use the tenant used for auth | |
| 21:27:32 | slaweq | so You propose to add some option (detail) to list command instead of show command | |
| 21:27:40 | dtroyer | yes | |
| 21:27:48 | slaweq | and this option will change behaviour of List command | |
| 21:27:54 | dtroyer | yes | |
| 21:27:59 | slaweq | and different data will be returned | |
| 21:28:09 | slaweq | ok, I understand now | |
| 21:28:20 | slaweq | thx | |
| 21:28:20 | slaweq | I will try to do it this way then | |
| 21:28:24 | dtroyer | by different, you mean they are still quotas, but filtered differently? | |
| 21:28:32 | slaweq | yes, still quota | |
| 21:28:42 | dtroyer | yes, that is it | |
| 21:28:49 | slaweq | but only for one tenant but with details about used/reserved/limit values | |
| 21:28:59 | dtroyer | how do you select the tenant? | |
| 21:29:02 | slaweq | instead of list of tenants and limits for each of them | |
| 21:29:30 | slaweq | probably it will be taken from context or set by project_id parameter | |
| 21:29:39 | dtroyer | ok, that should work | |
| 21:29:42 | slaweq | I will see how it's done now for "show" command | |
| 21:30:37 | dtroyer | it may make sense to re-think the entire quota command set, we can make breaking changes for osc 4.0 if this is getting too awkward | |
| 21:31:34 | slaweq | ok, I will start doing that and I will see how it will go :) | |
| 21:31:43 | slaweq | but thx a lot for Your help on that | |
| 21:31:48 | dtroyer | thank you for staying with this! | |
| 21:32:02 | slaweq | no prob | |
| 21:37:35 | mordred | dtroyer: oh, like they are not even executing ... lemme look | |
| 21:38:38 | dtroyer | I'm not sure where to look for that | |
| 21:43:29 | htaccess | question did shade used to require openstacksdk and python-openstackclient but no longer does | |
| 21:44:26 | htaccess | it looks to me that in 1.24.0 and earlier shade was pulling in openstacksdk and python-openstackclient but not longer does with 1.25.0 | |
| 21:44:48 | dtroyer | before the merge, shade did not require the SDK or OSC, it originally used the project python libs, then brought al of that internal. | |
| 21:45:06 | dtroyer | I don't have an older shade handy, is either of those listed in requirements.txt? | |
| 21:45:16 | dtroyer | they may be in test-requirements.txt to do testing | |
| 21:50:57 | htaccess | diff shade-1.25.0/requirements.txt shade-1.24.0/requirements.txt only shows six and futures version changes | |
| 21:51:26 | htaccess | are the packages in test-requirements.txt actually installed when you do a pip install? | |
| 21:52:38 | mordred | htaccess: openstacksdk and python-openstackclient have in the past come as transitive depends via python-ironicclient | |
| 21:52:49 | htaccess | mordred: that makes more sens | |
| 21:53:36 | htaccess | so its probably due to python-ironicclient not pulling them in any more | |
| 21:53:54 | mordred | yah. it looks like python-ironicclient finally removed the dep on python-openstackclient | |
| 21:54:14 | htaccess | anyway it was silly to be relying on them being pulled in ill actaully put them in my requirements.txt if they are requirments :) | |
| 21:54:17 | htaccess | thanks | |
| 21:54:21 | mordred | although we're also about to drop our depend on python-ironicclient ... so the transitive depend wouldhave cone away as of 1.26 anyway | |
| 21:54:30 | mordred | htaccess: woot! | |
| 21:54:33 | htaccess | :) | |
| 21:54:53 | mordred | htaccess: now - in the not-too-distant future shade will grow a depend on openstacksdk ... but that's unlikely before january I'd guess | |
| 21:55:18 | htaccess | ah, interesting | |
| 21:57:33 | mordred | htaccess: tl;dr - we merged the shade code in to sdk, and once we're happy with the shape of that, we'll turn shade the library into a thin backwards-compatibility shim on top of the shade-derived functionality in sdk | |
| 21:58:12 | mordred | but I want to make sure we're happy with how it's working sdk-side before we start mucking with consuming it from shade | |
| 22:01:01 | openstackgerrit | Carlos Konstanski proposed openstack/python-openstackclient master: openstack subnet create fails when tags is None https://review.openstack.org/524739 | |
| 22:01:23 | mordred | dtroyer: hrm. | |
| 22:02:02 | htaccess | ah ok, so in the future i will be able to install openstacksdk and get shade plus whats already in the sdk? | |
| 22:03:25 | mordred | htaccess: yup | |
| 22:09:38 | mordred | dtroyer: I've looped in jeblair to help me look :) | |
| 22:10:01 | dtroyer | kk | |
| 22:17:28 | openstackgerrit | Carlos Konstanski proposed openstack/python-openstackclient master: openstack subnet create fails when tags is None https://review.openstack.org/524739 | |
| 23:09:54 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | |
| 23:33:57 | openstackgerrit | Merged openstack/python-openstackclient master: Make py27 and py35 versions and template of unit-tips jobs https://review.openstack.org/524670 | |
| 23:37:46 | openstackgerrit | Merged openstack/python-openstackclient master: Avoid tox_install.sh for constraints support https://review.openstack.org/524671 | |
| #openstack-sdks - 2017-12-02 | |||
| 00:04:19 | openstackgerrit | Merged openstack/cliff master: add support for legacy command name translation https://review.openstack.org/522380 | |
| 00:04:20 | openstackgerrit | Merged openstack/cliff master: Updated from global requirements https://review.openstack.org/520389 | |
| 00:30:18 | openstackgerrit | Merged openstack/osc-lib master: Avoid tox_install.sh for constraints support https://review.openstack.org/524604 | |
| 01:35:39 | openstackgerrit | Merged openstack/python-openstackclient master: openstack subnet create fails when tags is None https://review.openstack.org/524739 | |
| 04:57:13 | openstackgerrit | zhangyifan proposed openstack/osc-lib master: parameter attr in def find can not be None https://review.openstack.org/524784 | |
| 14:51:32 | mordred | dtroyer: the issue with the tips jobs actually exposed a zuul bug that got introduced recently | |
| 14:52:32 | mordred | dtroyer: jeblair has some patches up to fix it, but the osc patches will be also a good verification case | |
| 15:15:40 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986 | |
| 15:15:41 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988 | |
| 15:15:41 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987 | |
| 15:15:42 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989 | |
| 15:16:46 | mordred | TheJulia: ^^ rebased those last 4 for you | |
| 15:17:35 | mordred | TheJulia: also - I think we should perhaps add some of the bifrost jobs to shade - I'm not sure which ones would be the best choice | |
| 15:18:09 | openstackgerrit | Rosario Di Somma proposed openstack-infra/shade master: Merge OpenstackCloud and OperatorCloud files https://review.openstack.org/524877 | |
| 15:20:16 | TheJulia | mordred: thanks, likely a good idea, our tinyIPA leveraging one should just be using cirros | |
| 15:21:03 | mordred | TheJulia: cool - I'll get a patch up fr that | |
| 15:21:16 | mordred | rods: did you catch my rebase in that one? | |
| 15:22:21 | dtroyer | mordred: oh, nice? :) so when they run again we're good? | |
| 15:23:23 | mordred | dtroyer: yup! | |
| 15:23:52 | mordred | dtroyer: I think we should be good to recheck them on monday once the fix is rolled out | |
| 15:24:11 | dtroyer | great, thanks | |