Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

[ci] Add ssh token for publishing gh-pages #10338

Merged
merged 5 commits into from
Nov 23, 2021
Merged

[ci] Add ssh token for publishing gh-pages #10338

merged 5 commits into from
Nov 23, 2021

Conversation

alvicsam
Copy link
Contributor

Using ssh key is more secure because it has less scope of permissions

paritytech/ci_cd#259

@alvicsam alvicsam added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Nov 22, 2021
@alvicsam alvicsam requested a review from a team as a code owner November 22, 2021 16:13
@TriplEight
Copy link
Contributor

please try to substitute the ssh setup with adding the key to the repo config, it should be more robust

@alvicsam alvicsam requested a review from TriplEight November 23, 2021 11:20
@alvicsam alvicsam changed the title [ci] Added ssh token for publishing gh-pages [ci] Add ssh token for publishing gh-pages Nov 23, 2021
@TriplEight
Copy link
Contributor

yeah looks better, can you please add this to the templates, too?

@alvicsam
Copy link
Contributor Author

yeah looks better, can you please add this to the templates, too?

sure, I'll make changes to other repos and add to documentation

@alvicsam alvicsam merged commit 1d8f7bf into master Nov 23, 2021
@alvicsam alvicsam deleted the as-ci-add-ssh-key branch November 23, 2021 16:04
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
* [ci] Added ssh token for publishing gh-pages

* changed ssh commands for gh-pages

* return github_token back to vault secrets

* check simnet-tests-quick without vault secrets

* remove vault secrets from simnet jobs
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* [ci] Added ssh token for publishing gh-pages

* changed ssh commands for gh-pages

* return github_token back to vault secrets

* check simnet-tests-quick without vault secrets

* remove vault secrets from simnet jobs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants