| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-03-16 | |||
| 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. | |
| 20:32:07 | elmiko | but, i don't think that changes any of the stuff you said, which btw i ++ | |
| 20:33:31 | cdent | elmiko: I'm relieved. I was trying to be balanced but honest | |
| 20:42:28 | openstackgerrit | Mohammed Naser proposed openstack/os-client-config master: Add image_format for VEXXHOST profile https://review.openstack.org/554351 | |
| 20:45:22 | mnaser | could a very kind sdk-core have a quick eye on this simple change ^ | |
| 20:51:07 | elmiko | cdent: yeah, i'm not really sure what more we can do. i think providing the venue and helping keep the ball rolling is a big part of our goal, but like you suggested, we can't really assign work to people. the api schema topic is not new | |
| 20:57:28 | elmiko | cdent edleafe dtantsur|afk re: meeting times and apac friendliness. i wonder, if instead of changing the meeting time, we could offer to hold an office hours time slot that would be more appropriate? | |
| 20:57:57 | elmiko | like, i'm just wondering what can we do to help Gilles out and also not set ourselves up for failure in the meeting dept. | |
| 20:58:15 | cdent | elmiko: I think an office hour is likely a good idea anway, but I think the core issue is that things need to happen asynch if they want to happen | |
| 20:58:25 | elmiko | ++ | |
| 20:59:01 | elmiko | i feel like we have a great potential contributor/participant in Gilles, but for lack of hours in the day we can't make something happen. i'm just spitballing about how we help | |
| 20:59:26 | elmiko | the other part too is what does Gilles want as output? | |
| 21:00:53 | cdent | yes, that's what my final question to him was sort of about | |
| 21:08:13 | elmiko | hopefully it all works out, i'd like to see the api-schema stuff pick up some momentum | |
| 21:13:30 | cdent | elmiko: you should probalby tell Gilles that, otherwise it might seem he's standing sort of solo, which is not the case | |
| 21:23:28 | elmiko | cdent: ack, i will reply. any suggestions for replying from a digest copy of the email? | |
| 21:24:39 | elmiko | ah, nvm. i'll reply the individual email | |
| 21:24:47 | cdent | you have both? | |
| 21:26:45 | elmiko | yeah, i think my email filter separated them out somehow | |
| 21:26:56 | elmiko | (i just switched to our new hosted email, it's driving me nuts) | |
| 21:28:08 | elmiko | well, we'll see if you actually get it lol. bbl, dinner | |
| #openstack-sdks - 2018-03-20 | |||
| 00:21:58 | EmilienM | hi, any chance to have a review on https://review.openstack.org/#/c/553374/ please | |
| 02:50:08 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Fix 'block_store' aliases define error https://review.openstack.org/554403 | |
| 07:09:02 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Fix 'block_store' aliases define error https://review.openstack.org/554403 | |
| 07:25:15 | openstackgerrit | Peter Balogh proposed openstack/python-openstacksdk master: Fix TypeError in case of FloatingIP add and remove https://review.openstack.org/552112 | |
| 07:26:56 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Fix 'block_store' aliases define error https://review.openstack.org/554403 | |
| 10:28:06 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Use defined version instead of service.version parameter https://review.openstack.org/554223 | |
| 12:36:21 | mugsie | elmiko: if you have any questions, please give me a ping :) | |
| 12:38:52 | elmiko | mugsie: ack, will do! | |
| 12:45:18 | cdent | elmiko, edleafe, anything blocking merging those microversion-parse changes? You guys +1d, but you should have the power to +2 if you want. I'd like to release a new version, even if its not yet perfect (as long as it doesn't break stuf, that's good enough) | |
| 12:49:28 | elmiko | cdent: nothing blocking from me | |
| 12:49:52 | elmiko | gimme like an hour and i can start adding the +2s | |
| 12:50:01 | elmiko | unless edleafe gets to it first ;) | |
| 12:50:16 | cdent | cool, thanks, no huge rush, just got reminded of it by my to do list | |
| 12:59:17 | elmiko | ack | |
| 13:08:45 | edleafe | cdent: elmiko: I had one little comment, but added a +2. elmiko, if you agree, wanna push it through? | |
| 13:09:50 | elmiko | edleafe: sounds good! | |
| 13:53:25 | elmiko | cdent: i +w to the 2 that had other support | |
| 13:53:38 | elmiko | i'd prefer more than just my lone review on these: | |
| 13:53:40 | elmiko | https://review.openstack.org/#/c/550251/3 | |
| 13:53:51 | elmiko | https://review.openstack.org/#/c/550263/3 | |
| 13:54:01 | elmiko | https://review.openstack.org/#/c/550264/3 | |
| 13:54:27 | openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Do not use redundant attr check for keys in fold_headers https://review.openstack.org/554547 | |
| 13:55:11 | cdent | elmiko: yeah. edleafe I added another patch to the end fixing the bug. | |
| 13:55:29 | elmiko | cool | |
| 13:55:36 | elmiko | is that the one you just posted? | |
| 13:55:39 | cdent | yeah | |
| 13:55:44 | elmiko | looked sensible | |
| 13:59:29 | edleafe | cdent: cool. I would have just fixed it in place, but... meh | |
| 13:59:43 | openstackgerrit | Merged openstack/microversion-parse master: Provide a helper method to get headers from environ https://review.openstack.org/495356 | |
| 13:59:49 | cdent | edleafe: for a change that didn't check that method at all? dirty! | |
| 14:00:31 | openstackgerrit | Merged openstack/microversion-parse master: Import functions from placement https://review.openstack.org/496212 | |
| 14:01:17 | edleafe | Guess I'm a dirty guy | |
| 14:02:18 | cdent | Newflash: edleafe states well known fact, wins t-shirt | |
| 14:02:46 | edleafe | \o/ | |
| 14:02:56 | edleafe | My wardrobe needs sprucing up | |
| 14:04:12 | openstackgerrit | Jens Harbott (frickler) proposed openstack/keystoneauth master: Be more verbose when version discovery fails https://review.openstack.org/554044 | |