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

Support Github App Values #11

Closed
heruscode opened this issue Nov 6, 2020 · 7 comments
Closed

Support Github App Values #11

heruscode opened this issue Nov 6, 2020 · 7 comments

Comments

@heruscode
Copy link
Contributor

Hello,

We would like to use this chart with our Github authentication being a Github App instead of a common user.

Would you like a PR for this?

@Eugst
Copy link

Eugst commented Mar 3, 2021

for github App setup I found workaround on a current setup without any changes by providing helm chart values:

environmentRaw:
  - name: ATLANTIS_GH_APP_ID
    value: "{{githubApp_id}}"
  - name: ATLANTIS_GH_APP_KEY_FILE
    value: "/var/github-app/key.pem"
  - name: ATLANTIS_WRITE_GIT_CREDS
    value: "true"
environmentSecrets:
  - name: ATLANTIS_GH_WEBHOOK_SECRET
    secretKeyRef:
      name: github-app
      key: webhook-secret

extraManifests:
 - apiVersion: v1
   kind: Secret
   metadata:
    name: github-app
   data:
    key.pem: {{githubApp_pem}}
    webhook-secret: {{githubApp_secret}}

extraVolumes:
  - name: github-app-vol
    secret:
      secretName: github-app
extraVolumeMounts:
  - name: github-app-vol
    mountPath: "/var/github-app"

Later(when I will have a bit more spare time) i will add PR with extending chart values.

@lkysow
Copy link
Member

lkysow commented Mar 14, 2021

@Eugst
Copy link

Eugst commented Mar 14, 2021

@lkysow its supported by atlasian app, but helm doesn’t have values to forward it.

@blacksd
Copy link
Contributor

blacksd commented Apr 2, 2021

@Eugst I went ahead and opened a PR - thanks for the workaround!

spirosoik pushed a commit to spirosoik/helm-charts that referenced this issue Aug 3, 2021
* add 2 images

* create gif, update readme

* Make readme template helm V3 style

* add this back

* massage for lint

* regenerate readme after changing template

* add one more alt text

* bump all versions

* update these again

* update this chart with new templated values

* change this

* create banner image, remove old images, add note about generating readmes

* fix domain typo
@fishpen0
Copy link

fishpen0 commented Dec 28, 2021

Now that https://github.com/runatlantis/atlantis/pull/1706/files (runatlantis/atlantis#1636) is merged, it seems like the chart can just be updated to support --gh-app-key being passed in without all the file nonsense.

@frank-bee
Copy link

I tried to use the latest chart with githuh app values configured.
At least the github org cannot be configured yet which is required in our case (in the underlying go app there is a parameter for the org)

@nitrocode
Copy link
Member

Isnt this now already supported by the continuation of closed pr #46 in merged pr #76?

@lkysow lkysow closed this as completed Aug 22, 2022
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

Successfully merging a pull request may close this issue.

7 participants