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

GitHub app scm project alt #15481

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from

Conversation

TheRealHaoLiu
Copy link
Member

@TheRealHaoLiu TheRealHaoLiu commented Aug 29, 2024

SUMMARY

Alternative to #15472

  • Add new scm credential type scm_github_app
  • Add ability to generate github app token to clone project with git for github

IMPORTANT: UI change needed currently UI hard codes to namespace='scm' need to change to kind='scm'

ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API
AWX VERSION

ADDITIONAL INFORMATION

- Add github_app_id, github_app_installation_id and github_api_url to scm credential type
- Add ability to generate github app token to clone project with git for github
@TheRealHaoLiu TheRealHaoLiu force-pushed the github-app-scm-project-alt branch 2 times, most recently from ed5905e to b58d221 Compare August 30, 2024 03:40
@webknjaz
Copy link
Member

FTR, I think this is not that different from the first PR in that it mistreats the concept of GitHub Apps. It works, but it's more of a PoC hack rather than full integration with the native concepts GH Apps present.

Copy link

sonarcloud bot commented Sep 3, 2024

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

Successfully merging this pull request may close these issues.

2 participants