| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-02-23 | |||
| 01:14:07 | reedip_1 | ankur-gupta-f1 ^^ | |
| 01:16:29 | ankur-gupta-f4 | it needs a core review. Looks good to me. U need huanxuan or dtroyer | |
| 01:17:35 | ankur-gupta-f4 | reedip_1: just a tiny nit I saw in column_header but thats it | |
| 01:18:27 | reedip_1 | ankur-gupta-f4 : sure, lemme know the issue | |
| 01:18:48 | ankur-gupta-f4 | i commented the line in the patch | |
| 01:18:56 | ankur-gupta-f4 | Project_id to Project ID | |
| 01:21:47 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: OSC Quota List https://review.openstack.org/379813 | |
| 01:22:02 | reedip_1 | donr | |
| 02:14:21 | openstackgerrit | Shan Guo proposed openstack/api-wg master: Fix typo in the doc https://review.openstack.org/436865 | |
| 02:20:54 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient master: Support list commands by group name keyword https://review.openstack.org/436806 | |
| 02:21:58 | openstackgerrit | Merged openstack/python-openstacksdk master: Remove unsupported telemetry create_sample method https://review.openstack.org/437053 | |
| 04:13:37 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient master: Support list commands by group name keyword https://review.openstack.org/436806 | |
| 05:38:59 | openstackgerrit | Sindhu Devale proposed openstack/python-openstackclient master: OSC Quota List https://review.openstack.org/379813 | |
| 05:53:29 | openstackgerrit | Sindhu Devale proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 10:27:24 | openstackgerrit | Huanxuan Ao proposed openstack/python-openstackclient master: Fix "security group list" command to display project ID properly https://review.openstack.org/437320 | |
| 11:20:30 | openstackgerrit | David Rabel proposed openstack/python-openstackclient master: openstack image create : --file and --volume exclude each other https://review.openstack.org/437335 | |
| 13:05:37 | openstackgerrit | Huanxuan Ao proposed openstack/python-openstackclient master: Revert unit tests for quota commands https://review.openstack.org/437366 | |
| 13:10:35 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet-pools https://review.openstack.org/437367 | |
| 13:14:57 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet-pools https://review.openstack.org/437367 | |
| 15:03:39 | openstackgerrit | David Rabel proposed openstack/python-openstackclient master: openstack image create : --file and --volume exclude each other https://review.openstack.org/437335 | |
| 15:29:16 | elmiko | etoews, cdent, edleafe, no meeting this week, right? | |
| 15:29:40 | etoews | elmiko: i expect not | |
| 15:29:51 | openstackgerrit | Ankur proposed openstack/python-openstackclient master: OSC Network Flavor Profile https://review.openstack.org/396783 | |
| 15:29:58 | cdent | elmiko: no sir | |
| 15:30:12 | elmiko | ack, thanks | |
| 15:31:09 | cdent | elmiko: I announced the no meeting in the newsletter, so hopefully people will know not to show up | |
| 15:31:26 | elmiko | cdent: yeah, i thought it was in there. just wanted to dbl check | |
| 15:31:33 | openstackgerrit | Ankur proposed openstack/python-openstackclient master: Port set/unset SecGroup Bug Fix https://review.openstack.org/430087 | |
| 15:53:06 | ankur-gupta-f4 | dtroyer: https://review.openstack.org/#/c/387611/ amotoki_ : has made some valid points against how we decided to run the Network Agent advanced commands. Take a look when you have a chance. Thanks | |
| 15:58:21 | swatson_ | Hey channel. I'm working on the magnum OSC plugin and can't seem to get things to work. I can run setup.py just fine for both OSC and my magnum client, but my plugin changes don't seem to do anything when I run OSC. | |
| 15:58:49 | swatson_ | dtroyer ^ if you're up for letting me pick your brain today :) | |
| 15:59:30 | ankur-gupta-f4 | swatson_: do you have something up on territory? | |
| 15:59:32 | ankur-gupta-f4 | gerrit* | |
| 16:00:29 | swatson_ | ankur-gupta-f4: Right now I'm just trying what we have on python-magnumclient right now, just to see if what we have is set up correctly in the first place | |
| 16:00:52 | swatson_ | I see our global option coming through that we have in our plugin.py, but nothing about the one other command we've exposed | |
| 16:06:25 | ankur-gupta-f4 | so i just pulled down magnum client. ran python setup.py in it. Then pulled fresh OSC master and ran setup.py again. The single command exposed is cluster template list, which shows up for me (although I won't be able to run it since I don't have magnum installed) | |
| 16:06:30 | ankur-gupta-f4 | swatson_: ^ | |
| 16:07:21 | swatson_ | ankur-gupta-f4 hm. Did you run "python setup.py develop" for both of them? | |
| 16:07:33 | ankur-gupta-f4 | yes | |
| 16:10:02 | swatson_ | ankur-gupta-f4 I'm still not getting it. What did you run in OSC to see the magnum command? | |
| 16:10:54 | ankur-gupta-f4 | openstack help | grep cluster | |
| 16:11:26 | swatson_ | ah, were you seeing the dataprocessing cluster template list? | |
| 16:12:24 | ankur-gupta-f4 | http://paste.openstack.org/show/600255/ | |
| 16:12:51 | ankur-gupta-f4 | swatson_: making coffee gimme a sec | |
| 16:14:44 | swatson_ | ankur-gupta-f4: Hmm...I'm getting this for my cluster entries: http://paste.openstack.org/show/600256/ | |
| 16:15:06 | swatson_ | 0 cluster template apart from the dataprocessing ones :\ | |
| 16:18:25 | ankur-gupta-f4 | when running the open stack client? Currently there is only 1 command implemented for magnum via plugin. https://github.com/openstack/python-magnumclient/tree/master/magnumclient/osc/v1 . | |
| 16:18:58 | swatson_ | Right. And even with both master branches updated on my machine I don't see it coming up after running setup for both projects | |
| 16:19:20 | swatson_ | ankur-gupta-f4 the paste was via "openstack help | grep cluster" | |
| 16:20:45 | ankur-gupta-f4 | :( time to pick dean's brain | |
| 16:23:40 | ankur-gupta-f4 | swatson_: master branch with no local changes | |
| 16:23:47 | swatson_ | Correct | |
| 16:25:13 | ankur-gupta-f4 | 😕 | |
| 16:27:42 | dtroyer | fresh venv, "pip install -e ." in both master osc and magnumclient | |
| 16:27:50 | dtroyer | openstack help | grep cluster | |
| 16:27:51 | dtroyer | Could not load EntryPoint.parse('cluster_template_list = magnumclient.osc.v1.cluster_templates:ListTemplateCluster') | |
| 16:28:06 | dtroyer | so I'm clearly not in the same place either of you guys are yet | |
| 16:28:29 | swatson_ | dtroyer Just to verify: you pulled down python-magnumclient? | |
| 16:28:45 | dtroyer | yes, from git.o.o | |
| 16:29:06 | dtroyer | I see the same thing installing both from PyPI | |
| 16:29:07 | swatson_ | OK, just double-checking. We have a magnumclient directory in magnum | |
| 16:29:18 | dtroyer | ah, that's confusing | |
| 16:29:53 | swatson_ | I agree. We have plans for it but that's after I can get this plugin working :P | |
| 16:30:04 | dtroyer | full disclosure, this is on OS/X 10.10.5 | |
| 16:30:23 | dtroyer | but that hasn't been an issue for a long time | |
| 16:30:51 | swatson_ | I would hope that doesn't have much of an impact for this | |
| 16:31:03 | swatson_ | I'm on Ubuntu 14.04 with mine | |
| 16:31:20 | swatson_ | Though I have had to use sudo on my setup commands because of complaints about ownership of the /usr/local/lib dir | |
| 16:31:39 | swatson_ | specifically /usr/local/lib/python2.7/dist-packages | |
| 16:32:06 | dtroyer | that is normal if you are installing into the system python and not a venv | |
| 16:32:18 | ankur-gupta-f4 | i avoided pip and cloned repos from git. Don't have magnum installed, just python-magnumclient | |
| 16:32:56 | dtroyer | you shouldn't need magnum at all | |
| 16:33:34 | swatson_ | so far Ankur is the only one who's gotten the expected result from this | |
| 16:34:06 | dtroyer | I get the same thing on 14.04 in a venv | |
| 16:34:14 | dtroyer | from PyPI | |
| 16:34:27 | swatson_ | The EntryPoint.parse eror? | |
| 16:34:31 | swatson_ | s/eror/error | |
| 16:34:37 | dtroyer | is magnumclient master different from latest release in this area? | |
| 16:34:39 | dtroyer | yes | |
| 16:35:03 | swatson_ | I don't believe it's different. The cluster template list command was merged 4 weeks ago | |
| 16:36:02 | dtroyer | heh, you may want to reconsider using oslo_log in clients | |
| 16:36:20 | swatson_ | Dare I ask why? | |
| 16:36:29 | dtroyer | most of oslo makes assumptions about being used in servers, not clients | |
| 16:36:40 | swatson_ | ah | |
| 16:36:49 | dtroyer | sometimes it is safe, but as a default it may not be | |
| 16:37:02 | dtroyer | so you need to verify, and then hope it doesn't change out from under you | |
| 16:37:12 | dtroyer | what do you get from oslo_log here? | |
| 16:37:36 | swatson_ | I'm not sure, to be honest. This was implemented some time before I joined :P | |
| 16:39:21 | dtroyer | ok, just an observation | |
| 16:39:50 | swatson_ | I'll bring it up in the next magnum meeting :) | |
| 16:42:05 | swatson_ | In any case I'm grabbing the latest python3 so I can try a venv on this | |
| 16:43:54 | dtroyer | so, turns out my problem was in fact oslo_log | |
| 16:44:10 | dtroyer | it isn't in requirements.txt for magnumclient so was not installed | |
| 16:44:12 | dtroyer | installing it and I see the command in help | |
| 16:45:01 | swatson_ | I might need to add that to requirements.txt then, good catch | |
| 16:45:06 | swatson_ | That means there's something wrong on my system then | |
| 16:45:09 | dtroyer | or remove the import | |
| 16:45:31 | dtroyer | I don't see anything that need it over the syslib logging | |
| 16:45:42 | swatson_ | or that | |
| 16:47:34 | dtroyer | fwiw, python-manilaclient is the only other client that uses oslo_log | |