-
Notifications
You must be signed in to change notification settings - Fork 148
Locking fails when uploading without an architecture type #97
Comments
And by simple, I mean, getting a list of all arch types, getting a lock on each, storing that an array, and looping to unlock. |
I'd suggest the fix would actually be requiring the architecture to be explicitly specified if -l is used and exiting with an error if it isn't. Having multiple locks increases the complexity of something that is already fragile. That said if you consistently don't specify the architecture, then the behaviour of the locking mechanism is still consistent, albeit a bit dirty to the repo layout. The eventual consistency for removal of the lock doesn't matter much, as it would only be potentially dangerous to actually go ahead and write things to the repository while someone else is also doing it. There is a built-in polling wait mechanism in -l that retries to acquire a lock for some time before giving up. The -l option always checks for an existing lock first, and if and only if a lock is not found does it try to write a new lockfile, so a replace wouldn't ever occur. Read-after-write consistency is good enough for the purposes of at least adding packages. I strongly advise against deleting any without revoking permissions to the bucket to all other users first. Could you please elaborate on the issue you have uncovered regarding the eventual consistency @jjshoe ? |
@alexmreis I deleted the eventual consistency comment from here, I'll make a new issue when I come up with a great way to reproduce. |
@alexmreis also, see #95. |
@alexmreis - Actually, I have the nail in the coffin, and the safest possible way for you to do this anyway. You should lock at the bucket level. I know, then it's not as fast as it could be, but |
Also worth noting, specifying an architecture type out front makes it difficult to upload a whole mess of deb files of different arch types. |
A bucket level lock breaks the idea of multiple repositories in the same bucket that the --prefix option allows for. It should at least be on the prefix level, but I do like the idea @jjshoe |
@alexmreis yes, sorry, I'm all for prefix, replace bucket with repository. |
Steps to reproduce:
sleep(60)
here: https://github.com/krobertson/deb-s3/blob/master/lib/deb/s3/cli.rb#L233deb-s3 upload -l -p -b your-bucket -v private your-package.deb
your-bucket/dists/stable/main
you'll discover abinary-
folder holding the lockfileThe fix should be as simple as getting a list of all arch types up front, but this is complicated. Reasons:
The text was updated successfully, but these errors were encountered: