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

Add support for installing a GitHub App under an organization #263

Conversation

tampakrap
Copy link

what

Adding a new optional environment variable ATLANTIS_GH_ORG, which is equivalent to the --gh-org argument.
Relevant documentation: https://www.runatlantis.io/docs/server-configuration.html#gh-org

why

The installation of the GitHub App under a GitHub organization needs this extra argument / env var on the initial run.
Relevant documentation: https://www.runatlantis.io/docs/access-credentials.html#github-app

After the initial run, and after the GitHub App is installed under the desired organization, this env var can be commented out / unset again.

tests

I have tested locally by successfully creating a new GitHub App under an organization.

references

https://www.runatlantis.io/docs/server-configuration.html#gh-org
https://www.runatlantis.io/docs/access-credentials.html#github-app

@tampakrap tampakrap requested a review from a team as a code owner February 28, 2023 14:59
@tampakrap tampakrap changed the title Add support for installing a private GitHub App under an organization Add support for installing a GitHub App under an organization Feb 28, 2023
@tampakrap tampakrap force-pushed the add_support_for_gh_app_org_installation branch from 5aca7cf to ed27ad8 Compare February 28, 2023 17:40
@@ -8,6 +8,7 @@

# Replace this with your own repo allowlist:
orgAllowlist: <replace-me>
#githubAppOrg: myorgname
Copy link
Member

Choose a reason for hiding this comment

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

Could we move this to githubApp?

Copy link
Author

@tampakrap tampakrap Apr 28, 2023

Choose a reason for hiding this comment

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

Unfortunately it can't be under githubApp, because the org needs to be set up before the github app is created and set up in atlantis

Copy link
Member

Choose a reason for hiding this comment

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

Hi @tampakrap, can you please clarify why it wouldn't work? We can still set the value on the ATLANTIS_GH_ORG, it's just a different name for the variable.

@jseiser
Copy link
Contributor

jseiser commented Sep 11, 2023

Cant you just set this under https://github.com/runatlantis/helm-charts/blob/main/charts/atlantis/values.yaml#L384

@tampakrap
Copy link
Author

Cant you just set this under https://github.com/runatlantis/helm-charts/blob/main/charts/atlantis/values.yaml#L384

sure, that is what I'm currently doing

@jamengual jamengual added the waiting-for-review Waiting for a review from a maintainer label Nov 16, 2023
@GMartinez-Sisti GMartinez-Sisti added waiting-on-response Waiting for a response from the user and removed waiting-for-review Waiting for a review from a maintainer labels Mar 24, 2024
Copy link

This issue is stale because it has been open for 1 month with no activity. Remove stale label or comment or this will be closed in 1 month.

@github-actions github-actions bot added the Stale label Apr 25, 2024
@github-actions github-actions bot closed this May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale waiting-on-response Waiting for a response from the user
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants