| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-02-28 | |||
| 18:07:16 | ankur-gupta-f4 | No tap tho | |
| 18:09:44 | sindhu | reedip: what did I miss | |
| 18:10:11 | reedip | reordering the CLIs | |
| 18:10:33 | reedip | but also the sdk_session issue which I was talking about earlier | |
| 18:10:57 | reedip | the Funtional test fails because you are unsetting FLoating IP after Router's interface removal | |
| 18:11:29 | reedip | but they fail even before that , because there is no sdk_session in ClientManager | |
| 18:14:14 | reedip | sindhu: you can check the issue by running the tox command I pasted earlier | |
| 18:17:16 | sindhu | reedip: but I thought sdk_connection error was coz my devstack was not re based | |
| 18:17:27 | reedip | i dont think so ... | |
| 18:17:38 | dasanind | reedip: I didn't get the request | |
| 18:17:56 | reedip | what made u consider that the devstack rebase could cause that error sindhu? | |
| 18:20:25 | dhellmann | wklm : if you could put the entire output in a pastebin, that might help. formatting and some of the message (I think) was lost when you pasted it in channel | |
| 18:21:06 | dhellmann | wklm : it would also help to know which version of pip and setuptools you have installed | |
| 18:26:49 | sindhu | reedip: coz I had the same error before and when I asked dean, he said that was the latest change which was merged and I should get those changes when rebaked | |
| 18:27:22 | reedip | Oh ok | |
| 18:30:15 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Reorganize object_store docs https://review.openstack.org/439093 | |
| 18:31:03 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 18:34:05 | reedip | sindhu | |
| 18:34:09 | reedip | updated ^^ | |
| 18:36:52 | sindhu | reedip: cool thanks 😊 | |
| 18:43:37 | stevemar | ankur-gupta-f1: i was going to wait until the google doc was finalized | |
| 18:45:16 | ankur-gupta-f1 | stevemar: nearly finalized. Just need to add the ones that are plugins from Neutronclient (bgp, tap). Lbaas commands wont be for awhile and the rest are in progress. | |
| 18:46:20 | ankur-gupta-f1 | Though I am not seeing the BGP stuff, just BGP VPN | |
| 18:50:20 | ankur-gupta-f1 | stevemar: it looks like someone deleted some items. | |
| 18:57:15 | thingee | mordred: why are some clouds optional is specifying here? https://review.openstack.org/#/c/430967/ | |
| 18:59:44 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Reorganize cluster docs https://review.openstack.org/439110 | |
| 19:04:10 | stevemar | ankur-gupta-f1: you can check the history | |
| 19:04:35 | ankur-gupta-f1 | stevemar: its okay. Ignorance is bliss, readding now | |
| 19:04:44 | stevemar | :) | |
| 19:05:29 | reedip | stevemar, ankur-gupta-f1 : the doc needs to be updated, aint it ? Seems like its missing some things maybe from network side | |
| 19:06:44 | reedip | Note : I dont think we would be migrating LBaaS v1 to OSC | |
| 19:06:51 | reedip | confirming the same from lbaas team | |
| 19:07:10 | ankur-gupta-f1 | yea | |
| 19:07:16 | ankur-gupta-f1 | waiting for Lbaasv2 | |
| 19:07:21 | ankur-gupta-f1 | reedip: ^^ | |
| 19:07:43 | reedip | So we can remove the LBaaS v1 from the sheet | |
| 19:08:08 | reedip | as per johnsom: neutron-lbaas v1 API will not be supported by OSC, you will have to use the legacy neutron client ] | |
| 19:08:18 | ankur-gupta-f1 | thats what I was saying. It won't be done probably this cycle either | |
| 19:09:03 | johnsom | Yeah, no nlbaas v1 API support in OSC | |
| 19:09:05 | reedip | so we can remove that from the sheet, to get a better report | |
| 19:09:31 | reedip | I mean we do not need to track what we do not want to move , aint it ? | |
| 19:10:43 | ankur-gupta-f1 | agreed. Plus it takes up so much space haha | |
| 19:11:07 | reedip | updated the sheet , please check | |
| 19:11:31 | reedip | I am not removing them though, changed their status for now : stevemar, please confirm :) | |
| 19:12:05 | ankur-gupta-f1 | reedip: also the 'lbaas-*' commands | |
| 19:13:13 | reedip | ankur-gupta-f1 : that we need to track | |
| 19:13:23 | reedip | that would be the basis of our movement from NC to OSC | |
| 19:13:55 | ankur-gupta-f1 | fine :| | |
| 19:14:09 | reedip | what ?? dissapointed ?? :D | |
| 19:16:42 | ankur-gupta-f1 | never :P | |
| 19:17:51 | reedip | Updated the sheet, now we are 42% complete ! | |
| 19:17:53 | mordred | thingee: it's a new flag and I just didn't get through checking all of them - not having a setting should be safe, since that's the current "I dunno, you go figure it out" situation. adding positive attestations should allow us to optimize calling code to not have to figure out | |
| 19:17:56 | mordred | thingee: I hope | |
| 19:18:53 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Pass ironic microversion through from api_version https://review.openstack.org/439118 | |
| 19:18:55 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Add support for bailing on invalid service versions https://review.openstack.org/439119 | |
| 19:18:56 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Add helper factory functions for use in OpenStack services https://review.openstack.org/439120 | |
| 19:21:57 | rabel_b1 | hi there, is this the right place for questions regarding the development of python-openstackclient? | |
| 19:22:09 | ankur-gupta-f1 | rabel_b1: indeed. fire away | |
| 19:22:13 | reedip | rabel_b1 : fire | |
| 19:22:16 | reedip | hehehe | |
| 19:23:19 | rabel_b1 | I submitted a patch a few days ago and it got merged. https://review.openstack.org/#/c/437335/ Now I wanted to ask if I have to change something in the docs, too | |
| 19:24:08 | ankur-gupta-f1 | rabel_b1: no need. You just changed the logic of an existing command | |
| 19:24:17 | reedip | Not for this | |
| 19:24:30 | reedip | ankur-gupta-f1 : you are beating me on the keyboard ! | |
| 19:24:38 | rabel_b1 | :D | |
| 19:25:18 | rabel_b1 | what about doc/source/command-object/image.rst ? | |
| 19:25:44 | rabel_b1 | shouldn't it be adjusted to what openstack help image create gives? | |
| 19:26:39 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Add helper factory functions for use in OpenStack services https://review.openstack.org/439120 | |
| 19:27:53 | jkilpatr | hey I'm getting keystone errors when running some openstack sdk calls, but I'm authed as admin so I'm trying to figure out if it's a perms issue or if the api call (bare_metal.node_update(id, **kwargs)) is the culprit. | |
| 19:28:37 | jkilpatr | oh I guess I should say that a 400 is being returned by keystone, which is odd because I'm just using the oepnstack sdk to make the request and all other calls work | |
| 19:29:06 | jkilpatr | code here https://github.com/jkilpatr/ostag | |
| 19:29:08 | briancurtin | there is no node_update method in the bare_metal part of openstacksdk | |
| 19:29:43 | jkilpatr | https://developer.openstack.org/sdks/python/openstacksdk/users/proxies/bare_metal.html | |
| 19:29:45 | jkilpatr | sorry update_node | |
| 19:30:24 | briancurtin | i guess you’ll need some sort of logs or example code. i don’t know how/why a 400 would be returned by whatever it is that you’re doing | |
| 19:30:54 | rabel_b1 | reedip, ankur-gupta-f1: what about doc/source/command-object/image.rst ? | |
| 19:31:00 | rabel_b1 | shouldn't it be adjusted to what openstack help image create gives? | |
| 19:31:27 | briancurtin | jkilpatr: enable the logs and then run what you’re doing: https://developer.openstack.org/sdks/python/openstacksdk/users/utils.html#openstack.utils.enable_logging | |
| 19:31:29 | reedip | rabel_b1 : Just min | |
| 19:31:45 | jkilpatr | briancurtin, ok | |
| 19:32:06 | stevemar | ankur-gupta-f1: should i convert the spread sheet to a csv and upload it now? | |
| 19:32:26 | ankur-gupta-f1 | stevemar: yea and add me so i can double check. but it looks updated thus far | |
| 19:32:27 | reedip | No need to update the rst but a release note may be required to specify that now -file and --volume cannot be specified each other | |
| 19:32:40 | briancurtin | jkilpatr: or does the exception message say what exactly about the request returning a 400 is bad? I’m guessing we’re probably sending something we shouldnt | |
| 19:33:03 | briancurtin | (that might have to come out of the logs though) | |
| 19:38:59 | reedip | is the gate OK ? Its taking longer than avg to process patches | |
| 19:39:52 | rabel_b1 | reedip : the behaviour actually did not change. it was not possible before my patch either. but now it is handled by the exclusive_group. the only real difference is, that now --file and --volume are shown as exclusive in the "openstack help image create" output. that's why I thought about changing the rst as well, so it fits again. | |
| 19:43:45 | reedip | rabel_b1 : https://github.com/openstack/python-openstackclient/blob/master/doc/source/command-objects/image.rst , yes change it | |
| 19:44:02 | jkilpatr | briancurtin, https://paste.fedoraproject.org/paste/NYk1V7ShVeenT3qbEsl9oV5M1UNdIGYhyRLivL9gydE= | |
| 19:44:09 | jkilpatr | that's the error. | |
| 19:44:24 | rabel_b1 | reedip : thank you | |
| 19:44:40 | jkilpatr | briancurtin, it's caused by these lines | |
| 19:45:19 | jkilpatr | https://paste.fedoraproject.org/paste/kzWamEQ4InhEP5MHqRa~gF5M1UNdIGYhyRLivL9gydE= | |
| 19:46:09 | reedip | rabel_b1 : glad to help | |
| 19:51:35 | briancurtin | jkilpatr: something about your capabilities string makes it not a valid list. i don’t work on ironic and the documentation isn’t very helpful on that (“capabilities” doesnt even appear in the REST API docs). id maybe check with that team to figure out what kind of request should actually be sent | |
| 19:52:00 | dhellmann | dtroyer, stevemar : I just added you to this review to install OSC under python3 by default in python3 jobs: https://review.openstack.org/#/c/439112/ | |
| 19:55:41 | jkilpatr | briancurtin, happens with the same error if I try to lets say modify the node name, https://paste.fedoraproject.org/paste/mihOWmmHMQzcWfSr47zOaF5M1UNdIGYhyRLivL9gydE= | |
| 19:55:59 | jkilpatr | so that's just bare_metal.node_update(uuid, name="test") | |
| 19:56:10 | jkilpatr | nothing fancy, no list shenenegans simpliest possible call | |
| 19:56:35 | rabel_b1 | reedip : is there a way to automatically generate those files? because it looks like the file (image.rst) does not distinguish between v1 v2 for "image set" and "image create" while they actually do differ. | |