| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-03-16 | |||
| 09:12:10 | EmilienM | you know only gin tonic work nowadays | |
| 09:12:12 | cdent | surfboards and wetsuits for everyone! | |
| 09:12:23 | cdent | gunpowder gin | |
| 10:17:56 | openstackgerrit | Merged openstack/keystoneauth master: Update links in README https://review.openstack.org/551694 | |
| 14:17:06 | cdent | elmiko, figleaf is my response to gilles okay? If not, can you also respond please? | |
| 14:17:25 | trevormc | hi all o/ I have a question about the cliff sphinx-extension.. Does it support the ArgparseDefaultHelpFormatter? https://docs.python.org/2/library/argparse.html#argparse.ArgumentDefaultsHelpFormatter | |
| 14:19:46 | figleaf | cdent: will read shortly | |
| 14:19:56 | cdent | thanks | |
| 14:26:10 | figleaf | cdent: ok, read it. I think you strike a good balance between "we are open to the idea" and "none of the cores are interested in doing that work" | |
| 14:26:42 | figleaf | The point about emphasizing email over IRC meetings is important | |
| 14:26:45 | cdent | I'm trying to say "there may be people out there who are open to the idea, it's not us" | |
| 14:28:30 | cdent | thanks for checking it figleaf, i'll take your response as "it's fine" | |
| 14:29:40 | figleaf | cdent: a better tone than I would have come up with, for sure :) | |
| 16:25:24 | lkwan | In the stable/queens branch, since openstacksdk0.11.3 and os-service-types1.1.0 are described in openstack's upper-constraints.txt, | |
| 16:25:25 | lkwan | https://github.com/openstack/requirements/blob/stable/queens/upper-constraints.txt#L297 | |
| 16:25:25 | lkwan | https://github.com/openstack/requirements/blob/stable/queens/upper-constraints.txt#L411 | |
| 16:25:41 | lkwan | if I do > git clone https://git.openstack.org/openstack-dev/devstack -b stable/queens | |
| 16:25:42 | lkwan | We will see it is using openstacksdk-0.12.0 and os_service_types-1.2.0 | |
| 16:25:42 | lkwan | And then stack.sh | |
| 16:26:20 | lkwan | Need some help how to get the older version, how to configure devstack to use openstacksdk===0.11.3 and os-service-types===1.1.0 | |
| 16:26:34 | lkwan | tried to manually install it before stack.sh | |
| 16:27:03 | lkwan | then had a dependency on those two packages. | |
| 16:27:03 | lkwan | indirectly brought in to meet some dependency that didn't get uninstalled, which | |
| 16:27:03 | lkwan | but it seems it's possible there was a package | |
| 16:27:07 | lkwan | Any tip? | |
| 16:31:25 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Fix crashing "console log show" https://review.openstack.org/541609 | |
| 16:35:07 | dhellmann | trevormc : I'm not sure what would be needed to ensure support. How is the output of the sphinx extension different from what you were hoping to see? | |
| 16:40:31 | trevormc | hi dhellman: I think we should allow a custom formatter to be provided for the get_parser method, https://github.com/openstack/cliff/blob/master/cliff/command.py#L158 | |
| 16:40:55 | dhellmann | trevormc : which command are you working on? | |
| 16:42:09 | trevormc | dhellman: not any upstream at the moment :( | |
| 16:42:37 | dhellmann | ok. what is it that you're trying to do? maybe there's another way. | |
| 16:42:56 | dhellmann | I'm not saying no to making that class configurable, just trying to understand the root of the request you have | |
| 16:43:49 | trevormc | dhellman: yeah i understand, thanks for your time. I'm looking to add the ArgumentDefaultsHelpFormatter which will print the default value along with the help message. | |
| 16:44:09 | dhellmann | ok, so you just want the default value to show up in the help text? | |
| 16:44:25 | trevormc | dhellman, yeah it was a small improvement. | |
| 16:44:28 | dhellmann | ok | |
| 16:44:40 | dhellmann | if the string %(default)s appears in your help text, it will be filled in automatically | |
| 16:45:01 | dhellmann | so you can do something like help='this option is boring. It defaults to %(default)s.' | |
| 16:45:17 | trevormc | dhellman, yeah that's another approach, I can use that. Thanks :) | |
| 16:45:37 | dhellmann | that should help in the short term. in the longer term, we could add the default info to what the existing smart help thing does | |
| 16:45:57 | dhellmann | I think that is mostly about reformatting text to fit the width of the screen now, but there's no reason we couldn't extend it | |
| 16:46:23 | dhellmann | we would want to make it not add the default string if '%(default)s' appears in the help text, so we maintain backwards compatibility | |
| 16:48:11 | openstackgerrit | Colleen Murphy proposed openstack/python-openstackclient master: Correct application credential usage doc https://review.openstack.org/553852 | |
| 16:48:31 | trevormc | dhellman: ok I will think about proposing something similar to that if we *really* want it, but I think we'll be ok. | |
| 16:48:46 | dhellmann | trevormc : ok, that sounds good. | |
| 16:50:35 | frickler | lkwan: are you using the stable/queens branch of devstack? if not, then that is probably the solution | |
| 16:50:56 | lkwan | I am using stable/queens | |
| 16:52:00 | frickler | lkwan: and you are using a fresh instance? or might you have the newer versions preinstalled? | |
| 16:52:48 | frickler | iiuc devstack will not downgrade things that otherwise satisfy the constraints | |
| 16:53:22 | lkwan | Uninstall all packages | |
| 16:53:22 | lkwan | unstack.sh | |
| 16:53:22 | lkwan | reclone with -b stable/queens | |
| 16:53:22 | lkwan | I did master before but | |
| 16:53:23 | lkwan | >sudo pip uninstall -r ~/requirements.txt.all -y | |
| 16:53:27 | lkwan | What else I need to have a clean env? | |
| 16:54:02 | frickler | lkwan: using a fresh instance is the safest solution | |
| 16:54:03 | lkwan | I also manually uninstall them and install the previous versions I wanted | |
| 16:54:21 | lkwan | I am using a fresh instance with stable/queens | |
| 16:55:12 | lkwan | PIP_UPGRADE=False | |
| 16:55:12 | lkwan | even I tried | |
| 16:55:13 | lkwan | GITBRANCH["python-openstacksdk"]=0.11.3 | |
| 16:55:28 | lkwan | in the local.conf but it got upgraded | |
| 16:55:38 | lkwan | to 0.12.0 | |
| 16:57:00 | frickler | hmm, then we may have a devstack bug, probably best if you create one on launchpad and describe the steps to reproduce | |
| 17:02:10 | openstackgerrit | James Penick proposed openstack/python-openstackclient master: Stop prefetching images in server list https://review.openstack.org/553577 | |
| 17:20:40 | EmilienM | hello, can I have some reviews on https://review.openstack.org/#/c/553374/ - thanks! | |
| 17:25:52 | openstackgerrit | Merged openstack/python-openstackclient master: Fix typo in 'floating ip associate' command and doc https://review.openstack.org/553603 | |
| 17:33:09 | openstackgerrit | James Penick proposed openstack/python-openstackclient master: Stop prefetching images in server list https://review.openstack.org/553577 | |
| 18:11:10 | trevormc | dhellman: after applying what you recommended, the output on the cli does print the default value but it doesn't show up in the built docs, instead I get (default: %(default)s) | |
| 18:12:46 | trevormc | (default: %(default)s) is what I included in the help strings, im not asking for anything, just thought you should know. | |
| 18:27:13 | dhellmann | trevormc: hmm, I think I would count that as a bug against cliff's sphinxext. I'd be happy to work with you on a fix if you're interested in working on it. | |
| 18:28:45 | lkwan | frickler: recreate a new machine and it does install with the proper version. So ./unstack.sh and sudo pip uninstall -r ~/requirements.txt.all -y is not enough to clean up things | |
| 18:29:03 | lkwan | thanks anyway... | |
| 20:25:58 | dtroyer | lkwan: FWIW, clean.sh gets more aggressive in removing leftover things in DevStack. It may not have been enough for what you needed (it doesn't remove dependant packages for example) but would have been the next thing to try… | |
| 20:26:57 | dtroyer | but as you found, it is hard to beat a shiny new VM for these problems | |
| 20:28:54 | openstackgerrit | Merged openstack/python-openstackclient master: Correct application credential usage doc https://review.openstack.org/553852 | |
| #openstack-sdks - 2018-03-18 | |||
| 09:57:04 | openstackgerrit | Jens Harbott (frickler) proposed openstack/keystoneauth master: Be more verbose when version discovery fails https://review.openstack.org/554044 | |
| 17:09:07 | ivantretyak | Hi everyone | |
| 17:13:33 | ivantretyak | Hi, bobh | |
| 17:48:46 | openstackgerrit | Merged openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/549536 | |
| 19:12:37 | openstack | Launchpad bug 1756723 in python-openstackclient "Network commands ignore insecure option" [Undecided,New] | |
| 19:12:37 | frickler | dtroyer: mordred: I did a fair amount of debugging on this one, but gave up, lost in the maze between client, sdk and osc-lib https://bugs.launchpad.net/python-openstackclient/+bug/1756723 | |
| 19:15:53 | frickler | what confuses me most is why you have both config and session as arguments here, when the former always seems to take precedence http://git.openstack.org/cgit/openstack/osc-lib/tree/osc_lib/clientmanager.py#n224 | |
| 19:16:25 | frickler | if I remove the config and just leave the session argument, the issue gets resolved | |
| #openstack-sdks - 2018-03-19 | |||
| 11:54:32 | openstackgerrit | Merged openstack/python-openstacksdk master: Fix devstack tests https://review.openstack.org/551792 | |
| 12:50:21 | openstackgerrit | Jacky Hu proposed openstack/python-openstacksdk master: Allow cascade deletion of load balancer https://review.openstack.org/553404 | |
| 13:14:47 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Use define version parameter replace service.version https://review.openstack.org/554223 | |
| 13:50:46 | openstackgerrit | Ed Leafe proposed openstack/api-wg master: Break up the HTTP guideline into smaller documents https://review.openstack.org/554234 | |
| 14:00:28 | elmiko | cdent: missed your ping on friday, where was that meesage? | |
| 14:01:08 | cdent | elmiko: sigs list and dev list | |
| 15:23:23 | EmilienM | dtroyer: hey, could I get a review on https://review.openstack.org/#/c/553374/ please? | |
| 16:03:18 | elmiko | cdent: weird... somehow it dropped through my filter | |
| 16:03:46 | cdent | elmiko: gilles origina message was cc'd to all of us, which might have thrown things off | |
| 17:22:23 | openstackgerrit | Ed Leafe proposed openstack/api-wg master: Break up the HTTP guideline into smaller documents https://review.openstack.org/554234 | |
| 20:29:41 | dhellmann | dtroyer : if we're going to do https://review.openstack.org/#/c/538335/ we probably want to do it early in the cycle so we can deal with anything it breaks | |
| 20:31:00 | dtroyer | dhellmann: thanks for the reminder, agreed | |
| 20:31:25 | elmiko | cdent: yup, caught up now | |
| 20:31:35 | cdent | elmiko: cool, thanks | |
| 20:31:47 | elmiko | i would have replied as i started looking at mugsie's stuff and hope to be up to speed with it by the meeting this week | |
| 20:31:50 | dhellmann | dtroyer : I'm not even sure who else has +2 there any more to ask for for reviews. | |