-
Notifications
You must be signed in to change notification settings - Fork 335
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
deploy_site_github() is not finding the ssh key #1206
Comments
@lcolladotor I've just run into the issue and am stuck in the sample place (sshkeys configured and correctly set but build fail). My workaround is to pass deploy:
provider: script
script: Rscript -e 'pkgdown::deploy_site_github(ssh_id = Sys.getenv("TRAVIS_DEPLOY_KEY", ""))'
skip_cleanup: true |
Thanks @zachcp! Your solution worked ^^. |
Thanks for the workaround, but should this really count as closing the issue? I have the same issue using github/travis and this seems to be the most common use case and should imho be fixed in pkgdown (maybe by looking for the ssh key in several locations by default?). At least this solution should be documented in |
I submitted #1217 to expand the current documentation. I opted for this instead of changing the default argument values since (a) the default might work for others and (b) the SSH key name might change in the future (maybe it changed elsewhere and we are the first seeing this issue). |
I finally found this thread after posting on the Travis community site Thanks for both the workaround and the PR! |
use workaround at r-lib/pkgdown#1206
@jimhester would you mind taking a little time to figure out what's going on here? |
It seems ropensci-archive/travis#129 changed the travis package to add the deploy key to the We could I guess change |
Thanks for finding the root of this Jim! In my PR I recommended updating the docs, but I guess that if you have the |
There is already backward comp for the old All users starting fresh right now will not get the As for a solution, you could either check for both locations or change Line 81 in 48e12e0
to suggest stop("No deploy key found, please setup with travis::use_travis_deploy(key_name_private = "id_rsa"), call. = FALSE) Also I don't think #1217 is a good workaround for this problem. Sorry for the trouble. We were not aware that this change might cause issues here. |
We now recommend using the github actions workflow instead; which avoids all this configuration pain. |
Late to the party, I'm trying to figure out what the new GH-action process is. Is there a good reference for how to use the github action workflow (especially helpful would be a reference for people who are comfortable with the "old fashioned" travis workflow). |
@ldecicco-USGS I recommend using the usethis helper |
Brilliant! That's all I need! I wish I would have asked much...much...sooner :) |
Created both an introductory and a developer's notes vignette, updated README and docs with examples, added a second biocViews term, fixed some small bugs/typos. Related links (as many as I could remember): * https://rstd.io/tidytools19 * https://twitter.com/CVWickham * https://twitter.com/hadleywickham * https://www.rstudio.com/products/rstudio/download * https://comunidadbioinfo.github.io/post/building-tidy-tools-cdsb-runconf-2019/#.XrbLMxNKiu4 * http://bioconductor.org/ * https://lcolladotor.github.io/pkgs/ * https://stat.ethz.ch/pipermail/bioc-devel/2020-March/016365.html * https://www.bioconductor.org/help/docker/ * https://stat.ethz.ch/pipermail/bioc-devel/2020-April/016532.html * https://github.com/features/actions * https://stat.ethz.ch/pipermail/bioc-devel/2020-April/016650.html * r-lib/actions#84 * r-lib/usethis#1108 * r-lib/styler#636 * Bioconductor/BiocCheck#57 * Bioconductor/bioconductor.org#54 * http://bioconductor.org/developers/how-to/coding-style/ * https://style.tidyverse.org/ * https://twitter.com/lorenzwalthert * https://twitter.com/mt_morgan * https://docs.travis-ci.com/user/languages/r/ * r-lib/pkgdown#1206 * r-lib/pkgdown#1230 * https://twitter.com/jimhester_ * https://www.jimhester.com/talk/2020-rsc-github-actions/ * https://github.com/Bioconductor/BBS * https://github.com/Bioconductor/packagebuilder * https://www.appveyor.com/ * r-hub/rhub#52 * r-hub/rhub#38 * https://www.tidyverse.org/blog/2020/04/usethis-1-6-0/ * https://github.com/r-lib/actions/tree/master/examples * https://yihui.org/en/2018/03/second-pull-request/ * https://github.com/r-lib/actions/blob/master/examples/check-standard.yaml * https://help.github.com/en/actions * https://ropenscilabs.github.io/actions_sandbox/ * https://twitter.com/seandavis12 * https://github.com/seandavi/BiocActions/blob/master/.github/workflows/main.yml * https://twitter.com/CSoneson * https://github.com/csoneson/dreval/blob/master/.github/workflows/R-CMD-check.yaml * https://bioc-community.herokuapp.com/ * https://github.com/leekgroup/derfinderPlot/blob/master/.github/workflows/check-bioc.yml * https://github.com/LieberInstitute/recount3/blob/master/.github/workflows/check-bioc.yml * https://github.com/hpages * r-lib/actions#68 * r-lib/actions#85 * https://twitter.com/opencpu * https://community.rstudio.com/u/const-ae * https://community.rstudio.com/t/compiler-support-fo-c-14-features-on-windows/57284/4 * r-lib/xml2#296 * r-lib/xml2#302 * https://github.com/r-lib/usethis/blob/master/.github/workflows/R-CMD-check.yaml * https://github.com/r-lib/usethis/commits/master/.github/workflows/R-CMD-check.yaml * https://stat.ethz.ch/pipermail/bioc-devel/2020-April/016703.html * https://stat.ethz.ch/pipermail/bioc-devel/2020-April/thread.html * r-lib/remotes#296 * r-lib/actions#86 * r-lib/covr#427 * https://github.com/r-lib/actions/blob/master/examples/pr-commands.yaml * https://www.digitalocean.com/community/tutorials/how-to-install-git-on-ubuntu-18-04 * r-lib/actions#50 * actions/checkout#238 * https://github.com/rocker-org/rocker-versioned2/blob/master/dockerfiles/Dockerfile_rstudio_4.0.0-ubuntu18.04 * https://twitter.com/niteshturaga * https://twitter.com/cboettig * rocker-org/rocker-versioned#208 * https://github.community/t5/GitHub-Actions/bd-p/actions * https://www.r-consortium.org/blog/2020/03/18/cdsb-diversity-and-outreach-hotspot-in-mexico * https://github.com/maxheld83 * r-lib/actions#87 * https://github.com/yutannihilation
Hi,
I haven't been able to get
pkgdown::deploy_site_github()
working at https://github.com/ComunidadBioInfo/regutools as I keep getting theno deploy key found
error as shown for example at https://travis-ci.com/ComunidadBioInfo/regutools/builds/144932233. Initially I started by usingusethis::use_pkgdown(); usethis::use_pkgdown_travis()
.Initially yesterday, the Travis endpoint was
.org
https://travis-ci.org/ComunidadBioInfo/regutools but I moved it to.com
sincetravis::use_travis_deploy()
by default wouldn't work (travis::use_travis_deploy(endpoint = '.org')
did work). In any case, at.com
, I deleted the keys in both Travis and GitHub as this worked for another user #970: specifically at https://github.com/ComunidadBioInfo/regutools/settings/keys and https://travis-ci.com/ComunidadBioInfo/regutools/settings. I then rantravis::use_travis_deploy()
and it all seems to work (I see the new keys in both Travis and GitHub)At
pkgdown::deploy_site_github()
I see that the error is triggered when!nzchar(ssh_id)
isTRUE
and the defaultssh_id
value isssh_id = Sys.getenv("id_rsa", "")
. At https://travis-ci.com/ComunidadBioInfo/regutools/builds/144932233 I do see thatTRAVIS_DEPLOY_KEY
is set but maybe I need to add some other environment variable on my.travis.yml
file https://github.com/ComunidadBioInfo/regutools/blob/c4e30073e3e0aa0d75aa824c5904b4e1eb721360/.travis.yml. I looked at a few other.travis.yml
files atgithub.com/ropenscilabs
likecircle
andtravis
but they use other tools (tci
as intic::deploy()
for https://github.com/ropenscilabs/circle/blob/master/.travis.yml) which feels like a new rabbit hole :P.If you have other ideas on what else I should try or how I can get more information for debugging this please let me know. Thank you!
Best,
Leo
R session information
The text was updated successfully, but these errors were encountered: