| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-07-19 | |||
| 18:21:22 | Shrews | speechless | |
| 18:21:23 | corvus | i'm not weighing in on what anyone should do in the long run. i'm only just trying to write the code to perform the experiment to formulate whatever may or may not become a bug report. :) | |
| 18:21:32 | mordred | Shrews: although to be fair to swift in this case, it's an inconsistency wtih swift and something else that is not swift that iplements the same api | |
| 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 create_directory_marker_object method https://review.openstack.org/584050 | |
| 20:12:48 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add ability to pass data to create_object https://review.openstack.org/584045 | |
| 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 ability to pass data to create_object https://review.openstack.org/584045 | |
| 20:47:49 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add flag for disabling object checksum generation https://review.openstack.org/584044 | |
| 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 | ++ | |