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

Build and push Docker image not working #1

Open
lwaldron opened this issue May 18, 2022 · 8 comments
Open

Build and push Docker image not working #1

lwaldron opened this issue May 18, 2022 · 8 comments
Assignees

Comments

@lwaldron
Copy link
Contributor

E.g. see https://github.com/waldronbios2/cunybios2/runs/6498265129?check_suite_focus=true

I am having difficulties, @jwokaty can you help?

@lwaldron
Copy link
Contributor Author

Related to problems I have had with waldronlab/curatedMetagenomicDataAnalyses#35

@jwokaty
Copy link
Collaborator

jwokaty commented May 20, 2022

I'll look into these.

@jwokaty
Copy link
Collaborator

jwokaty commented Jul 23, 2022

@lwaldron I made some changes, but the deployment likely failed because it's expecting your credentials. Could I ask you to try running the action?

@jwokaty
Copy link
Collaborator

jwokaty commented Aug 9, 2022

@lwaldron Can you run the workflow https://github.com/waldronbios2/cunybios2/actions/workflows/build.yaml to see if that clears the insufficient_scope permissions error?

@lwaldron
Copy link
Contributor Author

Thank you @jwokaty , running at https://github.com/waldronbios2/cunybios2/actions/runs/2831100806. I also just gave you admin access to this repository to hopefully allow you to do anything I could do.

@jwokaty
Copy link
Collaborator

jwokaty commented Aug 10, 2022

@lwaldron Thank you. I noticed it still failed, so I will continue investigating.

Note (to @jwokaty): See docker/build-push-action#160. May need to change the tag.

@jwokaty
Copy link
Collaborator

jwokaty commented Aug 15, 2022

@lwaldron What should Ncpus=max(1, parallel::detectCores(), na.rm=TRUE) be doing?

I'm getting the following error on build:

#10 [6/7] RUN Rscript -e "devtools::install('.', dependencies=TRUE, build_vignettes=FALSE, Ncpus=max(1, parallel::detectCores(), na.rm=TRUE))"
#10 41.11 Error: JSON: EXPECTED value GOT <
#10 41.11 Execution halted
#10 41.11 Warning message:
#10 41.11 1 components of `...` were not used.
#10 41.11 
#10 41.11 We detected these problematic arguments:
#10 41.11 * `Ncpus`
#10 41.11 
#10 41.11 Did you misspecify an argument? 
#10 ERROR: process "/bin/sh -c Rscript -e \"devtools::install('.', dependencies=TRUE, build_vignettes=FALSE, Ncpus=max(1, parallel::detectCores(), na.rm=TRUE))\"" did not complete successfully: exit code: 1

@lwaldron
Copy link
Contributor Author

Huh, trying to automatically set number of cores for compiling packages, but it's really not necessary you could just get rid of that argument. They're probably all installed from binaries anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants