-
Notifications
You must be signed in to change notification settings - Fork 582
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
Repository/image name should be lowercased automatically. #37
Comments
For those interested in a workaround until/if this Action supports this out-of-the-box. I just added a preceding step to pipe
|
Version 2 has been merged to the main branch and is therefore available via As a reminder, this new version changes drastically and works with 3 new actions (login, setup-buildx and setup-qemu) that we have created. Many usage examples have been added to handle most use cases. This new version should fix your issue. Don't hesitate if you have any questions. |
I'm somewhat confused i still get this error with v2, as i don't set any repository field I'm a bit confused what i need to change.. https://github.com/AnderssonPeter/test-actions/runs/1134569521?check_suite_focus=true Also it would be nice if one of the examples in the readme was how to set it up with github packages. |
Hi @AnderssonPeter,
This line in your workflow should reflect your repository name on DockerHub, not the one from GitHub actually.
Yeah maybe we could add an example to use this action with the GitHub Container Registry. In the meantime you can take a look at this workflow. See also this section about usage of our login action. |
Thanks for the fast response, im trying to push to github instead of docker hub. |
@crazy-max Thanks for you help, i managed to get it working, one thing it would be nice if the tool converted the repository name to lowercase, that way i could use Again thank you for the fast help! |
@AnderssonPeter Is it still not the case? I filed this issue to get this Action to already lowercase the name. I haven't tried the v2 yet though. |
No, it doesn't work. You can't use |
This is the "modern" version for setting env. (see https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-commands-for-github-actions#environment-files )
|
Man that's so ugly. I wish they would add a simple lowercase() function |
…g tags. Hence, we can cast the Image_Repository, and image tag to lower case as per: docker/build-push-action#37 (comment)
The comment above says this was implemented with the v2 of the docker/build-push-action. This doesn't seem to be the case -- I still get an error when the username of the owner of the repository is not all lowercase. |
Attempt to apply workaround in docker/build-push-action#37
Attempt to apply workaround in docker/build-push-action#37
I have no idea if the same rule applies to username or registry, but I have a failing build due to this. Note that repository name is feeded with
${{ github.repository }}/gateway
. Currently, there is notoLower
function in Actions.The text was updated successfully, but these errors were encountered: