| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-07-19 | |||
| 18:22:07 | Shrews | mordred: well, i know nothing of the other swift-like-yet-not-swift things | |
| 18:22:47 | Shrews | this just bugged me b/c i know i went through this at some point | |
| 18:23:41 | mordred | yah | |
| 18:33:55 | mnaser | mordred: did that end up resolving it? | |
| 18:36:20 | mordred | mnaser: we are still validating | |
| 18:40:42 | openstackgerrit | Merged openstack-infra/shade master: Fix doc mistake https://review.openstack.org/584010 | |
| 19:03:00 | corvus | mnaser, mordred: yeah, this works as expected with the directory marker: https://object-storage-ca-ymq-1.vexxhost.net/v1/86bbbcfa8ad043109d2d7af530225c72/corvustest-1532020307/test/ | |
| 19:15:42 | mnaser | sweet | |
| 19:29:14 | mordred | corvus: woot! | |
| 19:30:17 | openstackgerrit | Hongbin Lu proposed openstack-infra/shade master: Use valid filters to list floating IPs in neutron https://review.openstack.org/584076 | |
| 19:35:25 | mordred | corvus, mnaser: ok - so at this point I think I should send a note to sage about the behavior difference, yeah? | |
| 19:35:57 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: Use valid filters to list floating IPs in neutron https://review.openstack.org/584029 | |
| 19:41:03 | mnaser | mordred: I think so. It looks like rados is much more explicit | |
| 19:41:27 | mnaser | Where as swift is okay with the directory not actually existing | |
| 19:46:55 | mordred | https://etherpad.openstack.org/p/EAYOUb8toz <-- mnaser, corvus this look ok and/or correct? | |
| 19:50:25 | mnaser | mordred: makes sense. I have a long 6 hour flight today. Maybe I can try hacking on this but I haven’t touched C in a while :) | |
| 19:52:56 | mordred | mnaser: :) | |
| 19:53:07 | mordred | mnaser: 6 hours is plenty of time to get your compiler working | |
| 19:53:33 | mnaser | Compiling ceph, hah | |
| 19:57:52 | openstackgerrit | Toure Dunnon proposed openstack-infra/shade master: python-shade expose MTU setting. https://review.openstack.org/578861 | |
| 20:03:37 | corvus | mordred: oh i'd like to suggest some changes to the etherpad | |
| 20:06:08 | corvus | mordred: 2 things. 1) this is specifically the behavior where swift serves existing foo/index.html when you request foo/. that's like "DirectoryIndex" in apache, which is distinct from mod_autoindex, which is auto-generate a directory listing if there is no index.html. swift *also* has that capability, but i haven't done anything with it yet, and i worry that the message appears a little muddied. | |
| 20:06:39 | corvus | mordred: 2) i believe i can point at some swift docs that describe the swift behavior (so the ceph behavior doesn't match the docs) | |
| 20:06:45 | corvus | gimme a sec to dig those up | |
| 20:07:19 | corvus | okay, well, the docs are ambiguous | |
| 20:07:26 | corvus | to me at any rate | |
| 20:07:29 | corvus | https://docs.openstack.org/mitaka/user-guide/cli_swift_static_website.html#static-website | |
| 20:07:51 | corvus | If you create sub-directories for your site by creating pseudo-directories in your container, the index page for each sub-directory is displayed by default. If your pseudo-directory does not have a file with the same name as your index file, visits to the sub-directory return a 404 error. | |
| 20:08:55 | corvus | so, i guess it hinges on the question "what is a pseudo-directory?". I had assumed it was merely including '/' in the paths of uploaded files. if, instead, it means "actually create an application/directory object" then i agree ceph behavior matches docs. | |
| 20:12:47 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add flag for disabling object checksum generation https://review.openstack.org/584044 | |
| 20:12:48 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add ability to pass data to create_object https://review.openstack.org/584045 | |
| 20:12:48 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add create_directory_marker_object method https://review.openstack.org/584050 | |
| 20:13:28 | mordred | corvus: yeah - I agree, those are a bit ambiguous | |
| 20:14:01 | mordred | Shrews, corvus: ^^ same patches, but this time with tests | |
| 20:16:23 | mordred | corvus: it might be worth pinging notmyname about the doc ambiguity | |
| 20:16:49 | corvus | mordred: i just started doing that actually in -swift | |
| 20:16:53 | mordred | neat | |
| 20:18:06 | mordred | corvus: should I wait for his response before sending email? | |
| 20:21:06 | corvus | mordred: yeah, maybe a few mins | |
| 20:21:09 | mordred | kk | |
| 20:23:31 | corvus | the docs i linked are old and don't exist anymore. the closest analog is https://docs.openstack.org/swift/latest/middleware.html#staticweb | |
| 20:23:59 | corvus | that doesn't appear to be any less ambiguous | |
| 20:27:15 | mordred | corvus: agree. it mentions directory marker objects but does not indicate that they are required or why you'd want them | |
| 20:27:23 | mordred | corvus: maybe they ARE required in swift for listings | |
| 20:27:38 | mordred | corvus: but not for index | |
| 20:27:49 | mordred | corvus: since the doc section is in the listings section | |
| 20:35:30 | corvus | mordred: apparently not? at least not according to notmyname. i pasted the relevant parts of my convo with him in the etherpad | |
| 20:36:04 | corvus | tldr -- it seems like they are mostly leftover from earlier stuff and not needed anymore | |
| 20:36:41 | mordred | cool | |
| 20:37:08 | mordred | SO - it seems like it would be a correct action for radosgw to stop requiring them | |
| 20:37:18 | corvus | yep i think that's the best way forward | |
| 20:41:33 | mordred | corvus: hows' that look now? | |
| 20:42:20 | corvus | mordred: lgtm! | |
| 20:47:49 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add flag for disabling object checksum generation https://review.openstack.org/584044 | |
| 20:47:49 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add ability to pass data to create_object https://review.openstack.org/584045 | |
| 20:47:50 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add create_directory_marker_object method https://review.openstack.org/584050 | |
| 22:08:24 | openstackgerrit | Merged openstack-infra/shade master: Use valid filters to list floating IPs in neutron https://review.openstack.org/584076 | |
| 23:04:21 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add create_directory_marker_object method https://review.openstack.org/584050 | |
| #openstack-sdks - 2018-07-20 | |||
| 00:33:24 | openstackgerrit | Sean McGinnis proposed openstack-infra/shade master: Remove redundant target in README https://review.openstack.org/584169 | |
| 00:53:19 | openstackgerrit | Sean McGinnis proposed openstack-infra/shade master: Remove redundant target in README https://review.openstack.org/584169 | |
| 01:10:02 | openstackgerrit | Merged openstack-infra/shade master: Remove redundant target in README https://review.openstack.org/584169 | |
| 02:17:31 | openstackgerrit | Merged openstack/openstacksdk master: Use valid filters to list floating IPs in neutron https://review.openstack.org/584029 | |
| 06:04:44 | openstackgerrit | Benoît Knecht proposed openstack/python-openstackclient master: compute: host: expand kwargs in host_set() call https://review.openstack.org/583225 | |
| 06:33:55 | openstackgerrit | Merged openstack/openstacksdk master: Add flag for disabling object checksum generation https://review.openstack.org/584044 | |
| 06:33:56 | openstackgerrit | Merged openstack/openstacksdk master: Add ability to pass data to create_object https://review.openstack.org/584045 | |
| 06:33:57 | openstackgerrit | Merged openstack/openstacksdk master: Add create_directory_marker_object method https://review.openstack.org/584050 | |
| 08:38:17 | openstackgerrit | Benoît Knecht proposed openstack/python-openstackclient master: compute: host: expand kwargs in host_set() call https://review.openstack.org/583225 | |
| 11:44:47 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: correct the default timeout in Node.set_provision_state https://review.openstack.org/584331 | |
| 11:44:53 | dtantsur | mordred: sorry for this ^^^ | |
| 12:54:57 | mordred | dtantsur|brb: :) | |
| 15:11:06 | mordred | dtantsur: we're gonna have to stable-branch backport that change (it didn't make the freeze cutoff) | |
| 15:11:24 | mordred | corvus: 0.17.0 of sdk has been released which has those new methods we added | |
| 15:11:40 | corvus | mordred: sweet, i'll try 'em out soon | |
| 15:11:47 | dtantsur | ack, I'll propose it | |
| 15:12:52 | mordred | dtantsur: (I don't know if stable/rocky has been cut yet - I think maybe that's today?) | |
| 15:13:23 | dtantsur | mordred: the person releasing openstacksdk should propose that to releases repo | |
| 15:14:33 | dtantsur | mordred: on a related note, which deadline affects openstacksdk: the non-client yesterday or the client next week? | |
| 15:14:48 | dtantsur | I would expect openstacksdk to follow the same cadence as python-XXXclient stuff | |
| 15:15:19 | mordred | dtantsur: non-client | |
| 15:15:25 | dtantsur | that's weird to me | |
| 15:15:34 | mordred | dtantsur: it gets lumped in to that because there are python-*client libs thatuse it | |
| 15:15:41 | dtantsur | puts openstacksdk in a worse position wrt to catching up with features | |
| 15:15:52 | dtantsur | like, API additions can land till next week. | |
| 15:16:00 | dtantsur | but openstacksdk has no chances of exposing them | |
| 15:16:10 | mordred | yeah - I think it's maybe something we should talk about next cycle as we start to use sdk directly for more and more of the projects | |
| 15:16:35 | mordred | dtantsur: good point | |
| 15:16:46 | dtantsur | also for non-client you was supposed to create stable/rocky yesterday with 0.17 :) | |
| 15:16:53 | dtantsur | * for non-client deadline | |
| 15:21:32 | mordred | oh. piddle. | |
| 15:21:40 | mordred | I'm not used to these stable branch things ... | |
| 15:25:23 | openstackgerrit | Merged openstack/openstacksdk master: baremetal: correct the default timeout in Node.set_provision_state https://review.openstack.org/584331 | |
| 15:27:21 | dtantsur | mordred: if you don't know, this is how it looks https://review.openstack.org/#/c/584021/3/deliverables/rocky/sushy.yaml | |
| 15:33:25 | mordred | dtantsur: yeah - I just sent one up | |
| 15:33:29 | mordred | and thanks! | |
| 15:33:34 | dtroyer | osc-lib has the same issue as sdk wrt non-client freeze, I agree it is probably time to re-think that and move them to client-lib category | |
| 15:33:40 | dtroyer | for stein | |
| 15:42:03 | mordred | ++ | |
| 15:42:48 | mordred | dtroyer: speaking of - we're maybe 2 patches away from 1.0 - so I think targetting that for stein should be *totally* doable | |
| 15:42:53 | mordred | (for sdk) | |
| 15:43:33 | mordred | dtroyer: in fact, I'm hoping I can get those written during freeze-time ... so hopefully we can cut 1.0 very early in the stein cycle | |