| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-11-29 | |||
| 18:21:34 | Shrews | http://logs.openstack.org/93/523893/1/gate/openstack-tox-py27/8ddfac1/ara/result/38e984ce-db99-4dac-8399-5e4002a0025b/ | |
| 18:25:03 | mordred | Shrews: that sounds like a thing we should bring up over in infra land | |
| 18:25:12 | Shrews | k | |
| 18:35:09 | thingee | mordred: I'm still interested on working on oaktree. I read through the brain dump email and appreciate you writing that up, finally ;) | |
| 18:37:28 | thingee | mordred: I would like to keep my core seat please. | |
| 18:40:07 | cdent | I’d like to work on oaktree too, but I’m struggling to think I’ll have the time :( | |
| 18:51:47 | openstackgerrit | Merged openstack-infra/shade master: Implement availability_zone_hints for networks and routers. https://review.openstack.org/522004 | |
| 19:06:37 | Shrews | just for the record, if oaktree attempts to handle more than just openstack clouds (which is already insanely complex), it will be doomed to failure. <EOM> | |
| 19:35:30 | cdent | Shrews++ | |
| 19:43:18 | SamYaple | Shrews: all we gotta do is abstract a little bit more. easy peasy | |
| 19:44:52 | openstackgerrit | Harald Jensås proposed openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974 | |
| 19:48:02 | thingee | Shrews: +1 | |
| 20:18:38 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | |
| 20:32:38 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | |
| 20:55:44 | openstackgerrit | Harald Jensås proposed openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974 | |
| 21:04:56 | mordred | thingee: woot. sorry it took me so long to get that down on paper in any useful way | |
| 21:05:06 | mordred | SamYaple: agree | |
| 21:05:39 | mordred | cdent: I'm going to give you core for now, just because - if it doesn't work out, no biggie | |
| 21:06:20 | cdent | mordred: I think I can handle that responsibly. Thanks. | |
| 21:09:35 | openstackgerrit | Merged openstack/python-openstacksdk master: Add ability to work in other auth contexts https://review.openstack.org/522970 | |
| 21:09:35 | openstackgerrit | Merged openstack/python-openstacksdk master: Sort image update results before comparing https://review.openstack.org/523893 | |
| 21:09:37 | openstackgerrit | Merged openstack/python-openstacksdk master: Add helper property to get the current user id https://review.openstack.org/522971 | |
| 21:09:38 | openstackgerrit | Merged openstack/python-openstacksdk master: Document current_user_id in a release note https://review.openstack.org/522973 | |
| 21:09:38 | openstackgerrit | Merged openstack/python-openstacksdk master: Remove reference to context-managers from release note https://review.openstack.org/522972 | |
| 21:10:53 | SamYaple | mordred: ill try to get on the rest of the reviews after i land in denver | |
| 21:11:07 | SamYaple | keep them small and ill keep reviewing :P | |
| 21:18:18 | mordred | SamYaple, Shrews: I think we need a tag or a topic or something to mark "this already landed in shade or os-client-config and is just a cherry-pick to get them up to date" vs. "this is actual code that needs actual review other than 'do the tests pass'" | |
| 21:19:35 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | |
| 21:20:32 | dtroyer | mordred: at what point would you want to reverse that? ie, SDK is source of truth and shade/occ are 'backports'? when they become shims? | |
| 21:22:01 | openstackgerrit | Monty Taylor proposed openstack/cliff master: Use in-tree cliffdemo app for docs build https://review.openstack.org/515016 | |
| 21:22:02 | openstackgerrit | Monty Taylor proposed openstack/cliff master: Remove tox_install.sh and just pass -c in tox https://review.openstack.org/523984 | |
| 21:22:02 | openstackgerrit | Monty Taylor proposed openstack/cliff master: Move doc requirements to doc/requirements.txt https://review.openstack.org/522369 | |
| 21:22:25 | openstackgerrit | Monty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs https://review.openstack.org/522370 | |
| 21:25:11 | mordred | dtroyer: yah- when they become shims | |
| 21:25:35 | mordred | dtroyer: and hopefully at that point we don't have to do any cherry-picking/backports *fingers crossed* | |
| 21:35:17 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986 | |
| 21:35:18 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987 | |
| 21:35:20 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988 | |
| 21:35:21 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989 | |
| 21:37:32 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Avoid tox_install.sh for constraints support https://review.openstack.org/508061 | |
| 21:37:33 | Shrews | mordred: maybe. i just compared git logs to discover that, but if it would help others, fine with me | |
| 21:39:25 | Shrews | mordred: there is a git option to leave a "cherry picked" comment, iirc | |
| 21:39:37 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove use of tox-siblings role https://review.openstack.org/523993 | |
| 21:39:46 | Shrews | git cherry-pick -x ... or similar | |
| 21:39:59 | Shrews | i thought git-review carried it over, too | |
| 21:40:15 | TheJulia | -X | |
| 21:40:26 | TheJulia | -x doesn't create the comment | |
| 21:41:14 | Shrews | well boo | |
| 21:41:37 | Shrews | oh, but -X does? | |
| 21:41:38 | TheJulia | c'est la vie | |
| 21:41:44 | TheJulia | yes, it does | |
| 21:42:21 | Shrews | whoomp, there it is | |
| 21:44:05 | mordred | Shrews: neat! | |
| 21:44:17 | mordred | oh- also - wow ... TheJulia do I see a Remove python-ironicclient patch up there??? | |
| 21:44:52 | TheJulia | mordred: is it too early to consider it a birthday gift? | |
| 21:45:39 | TheJulia | mmm whisky | |
| 21:46:42 | TheJulia | Seriously though, Yes, I hope it is right. Unit tests pass locally, which is a good sign. I fired up a dnm test on bifrost to pull it in and see if it breaks. I also moved my wip tag accordingly. | |
| 21:48:59 | Shrews | we should, ya know, merge that and stuff | |
| 21:50:03 | TheJulia | Shrews: yeah, for some reason my removal of my wip tag earlier today from the patch chain didn't take... :\ | |
| 21:50:35 | mordred | TheJulia, Shrews: +2 with comment on https://review.openstack.org/#/c/499774/3 - comment applies to the previous patch too (which is already +A'd) - I think it's totally fine for us to deal with in a followup | |
| 21:52:06 | mordred | TheJulia: also, I think there is a rebase oops in https://review.openstack.org/#/c/500119/3 | |
| 21:52:24 | openstackgerrit | Merged openstack-infra/shade master: Fix basepython setting in tox.ini https://review.openstack.org/523915 | |
| 21:53:16 | TheJulia | mordred: replied, and agreed, I'll rip that out in the morning. | |
| 21:53:25 | Shrews | what's funny is, that it still passed tests | |
| 21:53:44 | Shrews | s/funny/sad/ ?? | |
| 21:54:18 | TheJulia | wow... fun rebase failure :( | |
| 21:57:07 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-clientify baremetal create/delete https://review.openstack.org/500119 | |
| 21:57:14 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify baremetal machine port list https://review.openstack.org/500120 | |
| 21:57:30 | mordred | TheJulia: AHA - normalize_machines only removes novaclient noise - I was wondering how the old method worked with normalize_machines(MachienPortList()) :) | |
| 21:58:06 | TheJulia | mordred: my guess, cp -a python-novaclient python-ironicclient | |
| 22:01:17 | mordred | TheJulia: also, left a note for later in https://review.openstack.org/#/c/500119 - I don't think we should do anything about it today or tomorrow | |
| 22:01:30 | mordred | TheJulia: thank you for leaving the comment there though - it was very helpful :) | |
| 22:01:43 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify machine patch operations https://review.openstack.org/500121 | |
| 22:02:37 | mordred | TheJulia, Shrews: stack up to the WIP is all +2 from me. | |
| 22:02:51 | adriant | tiple services in the same connection object was odd). | |
| 22:02:51 | adriant | Out of curiosity, for the openstackSDK, if "param profile: DEPRECATED" what am I meant to use in code programatically to define what region I need to talk to? The docs for openstackSDK do not mention the profile deprecation, and as much as I dislike the profile mechanism, it made sense (although the ability/need to define a different region for mul | |
| 22:03:16 | mordred | TheJulia: THANK YOU! I think as soon as that last patch lands we should cut another release, since removing that depend will make life **WAY** easier on packagers | |
| 22:03:38 | mordred | adriant: that'sa great question - and we should clearly make a document about that | |
| 22:03:46 | mordred | adriant: one second - lemme link you to something | |
| 22:04:01 | TheJulia | mordred: awesome, I'm going to disappear shortly for at least the next few hours | |
| 22:04:22 | TheJulia | mordred: I just feel bad I'm just now getting to it.... | |
| 22:05:00 | adriant | mordred: I'm trying to switch a bunch of tools to openstackSDK, but none of them rely on clouds.yaml (it doesn't make sense to) and I need to pass in config stuff. From keystoneauth I can get my session, find, but region I've always done with profile. | |
| 22:05:05 | mordred | adriant: well, this isn't nearly as useful a link as I'd hoped, but ... | |
| 22:05:15 | mordred | http://git.openstack.org/cgit/openstack/python-openstackclient/tree/openstackclient/network/client.py#n39 | |
| 22:05:38 | adriant | mordred: that still uses profile :P | |
| 22:05:41 | mordred | adriant: yah- the answer for you is gonna be to make a CloudConfig object basically just like you've been making a profile | |
| 22:06:02 | mordred | adriant: lemme make a quick patch to something else that shows the ACTUAL answer | |
| 22:06:02 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Release note cleanup https://review.openstack.org/524004 | |
| 22:06:07 | mordred | adriant: and also a doc patch :) | |
| 22:06:53 | adriant | mordred: k, I had a feeling i'd end up having to make a config object :P | |
| 22:07:03 | mordred | adriant: out of curiosity - do you want your use to explicitly *not* support clouds.yaml - or more you don't have any clouds.yaml and don't care | |
| 22:07:28 | mordred | (example patch is different depending on which of those is true - I should probably make both examples) | |
| 22:08:02 | adriant | mordred: the thing I'm working on in question is a django site/service that has it's own config file. So clouds.yaml doesn't make sense. | |
| 22:08:12 | mordred | adriant: nod. cool - example coming | |
| 22:08:15 | adriant | same again with a different API service I've got | |
| 22:08:33 | adriant | then the rest of my tools are built on top of the openstackclient, so they natively just use clouds.yaml | |
| 22:09:19 | adriant | I find that if I'm writing any scripts for openstack, I just make them an openstackclient plugin and forget all the tiresome stuff and focus on functionality | |
| 22:09:53 | adriant | mordred: and thanks :) | |
| 22:31:31 | mordred | adriant: https://review.openstack.org/524011 Update openstacksdk construction to be forward compatible | |
| 22:32:05 | mordred | adriant: I'm going to extract that into a proper doc - but that should have all the pieces you need (and more) | |