Skip to content
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

Images created against docker daemon failed to get saved against remote Artifactory #9

Closed
thisisnotashwin opened this issue Jul 8, 2019 · 4 comments
Labels
type/bug Something isn't working

Comments

@thisisnotashwin
Copy link
Member

Images created using the local package seem to experience errors when the remote package is used to save the previously created image against a remote artifactory. (When performing a remote.Write)

The artifactory returns the error MANIFEST_INVALID: manifest invalid;

Preliminary investigations make it seem like the Config files created are different for either image (remote vs local). What was a little weird is that we were able to use the docker CLI to push the image to artifactory but were unable to use GGCR to perform the same operation.
Additionally, performing an operation like mutate.CreatedAt on the image updated the config to become a "valid" one.

Let me know if I can provide any further information.

@jromero
Copy link
Member

jromero commented Jul 11, 2019

What version (aka commit hash) are you using? Is there a pared down sample of code to reproduce this issue?

@thisisnotashwin
Copy link
Member Author

thisisnotashwin commented Aug 1, 2019

Sorry. I just read this. I feel bad. This was behavior we witnessed with pack 0.2.1.
We created a builder without the publish flag, used docker push to push the image on gcr.io. We then used this relocation library https://github.com/pivotal/image-relocation to relocate the image to an artifactory registry.
We saw this relocation fail with the Invalid manifest error.
On a side note, relocating to gcr seems to behave correctly.

@ekcasey ekcasey added the type/bug Something isn't working label Aug 1, 2019
@jromero
Copy link
Member

jromero commented Oct 17, 2019

@ashwin-venkatesh Does #12 fix this issue as well?

@ekcasey
Copy link
Member

ekcasey commented Jan 31, 2020

resolved by #20

@ekcasey ekcasey closed this as completed Jan 31, 2020
natalieparellano pushed a commit that referenced this issue Apr 19, 2024
Prune more stuff from cnb_index.go, but add back setters!
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants