| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-07-18 | |||
| 17:32:34 | tosky | would the usage of --enable-this=True/False be allowed, or should it be avoided for consistency? | |
| 17:32:58 | tosky | pro: it may help distinguish between an explicit choice and the usage of the internal default | |
| 17:33:46 | tosky | (context: discussion in https://review.openstack.org/#/c/580727/ ) | |
| 17:34:58 | Shrews | that seems like an awkward usage, tbh. one that i personally cannot recall ever seeing in any cli | |
| 18:23:15 | openstackgerrit | Hongbin Lu proposed openstack/python-openstackclient master: [DNM] Test neutron change https://review.openstack.org/583690 | |
| 20:54:22 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add /v3 to the auth_url for vexxhost https://review.openstack.org/583737 | |
| 21:14:32 | mordred | Shrews: you seem to be back ... have a sec for that ^^ ? | |
| 21:14:45 | mordred | dtantsur|afk: I started looking at it but haven't properly read it yet | |
| 21:15:06 | Shrews | mordred: sure | |
| 21:16:40 | Shrews | mordred: done | |
| 21:18:48 | mordred | Shrews: thanks | |
| 21:18:55 | mordred | Shrews: I think we need to split that into a library | |
| 21:19:09 | mordred | Shrews: like service-types-authority/os-service-types | |
| 21:19:24 | mordred | Shrews: because it kinda sucks to need to release a whole new sdk just to fix a vendor profile | |
| 22:01:43 | corvus | con = sw.create_container(name='corvustest', read_ACL=".r:*") | |
| 22:01:53 | corvus | when i do that, the read acl isn't set | |
| 22:02:02 | corvus | i have to do a set_container_metadata call | |
| 22:02:26 | corvus | (sw = conn.object_store) | |
| 22:26:06 | mordred | corvus: so - I'll have to dig in on that one a bit - another option is conn.create_container(name='corvustest', public=True) | |
| 22:26:30 | corvus | oh neat, i didn't see that | |
| 22:27:21 | corvus | mordred: also, the next thing i was about to dig into (once it's unbroken) is how do i set X-Container-Meta-Web-Index ? | |
| 22:27:37 | mordred | corvus: it's the shade layer rather than the sdk object layer - also, fwiw, conn.create_object will automatically use large object segments for you if any of your files are over the object size limit for the cloud | |
| 22:28:59 | corvus | mordred: oh, i feel like i missed an important fork in the road :) | |
| 22:29:13 | mordred | corvus: conn.update_container(headers={'X-Container-Meta-Web-Index': 'blah'}) should do the trick | |
| 22:29:14 | corvus | mordred: i went to https://docs.openstack.org/openstacksdk/latest/user/index.html#api-documentation which led me to https://docs.openstack.org/openstacksdk/latest/user/proxies/object_store.html and that's why i thought i should be using conn.object_store | |
| 22:29:26 | mordred | corvus: ah - that's a great feedback note | |
| 22:30:24 | mordred | corvus: I'll work on some better entrypoint roadmapping ... | |
| 22:30:33 | mordred | corvus: https://docs.openstack.org/openstacksdk/latest/user/connection.html might be interesting to look at | |
| 22:30:36 | corvus | mordred: i see https://docs.openstack.org/openstacksdk/latest/user/connection.html now | |
| 22:30:37 | corvus | ya | |
| 22:30:57 | mordred | corvus: although - I'm now going to apologize for lacking some docstrings | |
| 22:31:05 | mordred | corvus: you're making several todo list items for me :) | |
| 22:31:14 | corvus | mordred: i think i would have needed that to be the first thing, and probably the text "The Connection class is the primary interface to the Python SDK" in the introduction page. that would have gotten me to the right place. | |
| 22:31:47 | mordred | corvus: yeah ... there's a piece of narrative that needs to be written | |
| 22:32:20 | mordred | corvus: that describes the different interfaces in some short but descriptive terms and leads you to the one that you want | |
| 22:33:34 | corvus | mordred: when you said you were going to apologize for lacking some docstrings, was the docstring for "create_container" one of them? :) | |
| 22:34:34 | mordred | corvus: yes. also for set_container_access and get_container and update_container and get_object_metadata | |
| 22:45:53 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add missing swift docstrings https://review.openstack.org/583790 | |
| 22:45:57 | mordred | corvus: ^^ | |
| 22:46:00 | mordred | corvus: sorry about that | |
| 22:48:25 | corvus | mordred: oh noes you have the whitespace in that change | |
| 22:48:46 | mordred | NOT WHITESPACE OH NO WE'RE ALL DOOMED | |
| 22:49:25 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add missing swift docstrings https://review.openstack.org/583790 | |
| 22:50:09 | mordred | corvus: now - your use case brings up an interesting point ... which is that I could see you _not_ wanting the "create client-side-checksums and upload as metadata" behavior | |
| 22:50:28 | mordred | corvus: which is on by default to help prevent double-uploads of large files | |
| 22:50:45 | mordred | but there isn't a great way to disable it | |
| 22:51:09 | corvus | mordred: there's a really good chance we'll upload the same file more than once | |
| 22:51:18 | corvus | like a nova config file | |
| 22:51:28 | mordred | corvus: yah - but it would be to different paths | |
| 22:51:34 | mordred | so that's fine | |
| 22:51:35 | corvus | oh ok | |
| 22:51:47 | corvus | so you're just concerned about the time spent generating the checksum? | |
| 22:51:56 | mordred | yeah. it's wasted | |
| 22:52:01 | mordred | for the log upload case | |
| 22:52:05 | corvus | true. probably not huge though. | |
| 22:52:21 | mordred | yah. I'll make a flag and you can pick it up in an update | |
| #openstack-sdks - 2018-07-19 | |||
| 01:16:38 | openstackgerrit | Merged openstack/openstacksdk master: Add /v3 to the auth_url for vexxhost https://review.openstack.org/583737 | |
| 01:55:57 | openstackgerrit | wangxiyuan proposed openstack/keystoneauth master: Add netloc and version check for version discovery https://review.openstack.org/583215 | |
| 02:19:00 | openstackgerrit | wangxiyuan proposed openstack/keystoneauth master: Add netloc and version check for version discovery https://review.openstack.org/583215 | |
| 04:43:51 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: [DNM] Test neutron change https://review.openstack.org/583851 | |
| 07:27:16 | openstackgerrit | Merged openstack/openstacksdk master: Add missing swift docstrings https://review.openstack.org/583790 | |
| 08:40:11 | openstackgerrit | Josephine Seifert proposed openstack/python-openstackclient master: [WIP] osc-included image signing (using openstacksdk) https://review.openstack.org/580086 | |
| 09:14:56 | openstackgerrit | Josephine Seifert proposed openstack/python-openstackclient master: [WIP] osc-included image signing (using openstacksdk) https://review.openstack.org/580086 | |
| 09:54:15 | Luzi | mordred, could you review our osc-included image signing? parts have been uploaded to sdk: https://review.openstack.org/#/c/580080/ and osc: https://review.openstack.org/#/c/580086/ | |
| 10:50:35 | cdent | dtantsur|bbl, edleafe, elmiko : I may miss start of ap-sig meeting. need to cart sarah and her ankle around a bit before that and not sure how long it will be | |
| 12:02:14 | dtantsur | cdent: ack | |
| 12:31:04 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Support for microversions in base Resource https://review.openstack.org/583491 | |
| 13:51:00 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add missing release note about vexxhost auth_url https://review.openstack.org/583962 | |
| 13:59:13 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Support for microversions in baremetal resources https://review.openstack.org/583532 | |
| 14:21:39 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Support for microversions in baremetal resources https://review.openstack.org/583532 | |
| 14:23:50 | cdent | edleafe: oh my, the solar panel installers keep saying "huzzah" | |
| 14:27:17 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Support for microversions in baremetal resources https://review.openstack.org/583532 | |
| 14:29:26 | edleafe | cdent: were they saying it to me?? | |
| 14:29:48 | mordred | edleafe: yes | |
| 15:01:49 | dtantsur | cdent, edleafe, elmiko, having a headache the whole day, may skip the meeting if I don't recover by then :( | |
| 15:02:10 | cdent | dark rooms, plenty of water | |
| 15:06:01 | openstackgerrit | Merged openstack/openstacksdk master: Implement signature generation functionality https://review.openstack.org/580080 | |
| 15:14:42 | openstackgerrit | Merged openstack/openstacksdk master: Add set_provision_state and wait_for_provision_state for baremetal Node https://review.openstack.org/570895 | |
| 16:03:40 | openstackgerrit | Valentin Boucher proposed openstack-infra/shade master: Fix doc mistake https://review.openstack.org/584010 | |
| 16:28:34 | cdent | edleafe: https://etherpad.openstack.org/p/api-sig-newsletter please and thank you | |
| 16:31:22 | edleafe | cdent: it is a marvel of wordsmithery. Ship it! | |
| 16:31:34 | cdent | it took so many keypresses | |
| 16:31:48 | edleafe | every one was worth it | |
| 16:52:18 | openstackgerrit | Merged openstack/keystoneauth master: Add netloc and version check for version discovery https://review.openstack.org/583215 | |
| 16:58:34 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: Use valid filters to list floating IPs in neutron https://review.openstack.org/584029 | |
| 17:00:34 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: [DNM] Test neutron change https://review.openstack.org/583851 | |
| 17:06:42 | corvus | mordred: another issue with Connection.create_object: there's no way to upload a zero-byte file without creating one on disk | |
| 17:06:59 | corvus | mordred: (more generally, there's no way to upload data without it being on disk) | |
| 17:09:58 | openstackgerrit | Merged openstack/openstacksdk master: meta: don't throw KeyError on misconfigured floating IPs https://review.openstack.org/582460 | |
| 17:10:00 | openstackgerrit | Merged openstack/openstacksdk master: Add missing release note about vexxhost auth_url https://review.openstack.org/583962 | |
| 17:21:45 | Shrews | corvus: that's a strange comment (possibly b/c of lack of context) | |
| 17:22:14 | Shrews | why upload data you don't have? | |
| 17:22:50 | corvus | Shrews: continued conversation from yesterday about proxy vs connection | |
| 17:23:01 | corvus | Shrews: apparently i need to create a zero byte object in swift as a directory marker | |
| 17:23:31 | Shrews | corvus: hrm, that seems like something i had to deal with at some point in the past | |
| 17:23:38 | corvus | aiui, if i were to use the connection object, i would have to create a zero byte file on disk and pass the path to it. if i use the proxy, i can say data='' | |
| 17:24:12 | Shrews | corvus: but now i have context (so thx) :) | |
| 17:24:23 | corvus | np | |
| 17:27:37 | Shrews | actually, i think it was with the shade create_container code i originally wrote. there was weirdness with it | |
| 17:28:09 | Shrews | err, create_object | |