| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-02-28 | |||
| 16:47:02 | reedip | dtroyer : so , we shouldnt use += in Set attributes ( sounds like a doc update :P ) | |
| 16:47:08 | reedip | ?? | |
| 16:47:08 | dtroyer | I'm trying to make the handling of these arguments clearer, it took me a while to understand what was happening before | |
| 16:47:20 | dtroyer | maybe not in this case, no | |
| 16:47:35 | reedip | dtroyer : so this is a case by case basis ? | |
| 16:47:44 | dtroyer | at least when operating on an attribute obtained from an SDK object | |
| 16:48:20 | dtroyer | I think that is the distinction; if we used a copy of an object attribute it might have worked | |
| 16:53:39 | reedip | dtroyer : ok ... | |
| 16:54:50 | dtroyer | hmmmm, of the 3 options pairs I am looking at, security groups was the only one to not iterate over the existing list | |
| 16:57:34 | reedip | the only difference which I found was that we are using the client.find_xxx function for SG | |
| 16:57:58 | reedip | dtroyer: for fixed-ip and allowed-address-pair , we didnt execute the find function | |
| 16:58:21 | reedip | can that be of some impact , dtroyer ? | |
| 16:58:24 | dtroyer | I don' tthink that is involved, that is looping over the 'new' values | |
| 16:59:46 | dtroyer | actually, I think using [].extend() rather than += will be enough of a change | |
| 16:59:51 | reedip | dtroyer : ok, because += was used with fixed-ips as well, and if it was updated successfully, then maybe something else is wrong | |
| 17:00:18 | dtroyer | fixed_ips never directly re-used the list from the resource object, it iterated over it | |
| 17:00:29 | reedip | dtroyer : ok | |
| 17:00:30 | dtroyer | that forced __setitem__() to be called | |
| 17:00:38 | reedip | yeah , makes sense | |
| 17:01:03 | dtroyer | so I think we'll do both, it seems clearer to me | |
| 17:03:32 | reedip | dtroyer : ok :) | |
| 17:03:56 | reedip | sindhu :still causes an issue . Did you run tox -v -e functional openstackclient.tests.functional.network.v2.test_floating_ip.FloatingIpTests.test_floating_ip_set_and_unset_port | |
| 17:04:08 | reedip | sindhu : can you tell me the output of ^^ ? | |
| 17:17:16 | sindhu | reedip: y, what's wrong? | |
| 17:18:27 | reedip | sindhu : I am still getting a failure, thats why asking you if it is passing in your system | |
| 17:19:37 | reedip | sindhu : can u just run and check if it giving a positive result in your setup ? | |
| 17:21:41 | sindhu | reedip: something is wrong with my local it fails in the setup itself. Let me get a new env n check again. Thanks for looking :) | |
| 17:22:05 | sindhu | reedip: what error are u getting? | |
| 17:22:08 | reedip | sindhu : NP , I am just trying to close the NeutronClient CLI migration for now :) | |
| 17:22:58 | sindhu | reedip: I m not sure if cleanup is working the way it should | |
| 17:23:42 | reedip | sindhu : It isnt , I brought the router delete line to Line #234 | |
| 17:28:52 | sindhu | reedip: and it is working now? | |
| 17:29:14 | reedip | No, a different error is occurring, thats what I a trying to resolve | |
| 17:29:27 | sindhu | reedip: what's that ? | |
| 17:29:53 | reedip | unicode is not callable, str is not callable :) | |
| 17:30:03 | reedip | I have been trying to fix this since this evening :) | |
| 17:30:10 | reedip | my brain is not working with me ! | |
| 17:30:16 | sindhu | yup, that's the same failure even in gate | |
| 17:30:56 | wklm | hey guys, i'm having troubles installing python sdk on centos. I'm getting the following error: Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-gyuzq2sj/cliff/ How can I deal with that? | |
| 17:31:06 | sindhu | reedip: Ohhhh I think I got it !!!!! | |
| 17:31:21 | briancurtin | wklm: how are you installing it? | |
| 17:31:35 | reedip | sindhu : whats that ? | |
| 17:32:05 | sindhu | reedip: so in all the cleanup, I have missed self.openstack | |
| 17:32:13 | sindhu | reedip: will fix it now | |
| 17:32:25 | reedip | sindhu : thats not necessary , is it? | |
| 17:32:26 | wklm | briancurtin: simply: pip install python-openstackclient | |
| 17:32:35 | wklm | being root | |
| 17:33:23 | openstackgerrit | Sindhu Devale proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 17:33:48 | sindhu | reedip: it is, I have updated | |
| 17:33:58 | reedip | sindhu : testing the same | |
| 17:34:03 | reedip | I have made some other changes as well | |
| 17:34:05 | sindhu | reedip: can u check now with the latest patch | |
| 17:34:23 | reedip | sindhu : sure, gimme a minute. My changes are just running right now | |
| 17:34:31 | sindhu | reedip: sure | |
| 17:34:32 | briancurtin | wklm: someone else will have to take a look, i thought you were installing the sdk | |
| 17:35:09 | wklm | briancurtin: is there a more proper channel for it, or am i right here? :) | |
| 17:35:16 | briancurtin | fine here | |
| 17:35:26 | briancurtin | i just don’t know about that | |
| 17:39:04 | reedip | sindhu : my patch failed :P Now trying yours | |
| 17:39:23 | wklm | here more specific error stack trace: | |
| 17:39:24 | wklm | Traceback (most recent call last): File "/tmp/pip-build-mgg6jby7/cliff/.eggs/pbr-1.10.0-py3.4.egg/pbr/core.py", line 111, in pbr attrs = util.cfg_to_args(path, dist.script_args) File "/tmp/pip-build-mgg6jby7/cliff/.eggs/pbr-1.10.0-py3.4.egg/pbr/util.py", line 248, in cfg_to_args kwargs = setup_cfg_to_setup_kwargs(config, script_args) File "/tmp/pip-build-mgg6jby7/cliff/.eggs/pbr-1.10.0-py3.4. | |
| 17:39:30 | sindhu | reedip: ok, thanks :) | |
| 17:45:20 | reedip | sindhu : okay , your patch also didnt work on my system | |
| 17:45:25 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Reorganize network docs https://review.openstack.org/439074 | |
| 17:45:40 | sindhu | reedip: Ah really?? what does it say now? | |
| 17:46:05 | reedip | Just a min | |
| 17:46:10 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Reorganize workflow docs https://review.openstack.org/439028 | |
| 17:46:32 | reedip | http://paste.openstack.org/show/600823/ | |
| 17:46:37 | reedip | sindhu : ^^ | |
| 17:46:46 | reedip | I fixed that in my patch but failed somewhere else | |
| 17:49:10 | reedip | sindhu : this one is really tricky patch for a long time :) | |
| 17:49:30 | sindhu | reedip: lol, yeah !! | |
| 17:49:38 | ankur-gupta-f4 | Yea what is going on haha | |
| 17:50:34 | reedip | ankur-gupta-f4 : a patch !!!! | |
| 17:50:58 | sindhu | reedip: I am going to wait to see what jenkins says | |
| 17:51:05 | reedip | sindhu : sure :) | |
| 17:51:15 | reedip | till then I will make my patch :D | |
| 17:54:33 | sindhu | ankur-gupta-f4: I had missed a parameter in cleanup | |
| 18:00:23 | ankur-gupta-f1 | stevemar: are you going to do the network osc mapping? else I can | |
| 18:02:32 | dasanind | ankur-gupta-f1: I started the network osc mapping here https://docs.google.com/spreadsheets/d/18ZtWC75BNCwFqLfFpCGGJ9uPVBvUXX0xuXP1yYG0NDA/edit?ts=58ab0c43#gid=527671830 | |
| 18:02:48 | reedip | hey dasanind :) | |
| 18:03:14 | reedip | BTW u missed tap-as-a-service CLIs in this :) | |
| 18:03:23 | dasanind | hi reedip : | |
| 18:03:36 | dasanind | reedip: oops | |
| 18:03:42 | reedip | no issues :) | |
| 18:03:44 | dasanind | reedip: can you nplease add it | |
| 18:04:05 | reedip | I cannot until I have write access :P :D | |
| 18:04:21 | reedip | sindhu : My patch succeeded :D | |
| 18:04:51 | dasanind | reedip: I can add you what is your email id? | |
| 18:05:06 | reedip | I requested for the update | |
| 18:05:27 | reedip | didnt u get the request ? | |
| 18:07:10 | ankur-gupta-f4 | Yea i filled it | |
| 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 | |