| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-02-23 | |||
| 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 | |
| 16:47:54 | dtroyer | ugh, they import oslo.config too | |
| 16:48:20 | swatson_ | Copy pasting is good, right? Right?! | |
| 16:49:55 | swatson_ | new venv, ran setup.py for both projects, no dice | |
| 16:49:57 | dtroyer | heh, OpenStack does that? | |
| 16:50:18 | dtroyer | try using pip | |
| 16:50:31 | dtroyer | IIRC that is the only way we test installing anymore | |
| 16:50:35 | swatson_ | "pip install -e"? | |
| 16:50:45 | dtroyer | yes | |
| 16:50:53 | dtroyer | I think there are some subtle differences | |
| 16:51:05 | ankur-gupta-f4 | pip install -e . | |
| 16:51:20 | swatson_ | ankur-gupta-f4 Thanks for that, was about to ask | |
| 16:52:09 | swatson_ | "error in setup command: Error parsing /home/intel/osc_venv/build/cliff/setup.cfg: SyntaxError: '<' operator not allowed in environment markers | |
| 16:52:09 | swatson_ | " | |
| 16:52:31 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Fix can not set is_default in network https://review.openstack.org/434602 | |
| 16:53:26 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add new parameter "is_default" to Network QoS policy. https://review.openstack.org/432260 | |
| 16:53:40 | swatson_ | dtroyer ankur-gupta-f4 A-ha! Pip did it | |
| 16:54:31 | swatson_ | I went outside my venv and ran "pip install -e ." in python-magnumclient, then did the same in python-openstackclient and the command came up as expected | |
| 16:54:41 | ankur-gupta-f4 | woot woot | |
| 16:54:51 | ankur-gupta-f4 | All board the container train | |
| 16:55:31 | swatson_ | Thanks for the help, guys. I appreciate it | |
| 16:56:26 | ankur-gupta-f4 | np gl | |
| 17:36:34 | swatson_ | Got another question for the channel: Since another group has already used "cluster" for their OSC plugin, magnum needs to differentiate their commands. Is there any kind of guideline for these collisions? i.e. should I just preface each magnum-specific command with "container-infra"? | |
| 17:38:52 | ankur-gupta-f4 | swatson_: what is the service type name? | |
| 17:39:15 | swatson_ | ankur-gupta-f4 I'm not sure what you mean | |