Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-05-25
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
12:55:44 mnaser oh wait
12:55:47 mnaser might be an env issue
12:56:22 mnaser im wrong
12:56:22 mnaser nope
12:56:24 mnaser "Version 0.9.0+ of cmd2 supports Python 3.4+ only. If you wish to use cmd2 with Python 2.7, then please install version 0.8.x."
12:56:30 openstackgerrit Dmitry Tantsur proposed openstack/keystoneauth master: Add optional support for retrying certain HTTP codes https://review.openstack.org/570934
12:56:40 dtantsur mnaser: yep, we're all broken by that
12:56:44 mnaser oh i'm late
12:56:45 mnaser :p
12:56:57 dtantsur better late than never :D well, not for global breakages..
13:04:23 openstackgerrit Mohammed Naser proposed openstack/cliff master: Don't install cmd2>=0.9 https://review.openstack.org/570817
13:06:21 openstackgerrit Mohammed Naser proposed openstack/cliff master: Don't install cmd2>=0.9 https://review.openstack.org/570817
13:27:47 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add ansible functional tests https://review.openstack.org/570678
13:43:25 openstackgerrit Petr Kovar proposed openstack/os-api-ref master: Raise warning if top-level value is not a dict https://review.openstack.org/563178
14:05:47 vakuznet_ what do you think about this http://paste.openstack.org/show/722264/
14:10:06 vakuznet_ cloud.compute.live_migrate_server(vm.id) - does not work ^^^
14:10:50 vakuznet_ is it (known) bug?
14:14:23 mordred vakuznet_: it's not a known bug (at least not one that I know about) - seems like a potentially bad microversion negotiation - force as a parameter wasn't added until 2.30 but that call is sending 2.1
14:42:21 vakuznet_ mordred, thanks for info
15:03:26 dtantsur mordred: hey! did you see my pings above? I guess I may better understand your idea of microversions in openstacksdk on the example of my patch :)
15:08:57 openstackgerrit Hongbin Lu proposed openstack/openstacksdk master: Add 'port_details' to Floating IP https://review.openstack.org/533811

Earlier   Later