-
Notifications
You must be signed in to change notification settings - Fork 365
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove .tar extension from blob and config file names #419
Conversation
Makes sense overall, but this is an incompatible change, so it should increase the version number. Can you prepare a parallel PR to fix the skopeo tests, please? (Highlighting: This will break existing automated consumers of the |
LGTM |
FYI Buildah uses the dir: format pretty heavily in its bats tests. |
Change LGTM, Travis is unhappy and we'll need to roll into the other projects. |
The config file was being saved as digest.tar for the directory transport. This is misleading as the config file is not a tar archive. Dropped the .tar extension from all files now, including blobs. The user can use a tool like file to determine the format of the files in the directory. Signed-off-by: umohnani8 <umohnani@redhat.com>
Increased the version number to 1.1 from 1.0 |
@runcom PTAL |
@umohnani8 Why is test blowing up? |
@umohnani8 go ahead and fix containers/skopeo#479 |
skopeo removed the expectation that the file ends in .tar, which is not consistent with the tar spec. containers/image#419 fixes: #3705
skopeo removed the expectation that the file ends in .tar, which is not consistent with the tar spec. containers/image#419 fixes: #3703
Adds comparators for a new class Version which represents a skopeo directory transport version. This will allow us to easily support future changes. containers/image#419 fixes: #3703 [WIP] Alternative: read version file Version comparisons, done right Version comparator and unittests
Adds comparators for a new class Version which represents a skopeo directory transport version. This will allow us to easily support future changes. containers/image#419 fixes: #3703 [WIP] Alternative: read version file Version comparisons, done right Version comparator and unittests
Adds comparators for a new class Version which represents a skopeo directory transport version. This will allow us to easily support future changes. containers/image#419 fixes: #3703
Adds comparators for a new class Version which represents a skopeo directory transport version. This will allow us to easily support future changes. containers/image#419 fixes: #3703
Adds comparators for a new class Version which represents a skopeo directory transport version. This will allow us to easily support future changes. containers/image#419 fixes: #3703
Adds comparators for a new class Version which represents a skopeo directory transport version. This will allow us to easily support future changes. containers/image#419 fixes: #3703
Adds comparators for a new class Version which represents a skopeo directory transport version. This will allow us to easily support future changes. containers/image#419 fixes: #3703
The config file was being saved as digest.tar for the directory transport.
This is misleading as the config file is not a tar archive.
Dropped the .tar extension from all files now, including blobs.
The user can use a tool like file to determine the format of the files in the directory.
Fixes issue containers/buildah#481
Signed-off-by: umohnani8 umohnani@redhat.com