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

Use databags for githubapp yaml in ext #25

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ataylor-r7
Copy link
Contributor

the pem files and a temporary pac for github has been added to a new knife data bag called github-apps. the token needs to be updated to use a token from a service account

my one concern is that i know github-connector is open source so i'm not sure if we want to put information / a template file that contains information kind of hinting at how we're using github apps in our open source repo?

@@ -73,19 +73,34 @@
revision attrs['revision']
ssh_wrapper "/home/#{node['github_connector']['user']}/.ssh/#{engine}_ssh_wrapper.sh"
action :sync
notifies :run, 'rvm_shell[github-connector-gems]', :immediately
notifies :run, 'rvm_shell[github-csonnector-gems]', :immediately

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a typo.

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

Successfully merging this pull request may close these issues.

None yet

2 participants