github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Merge pull request #1006 from dmcgowan/content-writer-exists

Browse Source 
Content store writer checks expected
This commit is contained in:
Michael Crosby 2017-06-16 13:29:07 -07:00 committed by GitHub
commit d922f48735
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
content/store.go
View File

@ -227,8 +227,14 @@ func (s *store) total(ingestPath string) int64 {
// //
// The argument `ref` is used to uniquely identify a long-lived writer transaction. // The argument `ref` is used to uniquely identify a long-lived writer transaction.
func (s *store) Writer(ctx context.Context, ref string, total int64, expected digest.Digest) (Writer, error) { func (s *store) Writer(ctx context.Context, ref string, total int64, expected digest.Digest) (Writer, error) {
// TODO(stevvooe): Need to actually store and handle expected here. We have // TODO(stevvooe): Need to actually store expected here. We have
// code in the service that shouldn't be dealing with this. // code in the service that shouldn't be dealing with this.
if expected != "" {
p := s.blobPath(expected)
if _, err := os.Stat(p); err == nil {
return nil, ErrExists
}
}
path, refp, data := s.ingestPaths(ref) path, refp, data := s.ingestPaths(ref)