Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-07-19
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
15:43:41 dtroyer \o/ \O/ \o/
15:44:13 mordred dtroyer: wehave a few more weeks before osc itself is froze, yeah?
15:46:20 dtroyer next week I think, we've considered it a client lib due to other project dependencies
15:46:41 dtroyer so moving osc-lib and sdk would lump them all together but I think that is OK
15:47:24 dtroyer I was hoping to have more of the backlog in OSC merged but I'm planning to cut the to-be-stable release tuesday
15:48:15 mordred dtroyer: kk. I've got a fairly simple command I wanted to add - I'll try to get that done today
15:49:14 dtroyer mordred: sounds good
15:50:44 corvus mordred: is there a way to get the requests.Session() that keystoneclient/openstacksdk use?
15:51:07 mordred corvus: yes
15:51:43 mordred corvus: conn.config.get_session()
15:53:13 corvus mordred: perfect, thanks!
15:53:49 corvus (i wanted to set the max poolsize for the connection pool, since i'm multithreaded; this will allow more connection reuse)
15:53:50 mordred corvus: out of curiosity - are you wanting to use it to make a rest call against one of the openstack endpoints?
15:54:02 mordred ah!
15:54:04 corvus sess = cloud.config.get_session()

Earlier   Later