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

[RFC-007] Flux cli support for GitHub app authentication #5103

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

dipti-pai
Copy link
Member

@dipti-pai dipti-pai commented Dec 9, 2024

  • Add flux create secret githubapp command that accepts and validates the inputs to create a github app secret with options to export the secret yaml or create the secret directly in the Kubernetes cluster

  • Add tests for flux create secret githubapp command

  • Update flux create source git command to accepts and validate github provider with options to export the source yaml or create the github GitRepository source directly in the Kubernetes cluster.

  • Add tests for flux create source git command for github provider.

Closes #5097

@dipti-pai dipti-pai changed the title Add flux create secret githubapp and update flux create source git to include github provider [RFC-007] Flux cli support for GitHub app authentication Dec 9, 2024
@stefanprodan
Copy link
Member

This is on hold until we release source-controller v1.5.0

@stefanprodan stefanprodan added the hold Issues and pull requests put on hold label Dec 10, 2024
Copy link
Member

@matheuscscp matheuscscp left a comment

Choose a reason for hiding this comment

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

@dipti-pai Please rebase 🙏

I tested this PR and was able to both create a github app secret in-cluster and with --export, and also create a GitRepository source with provider github 👍

Left a comment about the create secret githubapp command example.

@dipti-pai dipti-pai force-pushed the github-app-auth branch 2 times, most recently from ec38a49 to 1776eed Compare February 11, 2025 20:00
@matheuscscp
Copy link
Member

@dipti-pai I'm so sorry I asked you to rebase the other day, we could have waited until source-controller was released 😅

source-controller 1.5.0 is now released and bumped in the dependencies already, so if you rebase one last time and remove the temporary dependency on your fork we can finally merge this PR 🙏

…the following changes -

- Add flux create secret githubapp command that accepts and validates the inputs to create a github app secret with options to export the secret yaml or create the secret directly in the Kubernetes cluster

- Add tests for flux create secret githubapp command

- Add flux create source git command that accepts and validates the inputs to create a gitrepository source with for github provider with options to export the source yaml or create the github gitrepository source directly in the Kubernetes cluster.

- Add tests for flux create source git command for github provider.

Signed-off-by: Dipti Pai <[email protected]>
@dipti-pai
Copy link
Member Author

@dipti-pai I'm so sorry I asked you to rebase the other day, we could have waited until source-controller was released 😅

source-controller 1.5.0 is now released and bumped in the dependencies already, so if you rebase one last time and remove the temporary dependency on your fork we can finally merge this PR 🙏

No worries, rebased :) Thanks.

Copy link
Member

@matheuscscp matheuscscp left a comment

Choose a reason for hiding this comment

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

Thanks @dipti-pai ❤️

@matheuscscp matheuscscp merged commit ce50286 into fluxcd:main Feb 13, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold Issues and pull requests put on hold
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RFC-007] Add command to generate GitHub App secret
3 participants