Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-05-25
17:38:12 gtema I was verifying by testing os_auth and trying a bad credentials
17:38:56 mordred gtema: yah ... so the ansible modules shoul duse sdk.exceptions.OpenStackCloudException yes?
17:39:26 gtema mordred: yupp
17:40:38 mordred cool - do you have time to fix the places that are using sdk.OpenStackCloudException ? Or we could just do it as a followup if you don't
17:40:59 gtema where have you found direct use?
17:41:36 gtema I see, os_itonic
17:41:38 gtema noing now
17:41:42 mordred gtema: I left inline comments on the PR - if you search for 'sdk.Open' it'll show them
17:41:44 mordred sweet
17:43:23 gtema mordred: went out
17:46:16 mordred gtema: crap. sorry - I just landed a pr that made a conflict with yours. can you rebase?
17:47:09 gtema yes, I see it already
17:47:12 gtema working
17:48:27 gtema so I should just remove this _get_aloowed_project, right?
17:48:47 mordred yah
17:51:58 gtema mordred: ready
17:59:45 mordred \o/
18:03:31 gtema mordred: crap, again shippable stucked
18:04:16 mordred gtema: yah - I just asked how to re-trigger it
18:04:45 gtema as I know either admin retriggers it or close-open PR
18:04:54 mordred ugh
18:06:27 gtema mordred: should I close-open?
18:06:56 mordred gtema: yah - let's try that
18:08:11 gtema ok, from second reopen started
18:09:11 gtema mordred: am away for now (8PM) will periodically come back to check IRC
18:09:30 mordred gtema: kk. I'll merge the pr as soon as it turns green
18:09:37 gtema great. Thanks
18:09:50 gtema oh crap, again cancelled
18:10:20 gtema in this case I think admins should retrigger
18:15:10 mordred gtema: ok. we'll just wait for it to run I suppose. it should be fine since it passed before. I'll keep watching it
18:28:07 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Remove app_name config tests https://review.openstack.org/569673
19:45:26 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: Don't display router's is_ha and is_distributed attributes always https://review.openstack.org/567620
19:54:57 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: Make max_burst_kbps option as optional for bw limit QoS rule https://review.openstack.org/567828
#openstack-sdks - 2018-05-26
02:55:21 openstackgerrit Merged openstack-infra/shade master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569425
03:10:55 openstackgerrit Merged openstack/os-client-config master: Remove app_name config tests https://review.openstack.org/569673
03:14:13 openstackgerrit Merged openstack/openstacksdk master: Avoid globally modifying yaml library https://review.openstack.org/568705
03:20:42 openstackgerrit Merged openstack/openstacksdk master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569422
15:29:00 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Plumb additional_user_agent support throughout https://review.openstack.org/569666
19:09:26 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Plumb additional_user_agent and collect_timings support throughout https://review.openstack.org/569666
#openstack-sdks - 2018-05-27
13:22:07 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Honor service-type aliases in config https://review.openstack.org/566929
13:22:21 mordred dtantsur|afk: ^^ updated - thanks for the review!
13:43:54 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add ansible functional tests https://review.openstack.org/570678
13:55:18 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add ansible functional tests https://review.openstack.org/570678
13:55:33 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Update ansible test job to run against stable-2.5 https://review.openstack.org/570680
13:58:03 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Remove shade-ansible-devel job https://review.openstack.org/570682
14:18:29 openstackgerrit Merged openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
15:17:12 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Honor service-type aliases in config https://review.openstack.org/566929
15:17:25 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935
15:17:37 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049
15:22:58 openstackgerrit Merged openstack/openstacksdk master: Several improvements to resource.wait_for_status https://review.openstack.org/570538
15:24:19 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add ansible functional tests https://review.openstack.org/570678
16:50:05 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935
16:50:06 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049
18:04:36 openstackgerrit Merged openstack/openstacksdk master: allow passing ansible variables to ansible tests https://review.openstack.org/570600
23:13:22 adriant mordred: if Daniel doesn't follow up, I might submit a new patch for https://review.openstack.org/#/c/544544/ that fix the unpacking part
23:13:31 adriant fixes*
#openstack-sdks - 2018-05-28
09:02:04 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: rename vars to defaults to allow overriding in ansible tests https://review.openstack.org/570751
13:43:30 pgaxatte hi
13:43:54 pgaxatte am I on the right channel to ask some questions about python-openstackclient?
13:44:30 slaweq pgaxatte: yes :)
13:44:38 slaweq You can try :)
13:45:06 pgaxatte great :)
13:46:06 pgaxatte i'm trying to understand why the mistral client (and precisely openstack workflow ...) does not allow the "shell" output format
13:46:24 pgaxatte i don't know where to look to add this formatter
13:47:44 pgaxatte the reason i'm looking for it is because i'd like to create a new provider in puppet-mistral and the class i should inherit from depends on the --format=shell option
13:49:43 pgaxatte (and hi slaweq, hope RedHat is treating you well :P)
13:51:19 slaweq pgaxatte: yes, it's very good in RH :)
13:52:10 slaweq pgaxatte: I have no idea about mistral but it looks at first glance that code for this is in openstacksdk repo: https://github.com/openstack/openstacksdk/blob/master/openstack/workflow/workflow_service.py
13:52:19 slaweq so maybe You should look there
13:55:35 pgaxatte slaweq: it doesn't look like the filtering abilities are here
13:56:08 pgaxatte if I compare with openstack image (which has the --format=shell) there aren't any differences
14:00:21 slaweq to what command You are trying to use format=shell ?
14:03:01 pgaxatte "openstack workflow create" for instance
14:06:11 pgaxatte the output format, defaults to table
14:06:11 pgaxatte -f {csv,json,table,value,yaml}, --format {csv,json,table,value,yaml}
14:06:11 pgaxatte output formatter options
14:06:11 pgaxatte output formatters:
14:06:11 pgaxatte ➜ openstack workflow create --help | grep -A4 '^output formatters'
14:06:54 pgaxatte ➜ openstack image create --help | grep -A4 '^output formatters'
14:06:55 pgaxatte the output format, defaults to table
14:06:55 pgaxatte -f {json,shell,table,value,yaml}, --format {json,shell,table,value,yaml}
14:06:55 pgaxatte output formatter options
14:06:55 pgaxatte output formatters:
14:11:43 slaweq maybe it is some bug but according to docs only show command have this "shell" formatter: https://github.com/openstack/python-openstackclient/blob/9599ffe65d9dcd4b3aa780d346eccd1e760890bf/doc/source/contributor/humaninterfaceguide.rst
14:18:09 pgaxatte this is weird because the puppet-openstacklib provider (all puppet providers are base on it) is using shell formatter for show AND create commands: https://github.com/openstack/puppet-openstacklib/blob/stable/ocata/lib/puppet/provider/openstack.rb#L102
14:18:38 pgaxatte (this is ocata's branch but it's the same on master)
14:19:55 pgaxatte slaweq, clearly not all clients respect the rule you showed me
14:21:12 pgaxatte do you know where the formatting is done? in python-openstackclient or in a separate lib (like osc_lib maybe)?
14:21:33 slaweq pgaxatte: I have no idea
14:22:05 pgaxatte well i'll keep digging :) thanks for your help
14:22:40 pgaxatte but meanwhile i'll try a different approach in my puppet provider :)
14:35:54 slaweq pgaxatte: good luck then :)
16:32:43 openstackgerrit Lukáš Doktor proposed openstack/cliff master: Avoid py3 dependency on py2 https://review.openstack.org/570806
18:20:49 openstackgerrit Matt Thompson proposed openstack/cliff master: Don't install cmd2>=0.9 https://review.openstack.org/570817
22:51:59 openstackgerrit Hongbin Lu proposed openstack/openstacksdk master: [WIP] Add 'port_details' to Floating IP https://review.openstack.org/533811
#openstack-sdks - 2018-05-29
10:47:28 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Add set_node_provision_state to baremetal Proxy https://review.openstack.org/570895
10:47:38 dtantsur mordred and others: I'd like some early comments on the approach here ^^^ please. esp. around microversions.
10:51:27 dtantsur mordred: I think if we wire this ^^ into Cloud's node_set_provision_state, we won't need to set default_baremetal_microversion in your patch
12:04:56 dtantsur TheJulia: if you have any spare brain power today ^^^
12:55:27 mnaser heads up, i think the openstack client is no longer installable in py27 envs

Earlier   Later