| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-07-19 | |||
| 18:05:36 | mordred | corvus: your code was passing text/html - but the docs from mnaser said that the directory objects wanted to be application/directory so I put that in the helper method - but obviously I havent' tested that in the wild | |
| 18:05:56 | corvus | mordred: yeah my snippet was an example of an actual file | |
| 18:06:06 | mordred | Shrews: hrm that makes me want to make a list_object_directories method or something similar that does delimiter='/' and whatnot from that doc | |
| 18:06:13 | mordred | corvus: ah - gotcha | |
| 18:06:32 | mordred | I should probably add some unittests of those | |
| 18:06:38 | mordred | or else Shrews might be unhappy | |
| 18:17:57 | Shrews | mordred: corvus: so i confirmed that naming the object with heirarchy works as expected | |
| 18:18:23 | Shrews | corvus: so you can avoid the whole empty file workaround if you do that | |
| 18:19:31 | mordred | Shrews: I'm not sure what you mean? | |
| 18:20:01 | corvus | Shrews: apparently that's true with actual swift. but perhaps not with ceph/rados. at least, that's what i heard. so i wanted to do a quick 5 minute experiment to verify. :) | |
| 18:20:02 | mordred | Shrews: oh - the empty file thing is to make the directory listing work on some installations (it's not always required, but it never hurts) | |
| 18:20:03 | Shrews | mordred: create_container("foo") ; create_object("foo", "path/to/object.txt") | |
| 18:20:28 | corvus | i've got access to one of each right now, and am testing in parallel | |
| 18:20:28 | mordred | Shrews: yes. that will work - but on radosgw path/to won't render as a directory without the 0byte marker object | |
| 18:20:52 | Shrews | mordred: are you saying that we HAVE AN INCONSISTENCY WITH AN OPENSTACK API???????? | |
| 18:21:11 | mordred | Shrews: shocking right? | |
| 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 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 | |