-
Notifications
You must be signed in to change notification settings - Fork 690
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
Extracting config.json #308
Comments
Can you include the I bring it up because if that file's missing, something clearly went wrong earlier, and I'd like to diagnose. thanks for reporting! |
|
Any ideas what could be wrong? |
No, can you share the full output? I suspect the true error is buried :( |
I have the same issue with this
However, if I use
Running bazel like this |
Ah, this is almost certainly manifest lists. @jonjohnsonjr can you TAL? |
@schoren that digest points to a multi-arch image. We don't currently handle these in a very useful way, see #300. The amd64/linux form of that image would be this:
Pulling by tag works because dockerhub will default to returning the amd64/linux image in this scenario. @jeb2239 you might be seeing the same thing if the registry you're pulling from doesn't implement this backwards compatibility. @mattmoor I think we should probably be resolving these images based on platform, but I don't think the right thing to do is blindly assume amd64/linux. |
That sounds like a noble strategy for once we can build anything other than |
@jonjohnsonjr here's where we currently hardcode this. I specifically asked @dekkagaijin to make these configurable like so in some changes he made to the library a while back, so this should just be a matter of plumbing. (famous last words) |
Thanks guys. Very useful information. How do you get the digest for a specific image arch? |
Is there anyway to fix this for my case? |
@jeb2239 Can you please share the full error output from Bazel? Sorry for letting this get a bit hijacked. |
|
Interesting, does this happen without |
@mattmoor thanks, I'll take a look tomorrow. @schoren assuming you have jq installed, this should work:
|
The tarball looks like
|
@jeb2239 Was the JSON above answering Jon? Was it the DockerHub url, or did you adapt it for your registry? Can you share the contents of |
This is manifest.json of the centos image
|
And the
|
The output user root is necessary other wise the output would be on NFS. |
From my docker registry
|
Same happened:
[{"Config":"8052e565ea22da1c06c44f3177e9040eed7c2e55f0e88ed08ddbc5086fd63dea.json","RepoTags":null,"Layers":["b2039296b7a9d102dc9b1e5c87e0bcde21e4ce4d22bf486e975645334c81375f/layer.tar","7be7e0c8444bded7af57ff6341fa64ee959e49f5e229d9b21e29e649a47e8366/layer.tar","e9f3b5fdf6c177c82c2b36409f7702894030f9d86e54596501db02eff88b7210/layer.tar","3f62f94afe91db9fc1a5fbb8c1fdabb51026adfb1f1b30d1c926802e386aab3f/layer.tar","65272cc726485520b5480fa532809741a7181d32526aa86f492948d0de6024b9/layer.tar"]}] |
I am experiencing the same problem, and am confused about what change causes this.. All my dependencies are specified with commits/hashes, and even if I checkout a commit I know worked yesterday, I still get the same message. Edit:
Edit 2: |
is this still an issue? please reopen if so. |
/home/barriosj/starter_app/app/BUILD.bazel:30:1: //app:app_image: missing input file '@rhel7//image:config.json'
This error occurs with the following rules called in BUILD
There is no config.json in the tar but there is a manifest.json which points to a config json that is in the tar.
The text was updated successfully, but these errors were encountered: