Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-05-27
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 ➜ openstack workflow create --help | grep -A4 '^output formatters'
14:06:11 pgaxatte output formatters:
14:06:11 pgaxatte output formatter options
14:06:11 pgaxatte -f {csv,json,table,value,yaml}, --format {csv,json,table,value,yaml}
14:06:11 pgaxatte the output format, defaults to table
14:06:54 pgaxatte ➜ openstack image create --help | grep -A4 '^output formatters'
14:06:55 pgaxatte output formatters:
14:06:55 pgaxatte output formatter options
14:06:55 pgaxatte -f {json,shell,table,value,yaml}, --format {json,shell,table,value,yaml}
14:06:55 pgaxatte the output format, defaults to table
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 nope
12:56:22 mnaser im wrong
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
15:58:04 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add ansible functional tests https://review.openstack.org/570678
16:03:00 mordred dtantsur: I did! it's open in my browser to look at - although on first glance it seems to be inline with what I was thinking
16:03:10 dtantsur cool :)
20:22:38 harlowja mordred where is the super-node-pool taskmanager
23:47:21 openstackgerrit Matthew Thode proposed openstack/cliff master: update cmd2 dependency to handle py3 only versions https://review.openstack.org/571044
#openstack-sdks - 2018-05-30
03:22:35 openstackgerrit tianhui proposed openstack/python-openstackclient master: Fix doc mistakes https://review.openstack.org/571074
06:31:39 openstackgerrit yanpuqing proposed openstack/openstacksdk master: Modify the error message when unsetting gateway and setting FIP https://review.openstack.org/571101
06:33:56 openstackgerrit yanpuqing proposed openstack/openstacksdk master: Modify the error message when unsetting gateway and setting FIP https://review.openstack.org/571101
14:21:31 haleyb dtroyer: can you take a look at https://storyboard.openstack.org/#!/story/1760405 ? i see this happening with more than just a single network command, and don't know whether it's a more generic bug in OSC
14:33:30 frickler haleyb: dtroyer: there are at least two similar issues, maybe even duplicates depending on how this is handled. https://storyboard.openstack.org/#!/story/2001973 https://storyboard.openstack.org/#!/story/1757063
14:34:49 haleyb frickler: and i could probably file another, but they all do seem to be in the "neutron" osc space
14:38:17 frickler haleyb: fyi, I started tagging such stories with "osc-neutron" with the intent of working on some of them once my patches are starting to get merged again
14:39:32 frickler haleyb: I'd also like to create a board summarizing them, but sadly storyboard is lacking functionality for that
14:40:26 haleyb frickler: thanks. i haven't used storyboard much and was at first looking for a severity option but didn't see it, hence the ping here
14:40:33 frickler haleyb: but if you could apply that tag to related stories you come across, that would already be a good step I think
14:41:16 frickler haleyb: the intention as far as I understand it, is to implement everything via tags. but we would need some documented way of using them
14:44:01 haleyb right. i'd almost tag these as neutronclient regression, since it works fine
14:48:27 frickler that's a good idea I think. not sure though whether we need a specific tag though, or whether "osc-neutron" + "regression" would work
14:52:41 openstackgerrit Merged openstack/cliff master: update cmd2 dependency to handle py3 only versions https://review.openstack.org/571044
15:34:48 mordred haleyb, frickler: well, it's not a neutronclient regression since osc doesn't use neutronclient - but yeah, we should make sure that openstacksdk is throwing an exception that actually has the server-side error message (there is code in sdk to do this, but since error content isn't consistent it might not be doing the right thing for neutron)
15:37:25 haleyb mordred: so does that look like something in osc, or the sdk, or ?
15:38:17 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: Add system role functionality https://review.openstack.org/524416
15:39:14 mordred haleyb: not 100% sure - it'll take some investigation to be sure- just wanted to bring up that it's possible the sdk isn't giving osc enough data, so improvements might need to be made in either or both
16:02:00 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add ansible functional tests on stable-2.6 https://review.openstack.org/571240
16:10:54 frickler mordred: it's a regression for users moving from neutron cli to OSC, not a regression within osc itself, but still very important I think. if you have a better term for that kind of situation, feel free to propose a different tag
16:11:04 mordred ah- yes
16:11:20 mordred that makes sense to me - I thought the earlier suggestion was that it was a regression in neutronclient
16:11:37 lbragstad o/ curious if anyone would be interested in reviewing a python-openstackclient patch https://review.openstack.org/#/c/524416/5
17:09:03 openstackgerrit Merged openstack/openstacksdk master: Add ansible functional tests https://review.openstack.org/570678

Earlier   Later