Closed bschilder closed 2 years ago
I think I figured out what was going on here.
I interpreted DOCKER_PASSWORD
as "my DockerHub password".
In actuality, it should be a DockerHub access token generated via the following steps.
Could you please change the variable name (e.g. DOCKER_TOKEN
) and add some inline comments to make this a bit more clear?
Best, Brian
That makes sense.
For context, the wording comes from https://github.com/seandavi/BuildABiocWorkshop/blob/master/.github/workflows/basic_checks.yaml#L82-L83 which was the recommended example repository https://github.com/seandavi/BuildABiocWorkshop for making Bioconductor workshops for BioC2020.
I added some comments on the GHA template to help guide future users around this issue. I didn't change the name from DOCKER_PASSWORD
to DOCKER_TOKEN
to avoid breaking backwards compatibility in case others were already using the DOCKER_PASSWORD
GitHub secret.
Sounds great, thank you!
I'm using the GHA yaml produced by
biocthis::use_bioc_github_action()
. I modified it after initially generating it with the default settings so that it would build and push the container to DockerHub.I've done the following:
DOCKER_USERNAME
andDOCKER_PASSWORD
as GH Secrets in my repo.run_docker: 'true'
!contains(github.event.head_commit.message, '/nodocker')
condition.uses: docker/build-push-action@v2
.All of these had the same outcome (the DockerHub push section of the workflow wasn't run).
https://github.com/neurogenomics/orthogene/runs/3987203006?check_suite_focus=true
Here is the yaml workflow file I'm using: https://github.com/neurogenomics/orthogene/blob/main/.github/workflows/check-bioc.yml
Do you know what might be going on here?
Many thanks in advance, Brian
R session information
The output of
sessioninfo::session_info()
includes relevant GitHub installation information and other details that are missed bysessionInfo()
.