| *** openstack has joined #openstack-sdks | 00:05 | |
| *** boris-42 has quit IRC | 00:12 | |
| *** stevemar has joined #openstack-sdks | 00:23 | |
| *** Qiming has joined #openstack-sdks | 00:27 | |
| *** stevemar has quit IRC | 00:32 | |
| *** shaleh has quit IRC | 00:36 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 00:37 | |
| openstackgerrit | Merged openstack/python-openstackclient: Fix the way we call find_resource when only using ID https://review.openstack.org/202395 | 00:38 |
|---|---|---|
| openstackgerrit | Merged openstack/python-openstackclient: add --project-domain option for user v3 https://review.openstack.org/202645 | 00:38 |
| *** barra204 has quit IRC | 00:39 | |
| *** pm90_ has quit IRC | 01:17 | |
| *** pm90_ has joined #openstack-sdks | 01:36 | |
| *** Yanyanhu has joined #openstack-sdks | 01:37 | |
| openstackgerrit | Qiming Teng proposed stackforge/python-openstacksdk: Add missing docstrings https://review.openstack.org/202841 | 01:46 |
| *** tobe_ has joined #openstack-sdks | 01:49 | |
| *** hitalia has quit IRC | 02:08 | |
| *** chlong-afk has quit IRC | 02:37 | |
| *** chlong has joined #openstack-sdks | 02:42 | |
| *** stevemar has joined #openstack-sdks | 02:44 | |
| *** stevemar has quit IRC | 02:49 | |
| *** pm90_ has quit IRC | 02:54 | |
| *** Qiming has quit IRC | 03:02 | |
| *** boris-42 has joined #openstack-sdks | 03:14 | |
| *** Qiming has joined #openstack-sdks | 03:28 | |
| *** pm90_ has joined #openstack-sdks | 03:28 | |
| *** stevemar has joined #openstack-sdks | 03:46 | |
| openstackgerrit | Merged openstack/python-openstackclient: only return endpoints that have url https://review.openstack.org/201911 | 03:57 |
| *** hitalia has joined #openstack-sdks | 04:09 | |
| *** pm90_ has quit IRC | 04:12 | |
| openstackgerrit | heha proposed openstack/python-openstackclient: Add "project create --or-show" into the Doc In the Doc,there's not information of the "project create --or-show".Add it into the Doc. https://review.openstack.org/202875 | 04:37 |
| *** pm90_ has joined #openstack-sdks | 04:38 | |
| *** britthou_ has joined #openstack-sdks | 04:38 | |
| *** britthouser has quit IRC | 04:40 | |
| *** tobe_ has quit IRC | 04:53 | |
| openstackgerrit | heha proposed openstack/python-openstackclient: Add "project create --or-show" into the Doc In the Doc,there's not information of the "project create --or-show".Add it into the Doc. https://review.openstack.org/202875 | 04:58 |
| *** stevemar_ has joined #openstack-sdks | 05:04 | |
| *** stevemar has quit IRC | 05:07 | |
| *** dguerri` is now known as dguerri | 05:19 | |
| *** boris-42_ has joined #openstack-sdks | 05:20 | |
| *** boris-42 has quit IRC | 05:23 | |
| *** zz_steveortiz has quit IRC | 05:23 | |
| *** zz_steveortiz has joined #openstack-sdks | 05:24 | |
| *** boris-42_ is now known as boris-42 | 05:24 | |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Add functional tests for volume qos https://review.openstack.org/198823 | 05:32 |
| *** pm90__ has joined #openstack-sdks | 05:40 | |
| openstackgerrit | heha proposed openstack/python-openstackclient: Add "--os-show" to "project create" and "role create" In the Doc,there's not information of the "project create --or-show" and "role create --or-show".Add it into the Doc. Fixes Bug1475485 https://review.openstack.org/202886 | 05:43 |
| *** pm90_ has quit IRC | 05:43 | |
| openstackgerrit | Guojian Shao proposed openstack/python-openstackclient: enhance tests for user v3 https://review.openstack.org/202687 | 05:45 |
| *** tobe_ has joined #openstack-sdks | 05:46 | |
| openstackgerrit | Guojian Shao proposed openstack/python-openstackclient: enhance tests for user v3 https://review.openstack.org/202687 | 05:47 |
| openstackgerrit | heha proposed openstack/python-openstackclient: Add "--os-show" to "project create" and "role create" In the Doc,there's not information of the "project create --or-show" and "role create --or-show".Add it into the Doc. Fixes Bug1475485 https://review.openstack.org/202887 | 05:51 |
| *** pm90_ has joined #openstack-sdks | 05:55 | |
| *** pm90__ has quit IRC | 05:59 | |
| *** guojian has joined #openstack-sdks | 06:00 | |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Support multiple volume qos delete https://review.openstack.org/198822 | 06:02 |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: add functional tests for identity v2 https://review.openstack.org/201906 | 06:03 |
| *** stevemar_ has quit IRC | 06:03 | |
| *** stevemar has joined #openstack-sdks | 06:04 | |
| *** stevemar_ has joined #openstack-sdks | 06:06 | |
| *** ParsectiX has joined #openstack-sdks | 06:07 | |
| *** ig0r_ has joined #openstack-sdks | 06:08 | |
| *** stevemar has quit IRC | 06:09 | |
| ParsectiX | Good Morning :) | 06:09 |
| *** stevemar_ has quit IRC | 06:10 | |
| *** terrylhowe has quit IRC | 06:16 | |
| *** tobe_ has quit IRC | 06:16 | |
| openstackgerrit | Merged openstack/python-openstackclient: Drop py33 support for Liberty https://review.openstack.org/201331 | 06:26 |
| *** tobe_ has joined #openstack-sdks | 06:31 | |
| *** karimb has joined #openstack-sdks | 06:32 | |
| *** aufi has joined #openstack-sdks | 06:38 | |
| *** dguerri is now known as dguerri` | 06:40 | |
| *** jaosorior has joined #openstack-sdks | 06:43 | |
| *** ig0r_ has quit IRC | 06:45 | |
| *** ig0r_ has joined #openstack-sdks | 06:49 | |
| *** pm90_ has quit IRC | 06:49 | |
| *** boris-42 has quit IRC | 07:12 | |
| *** hitalia has quit IRC | 07:31 | |
| openstackgerrit | Guojian Shao proposed openstack/python-openstackclient: add functional tests for identity v3 https://review.openstack.org/201574 | 08:09 |
| *** zacksh has quit IRC | 08:10 | |
| *** zacksh has joined #openstack-sdks | 08:13 | |
| openstackgerrit | heha proposed openstack/python-openstackclient: Add "--os-show" to "project create" and "role create" In the Doc,there's not information of the "project create --or-show" and "role create --or-show".Add it into the Doc. Fixes Bug1475485 https://review.openstack.org/202921 | 08:30 |
| openstackgerrit | Guojian Shao proposed openstack/python-openstackclient: enhance tests for user v3 https://review.openstack.org/202687 | 08:46 |
| openstackgerrit | heha proposed openstack/python-openstackclient: Add "--os-show" to "project create". In the Doc,there's not information of the "project create --or-show".Add it into the Doc. Fixes Bug1475485 https://review.openstack.org/202931 | 08:59 |
| openstackgerrit | Guojian Shao proposed openstack/python-openstackclient: add functional tests for identity v2 https://review.openstack.org/201906 | 09:00 |
| *** ParsectiX has quit IRC | 09:00 | |
| *** ParsectiX has joined #openstack-sdks | 09:05 | |
| openstackgerrit | chengkunye proposed openstack/python-openstackclient: add volume type commands in v2 "volume type create and volume type list" https://review.openstack.org/202934 | 09:09 |
| *** Yanyanhu has quit IRC | 09:17 | |
| *** VeggieMeat has quit IRC | 09:18 | |
| *** VeggieMeat has joined #openstack-sdks | 09:18 | |
| *** tobe_ has quit IRC | 09:25 | |
| *** zacksh has quit IRC | 09:25 | |
| *** zacksh has joined #openstack-sdks | 09:26 | |
| *** VeggieMeat has quit IRC | 09:34 | |
| *** VeggieMeat has joined #openstack-sdks | 09:35 | |
| *** Qiming has quit IRC | 10:01 | |
| *** ParsectiX has quit IRC | 10:05 | |
| *** ParsectiX has joined #openstack-sdks | 10:06 | |
| *** ParsectiX has quit IRC | 10:15 | |
| *** openstackgerrit has quit IRC | 10:31 | |
| *** openstackgerrit has joined #openstack-sdks | 10:32 | |
| *** ParsectiX has joined #openstack-sdks | 10:35 | |
| *** jaosorior has quit IRC | 10:36 | |
| *** Qiming has joined #openstack-sdks | 10:53 | |
| *** VeggieMeat has quit IRC | 11:28 | |
| *** VeggieMeat has joined #openstack-sdks | 11:28 | |
| *** terrylhowe has joined #openstack-sdks | 11:30 | |
| *** VeggieMeat has quit IRC | 11:32 | |
| *** VeggieMeat has joined #openstack-sdks | 11:32 | |
| *** VeggieMeat has quit IRC | 11:40 | |
| *** VeggieMeat has joined #openstack-sdks | 11:40 | |
| *** thrash has quit IRC | 12:03 | |
| *** VeggieMeat has quit IRC | 12:08 | |
| *** VeggieMeat has joined #openstack-sdks | 12:08 | |
| *** trown|outttypeww is now known as trown | 12:14 | |
| *** stevemar has joined #openstack-sdks | 12:25 | |
| *** VeggieMeat has quit IRC | 12:26 | |
| *** VeggieMeat has joined #openstack-sdks | 12:26 | |
| *** stevemar has quit IRC | 12:29 | |
| *** guojian has quit IRC | 12:38 | |
| *** guojian has joined #openstack-sdks | 12:39 | |
| *** VeggieMeat has quit IRC | 12:44 | |
| *** VeggieMeat has joined #openstack-sdks | 12:44 | |
| *** thrash has joined #openstack-sdks | 12:47 | |
| *** pm90_ has joined #openstack-sdks | 12:50 | |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Add wait for delete method https://review.openstack.org/193793 | 12:55 |
| *** social has quit IRC | 12:57 | |
| *** edleafe is now known as figleaf | 13:03 | |
| openstackgerrit | Merged openstack/os-client-config: Remove py26 and py33 from tox.ini https://review.openstack.org/202784 | 13:09 |
| *** bnemec has joined #openstack-sdks | 13:11 | |
| openstackgerrit | Lin Hua Cheng proposed openstack/python-openstackclient: Support multiple volume qos delete https://review.openstack.org/198822 | 13:15 |
| *** stevemar has joined #openstack-sdks | 13:20 | |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Add functional tests for volume qos https://review.openstack.org/198823 | 13:21 |
| *** stevemar has quit IRC | 13:25 | |
| openstackgerrit | Brant Knudson proposed openstack/python-openstackclient: Query args for get-only https://review.openstack.org/202748 | 13:25 |
| openstackgerrit | Brant Knudson proposed openstack/python-openstackclient: Revert "Fix the way we call find_resource when only using ID" https://review.openstack.org/203025 | 13:25 |
| *** zz_steveortiz is now known as steveortiz | 13:25 | |
| *** VeggieMeat has quit IRC | 13:26 | |
| openstackgerrit | Brant Knudson proposed openstack/python-openstackclient: Same exception handling for gets() in find_resource https://review.openstack.org/202760 | 13:26 |
| *** VeggieMeat has joined #openstack-sdks | 13:26 | |
| openstackgerrit | Merged openstack/os-client-config: Have service name default to None https://review.openstack.org/201311 | 13:27 |
| openstackgerrit | Merged openstack/os-client-config: Rename 'endpoint_type' to 'interface' https://review.openstack.org/201704 | 13:28 |
| *** dguerri` has quit IRC | 13:37 | |
| *** pm90_ has quit IRC | 13:38 | |
| *** dguerri` has joined #openstack-sdks | 13:40 | |
| *** dguerri` is now known as dguerri | 13:41 | |
| *** dguerri has joined #openstack-sdks | 13:41 | |
| *** Shrews has quit IRC | 13:41 | |
| *** Shrews has joined #openstack-sdks | 13:43 | |
| *** stevemar has joined #openstack-sdks | 13:44 | |
| *** stevemar has quit IRC | 13:48 | |
| *** jaosorior has joined #openstack-sdks | 13:49 | |
| *** stevemar has joined #openstack-sdks | 13:51 | |
| *** stevemar_ has joined #openstack-sdks | 13:52 | |
| *** stevemar has quit IRC | 13:55 | |
| openstackgerrit | Merged openstack/python-openstackclient: Rename endpoint type to interface https://review.openstack.org/198506 | 13:55 |
| *** notmars has joined #openstack-sdks | 13:57 | |
| *** stevemar_ has quit IRC | 13:57 | |
| *** stevemar has joined #openstack-sdks | 13:58 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:06 | |
| *** ParsectiX has quit IRC | 14:12 | |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Support multiple volume qos delete https://review.openstack.org/198822 | 14:14 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Add functional tests for volume qos https://review.openstack.org/198823 | 14:15 |
| *** guojian has quit IRC | 14:21 | |
| *** aufi has quit IRC | 14:26 | |
| openstackgerrit | chengkunye proposed openstack/python-openstackclient: add volume type commands in v2 "volume type create and volume type list" https://review.openstack.org/202934 | 14:52 |
| openstackgerrit | Brant Knudson proposed openstack/python-openstackclient: Same exception handling for gets() in find_resource https://review.openstack.org/202760 | 14:54 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Remove unnecessary test extension mock https://review.openstack.org/199628 | 14:54 |
| stevemar | terrylhowe: do you have any idea whats going on with the patche? | 14:54 |
| stevemar | s | 14:54 |
| stevemar | i keep getting merge conflicts in gerrit | 14:54 |
| terrylhowe | Yeh, I’ve downloaded it twice and it is fine | 14:55 |
| terrylhowe | I’m just going to let it sit a little while and see if those responses are from earlier patches or something | 14:55 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: add functional tests for identity v3 https://review.openstack.org/201574 | 15:00 |
| *** bknudson has joined #openstack-sdks | 15:03 | |
| *** pm90_ has joined #openstack-sdks | 15:11 | |
| *** pm90__ has joined #openstack-sdks | 15:13 | |
| *** pm90_ has quit IRC | 15:15 | |
| *** shaleh has joined #openstack-sdks | 15:19 | |
| *** notmars has quit IRC | 15:21 | |
| *** notmars has joined #openstack-sdks | 15:23 | |
| shaleh | another newbie question | 15:32 |
| briancurtin | terrylhowe: i'm not sure i understand this linger_for_delete method. it makes me think we're solving the problem in a way that might be better off done by splitting the tests up differently. when i was thinking of the object_store functional tests, there are going to be a lot of classes since there are pre and post conditions that are very different between | 15:32 |
| briancurtin | account, container, and object | 15:32 |
| shaleh | why does OSC not just take a --config argument instead of using environment variables? | 15:33 |
| terrylhowe | shaleh: —os-cloud | 15:33 |
| briancurtin | terrylhowe: and i think trying to stuff things together in one class, this linger_for_delete makes sense, but if we did the wait_for_delete related test in its own class, it wouldnt need to linger and not affect other tests | 15:33 |
| shaleh | terrylhowe: I can set all of my OS_* options in an ini style file and give it to --os-cloud and OSC will just work? | 15:34 |
| briancurtin | terrylhowe: i'm not super opposed to that method right now, but i think we're going to find that functional tests get easier if they're organized perhaps around class-per-proxy-method instead of class-per-service | 15:34 |
| terrylhowe | IMO the server shouldn’t mark the object deleted until all this children are gone briancurtin | 15:34 |
| terrylhowe | but that is not the case | 15:34 |
| briancurtin | i dont know what that means (the children part of it) | 15:34 |
| terrylhowe | shaleh: —os-cloud uses a yaml file | 15:35 |
| terrylhowe | briancurtin: that linger is just for clean up of things we were using networks, etc | 15:36 |
| terrylhowe | you can’t delete the network until the ports are deleted. The server goes away and the ports are deleted yet, so you can’t delete the network | 15:37 |
| shaleh | terrylhowe: nifty. Why do so many wiki docs focus on the env then. A config file is sooo much more convenient. | 15:37 |
| terrylhowe | shaleh: the config file has only been around a couple months | 15:38 |
| shaleh | terrylhowe: I do not see a way to say "read this yaml" | 15:38 |
| terrylhowe | like os config show? | 15:38 |
| shaleh | terrylhowe: no, like "my config lives in proj/this_cloud/clouds.yaml" | 15:39 |
| briancurtin | terrylhowe: fyi im working on that "create doesn't update all attrs" thing, changing it around to use the nicely named "update_attrs", however, that's breaking some of the tests...but in a way that im trying to figure out is even worth keeping around | 15:39 |
| terrylhowe | there are ways to change the path, but on Linux ~/.config/openstack/clouds.yaml | 15:39 |
| terrylhowe | briancurtin: the object store save got broken in some mock upgrade BTW | 15:40 |
| terrylhowe | I’m trying to fix that | 15:40 |
| terrylhowe | shaleh: looks like mac uses the same path | 15:41 |
| terrylhowe | shaleh: I assume you are looking at http://docs.openstack.org/developer/python-openstackclient/configuration.html | 15:41 |
| shaleh | terrylhowe: yes | 15:42 |
| shaleh | looking at os-client-config the list of file names is hard coded | 15:42 |
| shaleh | terrylhowe: it would be easy enough for OpenStackConfig() to take another parameter like "preferred_configs" which it looks for first then it tries "config_files" | 15:46 |
| shaleh | terrylhowe: this way OSC could send it a user specified clouds(-public) yaml | 15:47 |
| terrylhowe | doesn’t it do /etc, ~/.config/openstack, and then current dir? shaleh ? | 15:48 |
| shaleh | terrylhowe: yes, only those three. Not "my team stores it in git and checks it out in /opt/our_config" | 15:48 |
| terrylhowe | I don’t have a need for anything else. It also seemed like you could change the name of the file with OS_CLOUD_NAME or something | 15:48 |
| *** chlong has quit IRC | 15:49 | |
| shaleh | terrylhowe: or I manage a bunch of clouds so I would like to be sure to use THIS config right now | 15:50 |
| terrylhowe | nevermind that one | 15:50 |
| terrylhowe | OS_CLOUD_NAME is for something else | 15:50 |
| shaleh | terrylhowe: for now I can propose an OSC patch which adds a --cloud-config <path> and a --cloud-public-config <path> and uses those to set the OpenStackConfig object. | 15:53 |
| shaleh | terrylhowe: unless you can suggest a better name for the arguments? | 15:53 |
| terrylhowe | shaleh: seems like the arguments should at least be prefixed with —os- like all the other options of those sort. It would be nicer if you could figure out how to work with occ without getting osc involved | 15:56 |
| shaleh | terrylhowe: I left out the OS because this is specific to the CLI. All of the other OS options specific actual cloud bits. | 15:56 |
| terrylhowe | last thing osc needs is more arguments | 15:56 |
| shaleh | terrylhowe: occ already supports this with no change. You just pass OpenStackConfig(config_files=[/path/to/my/yaml], vendor_files=[/path/to/my/public-yaml]) | 15:57 |
| shaleh | terrylhowe: using the yaml means way less options on the command line :-) | 15:57 |
| shaleh | it also means a shell oriented test rig can load the right yaml for a group of tests | 15:58 |
| terrylhowe | using the configuration file is the only way | 15:58 |
| *** notmars has quit IRC | 16:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cliff: Updated from global requirements https://review.openstack.org/203112 | 16:11 |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Update mock requirements https://review.openstack.org/203118 | 16:15 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/203143 | 16:18 |
| openstackgerrit | OpenStack Proposal Bot proposed stackforge/python-openstacksdk: Updated from global requirements https://review.openstack.org/203154 | 16:20 |
| *** jaosorior has quit IRC | 16:26 | |
| *** Qiming has quit IRC | 16:40 | |
| *** stevemar has quit IRC | 16:42 | |
| *** stevemar has joined #openstack-sdks | 16:43 | |
| *** hitalia has joined #openstack-sdks | 16:46 | |
| *** stevemar has quit IRC | 16:48 | |
| *** sigmavirus24 has quit IRC | 16:52 | |
| *** sigmavirus24 has joined #openstack-sdks | 16:53 | |
| *** stevemar has joined #openstack-sdks | 16:55 | |
| dtroyer_zz | s0lman11 | 16:56 |
| *** stevemar_ has joined #openstack-sdks | 16:56 | |
| *** sigmavirus24 has quit IRC | 16:57 | |
| *** stevema__ has joined #openstack-sdks | 16:58 | |
| terrylhowe | I should just call it a weekend and come back when everone is done breaking mock | 16:59 |
| *** stevemar has quit IRC | 17:00 | |
| *** sigmavirus24 has joined #openstack-sdks | 17:01 | |
| *** stevemar_ has quit IRC | 17:01 | |
| shaleh | terrylhowe: :-) | 17:01 |
| *** stevema__ has quit IRC | 17:02 | |
| dtroyer_zz | so, uh, yeah… WTF is goin on in Gerrit? | 17:10 |
| *** trown is now known as trown|lunch | 17:12 | |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Remove requirements.txt from tox.ini https://review.openstack.org/203168 | 17:17 |
| dtroyer_zz | FTR, -infra scrollback is helpful… | 17:18 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Remove requirements.txt from tox.ini https://review.openstack.org/203170 | 17:21 |
| openstackgerrit | Terry Howe proposed openstack/cliff: Remove requirements.txt from tox.ini https://review.openstack.org/203175 | 17:25 |
| openstackgerrit | Terry Howe proposed openstack/os-client-config: Remove requirements.txt from tox.ini https://review.openstack.org/203178 | 17:26 |
| openstackgerrit | Terry Howe proposed openstack/cliff: Drop py26 support for Liberty https://review.openstack.org/203182 | 17:38 |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Drop py33 support https://review.openstack.org/203183 | 17:39 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Drop py26 for Liberty https://review.openstack.org/203185 | 17:41 |
| *** notmars has joined #openstack-sdks | 17:42 | |
| briancurtin | terrylhowe: what's the reason behind dropping 3.3? i'm 100% for us having dropped 3.2 back when we did since 3.3+ made dual-version codebases a little better (i think this was way early on), but I don't know what dropping 3.3 means here | 17:43 |
| terrylhowe | hmm, there was some discussion about this a while back briancurtin let me see if I can find it | 17:43 |
| *** boris-42 has joined #openstack-sdks | 17:44 | |
| *** karimb has quit IRC | 17:44 | |
| terrylhowe | briancurtin: https://wiki.openstack.org/wiki/Python3 | 17:45 |
| openstackgerrit | Merged openstack/python-openstackclient: add functional tests for identity v3 https://review.openstack.org/201574 | 18:24 |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Fix spelling in proxy https://review.openstack.org/203204 | 18:37 |
| openstackgerrit | Sean Perry proposed openstack/python-openstackclient: Allow user to specify their own clouds.yaml or clouds-public.yaml file https://review.openstack.org/203211 | 18:54 |
| *** bknudson has quit IRC | 19:00 | |
| *** stevemar has joined #openstack-sdks | 19:03 | |
| *** stevemar has quit IRC | 19:12 | |
| *** stevemar has joined #openstack-sdks | 19:13 | |
| *** stevemar_ has joined #openstack-sdks | 19:15 | |
| *** stevemar has quit IRC | 19:17 | |
| *** stevemar_ has quit IRC | 19:20 | |
| *** trown|lunch is now known as trown | 19:20 | |
| *** stevemar has joined #openstack-sdks | 19:24 | |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Add configuration function using os-client-config https://review.openstack.org/191535 | 19:26 |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Remove unnecessary test extension mock https://review.openstack.org/199628 | 19:30 |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: enhance tests for user v3 https://review.openstack.org/202687 | 19:34 |
| *** ig0r_ has quit IRC | 19:47 | |
| shaleh | terrylhowe: you here? | 19:59 |
| *** pm90__ has quit IRC | 20:06 | |
| *** figleaf is now known as edleafe | 20:09 | |
| *** stevemar has quit IRC | 20:16 | |
| *** stevemar has joined #openstack-sdks | 20:29 | |
| *** stevemar has quit IRC | 20:34 | |
| *** pm90_ has joined #openstack-sdks | 20:39 | |
| shaleh | dtroyer_zz: you around? | 20:40 |
| *** pm90__ has joined #openstack-sdks | 20:40 | |
| dtroyer_zz | shaleh: for a little while yet | 20:41 |
| *** stevemar has joined #openstack-sdks | 20:42 | |
| *** pm90_ has quit IRC | 20:43 | |
| *** stevemar has quit IRC | 20:46 | |
| openstackgerrit | Merged stackforge/python-openstacksdk: Update mock requirements https://review.openstack.org/203118 | 20:47 |
| *** stevemar has joined #openstack-sdks | 20:53 | |
| shaleh | dtroyer_zz: I responded to your comments on my review. I was looking to hash some of it out. You mention documenting design of the config. Where should we do that? | 20:54 |
| *** bnemec has quit IRC | 20:54 | |
| shaleh | dtroyer_zz: while I like gerrit I find it is missing any form of useful way to express "yes I have thought of this". Writing up lots of design docs for what should be simple stuff is also a little frustrating. | 20:55 |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Set up every time record log in file https://review.openstack.org/186720 | 20:56 |
| *** stevemar has quit IRC | 20:57 | |
| dtroyer_zz | shaleh: agreed on doing design in Gerrit. But the results should be in the OSC docs. It looks like you made some minor adds to that, it could be expanded a bit. And it feels like somewhere we need to write out use cases describing in words how to accomplish some of these things (don't hold up this review for that though) | 20:59 |
| *** stevemar has joined #openstack-sdks | 20:59 | |
| dtroyer_zz | so to your point about multiple clouds… this is exactly the situation I had at Nebula, multiple coulds all defined in clouds-public.yaml that we could share | 20:59 |
| dtroyer_zz | my personal creds went into clouds.yaml. | 21:00 |
| shaleh | dtroyer_zz: if configuration.rst is sufficient I can add more there. | 21:00 |
| dtroyer_zz | and sometimes I still needed to override that for project name or something if I needed to be different for a particular task, then —os-project-name | 21:00 |
| shaleh | dtroyer_zz: I find ~/.foo files extremely annoying. I end up using lots of machines and VMs. Needing to ensure I have the right file in the right place annoys me when i can just say --config=/path/foo | 21:01 |
| dtroyer_zz | do yo do that for .vimrc and .gitconfig etc? | 21:01 |
| dtroyer_zz | that's why I have a dotfiles git repo… ;) | 21:01 |
| shaleh | git when needed, but i often check in from one main machine | 21:02 |
| shaleh | I use emacs and I can function just fine on the vanilla config | 21:02 |
| shaleh | when I live for a long time on a machine I bring my nest over | 21:02 |
| dtroyer_zz | so not everybody is like me, sure, but I want to be REALLY certain we only add useful things , so I'm pushing back a bit | 21:02 |
| dtroyer_zz | emacs, of course ;) | 21:02 |
| shaleh | dtroyer_zz: :-) | 21:03 |
| shaleh | I appreciate the push back. It is always good to question why when adding or removing. | 21:03 |
| dtroyer_zz | Im my Sa days I had a script to copy over the bare minimum stuff… | 21:03 |
| shaleh | As a newbie, I am impressed with the level of discipline and review here | 21:03 |
| dtroyer_zz | anyway, I know there are many ways of handling this, I just want to find the right subset | 21:03 |
| *** trown is now known as trown|outttypeww | 21:03 | |
| *** stevemar has quit IRC | 21:04 | |
| shaleh | dtroyer_zz: I am open to designs that give more flexibility than the hard coded list we have today | 21:04 |
| shaleh | my solution may not be the best | 21:04 |
| shaleh | it was easy to make and show the value | 21:04 |
| dtroyer_zz | so don't take it personally that I'm needing convincing on this. I don't recall if you were in the room in YVR Friday when we talked about config files vs env vars vs cli args. we beat that to death... | 21:04 |
| shaleh | I wasn't | 21:05 |
| dtroyer_zz | cool | 21:05 |
| shaleh | engineers need to bash on stuff. Arguments about what and why are fine by me. | 21:05 |
| shaleh | unless it becomes personal I do not take it personal | 21:05 |
| dtroyer_zz | but we came out with I think agreement. so the idea of being able to override the filenames is a good one, I'm not certain of how I want it to work. at this point its things like the rules for selection (you have doc'ed that bit) and if any 'merging' happens (currently no) etc. | 21:06 |
| shaleh | right | 21:06 |
| shaleh | The current OpenStackConfig class does not make adding to the search path easy | 21:07 |
| shaleh | but it makes sense if the user said "no this one" we do not need the search list. | 21:07 |
| dtroyer_zz | I'm OK with no env var forr this (even thoughj OS_CLOUD is a thing) | 21:07 |
| shaleh | Plus, they may think "clouds.yaml" is a dumb name :-) | 21:07 |
| shaleh | my-really-cool-name-with-git-id-efawf-asdfaw-adfa-adsfa-.yaml all the way, ya know? | 21:08 |
| dtroyer_zz | I didn't pick that name ;) but the idea is that this is a common paradigm for multiple tools and I don't want to diverge too much | 21:08 |
| dtroyer_zz | so if we can keep things easily adoptable by other tools (I only know of shade using o-c-c but I think there are others) that would be good | 21:09 |
| shaleh | fair enough. I figured by allowing short circuit but leaving existing behavior I kept with that | 21:09 |
| dtroyer_zz | This is a good first cut. Makes me thing about the right questions… hopefully we find all of the relevant questions here | 21:09 |
| shaleh | OCC supports what I did, no changes needed. I just passed in my own config_files=[] and vendor_files=[] | 21:10 |
| shaleh | so re: consistency I do not think it breaks it | 21:10 |
| dtroyer_zz | sure, but if there is login other tools might want it would be good to make it common, if it doesn't break the separation of duties | 21:10 |
| shaleh | provided other OpenStack clients also allow the overriding | 21:10 |
| dtroyer_zz | too often we over-normalize things, but we need to be sure not doing so is the right decision | 21:11 |
| dtroyer_zz | anyway, it must be Friday because I'm jabbering on here... | 21:11 |
| shaleh | I brought this up on Terry's review for the SDK earlier. By hiding the OpenStackConfig object I could not do the same thing as a user of the SDK | 21:11 |
| shaleh | My plan was to bring this up with the docs team once my patch was accepted | 21:12 |
| shaleh | things like the install guide need to mention clouds.yaml instead of the ENV vars | 21:12 |
| *** dguerri is now known as dguerri` | 21:12 | |
| shaleh | I was about to investigate making a patch when I found out it already existed | 21:13 |
| shaleh | dtroyer_zz: how does one go about bringing things up to a wider audience? The mailing lists are noisy as hell | 21:15 |
| dtroyer_zz | it depends on the audience… they are noisy, but lots of us use filtering so the things we want to see can be found. | 21:15 |
| dtroyer_zz | the docs team just needs some direct input on their guides about things like OSC. we'll have to write it ourselves to get it complete | 21:16 |
| shaleh | clearly there is some consensus missing on this. Gerrit may not be the best place to find it. | 21:16 |
| dtroyer_zz | remember, env vars and cli args were allw e had until april or so | 21:16 |
| shaleh | dtroyer_zz: right. | 21:16 |
| dtroyer_zz | so most folk are used to managing multiple .openrc-like files | 21:16 |
| dtroyer_zz | and there's your multiple config file handling ;) | 21:17 |
| shaleh | dtroyer_zz: :-) yeah, logging in and out to switch configs is frustrating as hell | 21:17 |
| dtroyer_zz | no need for that, I had a couple of scripts to do it all | 21:17 |
| dtroyer_zz | they even sourced the password from other private files. that's one thing we lose here | 21:18 |
| shaleh | we the -public versus just clouds.yaml provides it fairly well | 21:19 |
| shaleh | unrelated. Does anyone know why self.assertRaises is acting like the 2.6 version when I run a 2.7 test? "with self.assertRaises(Exception): blah" should work | 21:20 |
| briancurtin | whose assertRaises are you calling? unittest, unittest2, testtools? | 21:21 |
| briancurtin | testing in openstack is a real joy | 21:21 |
| shaleh | briancurtin: yeah, I am discovering there are choices. I was not expecting that | 21:21 |
| shaleh | TestCase.assertRaises from Python's unittest module works just fine. | 21:22 |
| shaleh | Must be something busted in the open stack test libs. | 21:22 |
| briancurtin | 100% chance of that | 21:22 |
| shaleh | briancurtin: yup, testtools uses the older format. Python 2.7 and newer allow it to be a ContextManager. Looks like testtools doesn't | 21:25 |
| shaleh | why are we using testtools? Another piece of legacy? | 21:26 |
| *** notmars has quit IRC | 21:27 | |
| briancurtin | shaleh: i've come across a couple of times where it was useful within SDK, but could have solved the problems otherwise without it if i had to. i don't remember how/why we picked that one up | 21:35 |
| shaleh | briancurtin: it looks like a 2.6 and earlier kind of thing. | 21:35 |
| shaleh | It has mocking/monkey patching built in etc. which was new back when | 21:36 |
| shaleh | testtool's concept of "Matcher"s is also useful | 21:36 |
| shaleh | my frustration is it looked like Unittest so I did not expect the difference | 21:36 |
| shaleh | I had to dig through the inheritance tree to find out it was external | 21:37 |
| shaleh | I am tempted to port the Context Manager to it | 21:38 |
| *** sorantis has joined #openstack-sdks | 21:47 | |
| openstackgerrit | Merged stackforge/python-openstacksdk: Updated from global requirements https://review.openstack.org/203154 | 21:50 |
| *** stevemar has joined #openstack-sdks | 22:04 | |
| stevemar | terrylhowe: ping | 22:06 |
| *** stevemar has quit IRC | 22:13 | |
| *** pm90_ has joined #openstack-sdks | 22:15 | |
| *** pm90_ has quit IRC | 22:15 | |
| *** stevemar has joined #openstack-sdks | 22:15 | |
| *** sorantis has quit IRC | 22:15 | |
| *** pm90__ has quit IRC | 22:17 | |
| *** stevemar_ has joined #openstack-sdks | 22:22 | |
| *** stevemar_ has quit IRC | 22:22 | |
| *** stevemar_ has joined #openstack-sdks | 22:23 | |
| *** stevemar has quit IRC | 22:24 | |
| *** stevemar_ has quit IRC | 22:56 | |
| *** steveortiz is now known as zz_steveortiz | 23:04 | |
| openstackgerrit | Merged openstack/python-openstackclient: Support multiple volume qos delete https://review.openstack.org/198822 | 23:10 |
| *** stevemar has joined #openstack-sdks | 23:12 | |
| openstackgerrit | Merged stackforge/python-openstacksdk: Generallize example so it can be modified easily https://review.openstack.org/201672 | 23:18 |
| openstackgerrit | Merged stackforge/python-openstacksdk: Fix spelling in proxy https://review.openstack.org/203204 | 23:18 |
| openstackgerrit | Sean Perry proposed openstack/python-openstackclient: Add 'image create' to openstack https://review.openstack.org/203280 | 23:23 |
| openstackgerrit | Merged openstack/python-openstackclient: add functional tests for identity v2 https://review.openstack.org/201906 | 23:30 |
| openstackgerrit | Sean Perry proposed openstack/python-openstackclient: Add 'image create' to openstack https://review.openstack.org/203280 | 23:32 |
| stevemar | shaleh: thanks :) | 23:32 |
| shaleh | stevemar: note this is just image create. I need to work on membership BP still | 23:32 |
| shaleh | the Image API v2 still needs a fair amount of supporting | 23:33 |
| shaleh | but I about have the lay of it now so adding new bits should not be too hard | 23:34 |
| shaleh | stevemar: we decided here to do 'create, attempt upload -- delete on failure' | 23:35 |
| stevemar | shaleh: is that what the glance cli does? | 23:36 |
| * stevemar really wishes he had time to write code | 23:36 | |
| shaleh | stevemar: no, glance cli leaves a dangling Image. You need to do glance image-list to find it and either use the id in the next run of create or delete it by id and try create again | 23:37 |
| shaleh | cleaning up felt like that right thing to do | 23:37 |
| shaleh | if I am wrong then it is just a couple of lines to yank | 23:38 |
| shaleh | no big deal | 23:38 |
| stevemar | shaleh: ++ for better UX | 23:42 |
| shaleh | bother, I always get unittest(expect, actual) backwares | 23:44 |
| *** trown|outttypeww is now known as trown | 23:46 | |
| openstackgerrit | Sean Perry proposed openstack/python-openstackclient: Allow user to specify their own clouds.yaml or clouds-public.yaml file https://review.openstack.org/203211 | 23:53 |
| shaleh | stevemar: a voice to add direction to this ^^ review would be welcomed as well | 23:56 |
| shaleh | dtroyer and I spoke some earlier about it | 23:56 |
| shaleh | it might need to bubble up to a small group conversation | 23:57 |
| stevemar | shaleh: will do | 23:57 |
| stevemar | shaleh: i plan on going over all OSC open reviews (well the important ones anyway) | 23:57 |
| shaleh | thanks. I am open to reworking things when appropriate. | 23:58 |
| *** stevemar_ has joined #openstack-sdks | 23:58 | |
| stevemar_ | shaleh: sorry, @ logan airport in BOS | 23:58 |
| shaleh | no worries | 23:58 |
| stevemar_ | shaleh: my free wifi ran out! had to re-login! | 23:58 |
| shaleh | bah | 23:58 |
| stevemar_ | shaleh: you should fix their UX :P | 23:59 |
| shaleh | heh | 23:59 |
| shaleh | some problems are bigger than one person | 23:59 |
| stevemar_ | by that i mean just get rid of boigo entirely | 23:59 |
| shaleh | plus, I will be working on keystone/mercador here soon | 23:59 |
| shaleh | I have been taking the chance to fix UX and learn at the same time | 23:59 |
| stevemar_ | shaleh: ohhhh fancy | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!