-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Manifest creation fails #131
Comments
The repository mentioned has |
I can try modify workflow and add |
Please do! If it still continues to error, please tell so I can try to find some solutions! |
The action actually forces amend: true in the |
Just tested with |
Ok, I'm waiting. |
I have started to work on this in PR #133 (and I'll work on it when I get home). |
I just got the issue right now:
The issue is that the tag was already merged with tags: <registry>/<image>
# or
platforms: linux/amd64 instead of tags: |
a,
b
# or
platforms: linux/amd64,linux/arm64 I'm sorry for all the hoops, but this is also really difficult to debug (since I didn't know how to reproduce this), but maybe try that solution or |
Not easier for me because Look at workflow graph to understand: https://github.com/PurpleI2P/i2pd/actions/runs/4001302421 |
Untill last week everything works this way:
|
Ah, I see that you only use one platform but multiple tags! I'm really sorry that I made this a bit worse (with a refactor that I been wanting to do for a while, but didn't have the time until now). I'll try to find a workaround for this, sorry for all the hoops! ;w; |
btw, I think that |
And one more link: https://docs.docker.com/engine/reference/commandline/manifest/#create-and-push-a-manifest-list |
Hello. I have also encountered this problem and have found a tentative solution, which I will share with you. The solution is to add - name: Build and push Docker image
uses: docker/build-push-action@v3.3.0
with:
context: .
push: true
platforms: ${{ matrix.platform }}
tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}
provenance: false Related issues may be found here: docker/build-push-action#771 For specific fixes, see the following Pull requests: tomacheese/twitter-dm-memo#31 |
Thank you for a solution! I'll add it to the README and keep this issue open (or not? I don't know in this case) |
@book000 thank you for solution, I'll try it soon. @auguwu I have question about options names, I agree with add:
add2: |
Isn't |
It works. Thank you again. |
"X is a manifest list" error Solution: disable provenance (Noelware/docker-manifest-action#131) as recommended in https://github.com/Noelware/docker-manifest-action (and use this action as well)
Hello.
Thank you for your workaround with docker containers merging with manifest.
Lately it start fail when trying to create manifest. Example output can be found here: https://github.com/PurpleI2P/i2pd/actions/runs/4001302421/jobs/6867928655
How can we fix this?
The text was updated successfully, but these errors were encountered: