Merge lp://staging/~joetalbott/ubuntu-ci-services-itself/data-store into lp://staging/ubuntu-ci-services-itself
Proposed by
Joe Talbott
Status: | Merged |
---|---|
Approved by: | Chris Johnston |
Approved revision: | 13 |
Merged at revision: | 99 |
Proposed branch: | lp://staging/~joetalbott/ubuntu-ci-services-itself/data-store |
Merge into: | lp://staging/ubuntu-ci-services-itself |
Diff against target: |
83 lines (+71/-0) 2 files modified
docs/components/data-store.rst (+63/-0) docs/components/planned.rst (+8/-0) |
To merge this branch: | bzr merge lp://staging/~joetalbott/ubuntu-ci-services-itself/data-store |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Francis Ginther | Approve | ||
Joe Talbott (community) | Needs Resubmitting | ||
Review via email: mp+198060@code.staging.launchpad.net |
Commit message
docs - Add data-store design document.
Description of the change
docs - Add data-store design document.
To post a comment you must log in.
On 12/06/2013 08:08 AM, Joe Talbott wrote:
> +put_file(filename, private=False)
> + Add a file to the data store.
Does this need logic to deal with the remote path on disk versus the
relative path in Swift? I'm not sure the mechanics in swift but maybe a
simpler approach would be something like:
def create_ file(filename, private=False):
# do something to create a Python FD.
yield fd
Then your logic for creating files might look like:
with open(resource, 'r') inf: file('/ bad_example. log') as outf:
outf. write(inf. read())
with ds.create_
This might not make any sense in swift though.
> +get_file(filename)
> + Get a file from the data store.