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 In ApplicationSet Pull Request generator #9078

Closed
agaudreault opened this issue Apr 12, 2022 · 4 comments
Closed

Support GitHub App In ApplicationSet Pull Request generator #9078

agaudreault opened this issue Apr 12, 2022 · 4 comments
Labels
component:applications-set Bulk application management related enhancement New feature or request

Comments

@agaudreault
Copy link
Member

Summary

The Pull Request generator of the ApplicationSet only supports a Personal Access Token in the tokenRef. ArgoCD already have the logic to authenticate with a GitHub App.

Motivation

Using a GitHub App is more secure and allows for more rate limits

Proposal

Reference to a secret containing the GitHub App information in the format already defined by ArgoCD

  - pullRequest:
      github:
        owner: org
        repo: repo
        # Reference to a Secret containing github app information. (optional)
        appRef:
          secretName: github-app
        # Labels is used to filter the PRs that you want to target. (optional)
        labels:
          - preview
@agaudreault agaudreault added the enhancement New feature or request label Apr 12, 2022
@crenshaw-dev crenshaw-dev added the component:applications-set Bulk application management related label Apr 12, 2022
@BeyerJC
Copy link

BeyerJC commented Apr 13, 2022

I opened the same proposal in the application controller repo --> argoproj/applicationset#565

Would be a great feature !

@iamstarkov
Copy link

and it would be great to support GitHub App In ApplicationSet SCM generator too

@iamstarkov
Copy link

@BeyerJC applicationset repo is deprecated afaik

@iamstarkov
Copy link

FYI #10092

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:applications-set Bulk application management related enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants