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

feat: data organization secret repositories #2247

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

moutonjeremy
Copy link

Resolves no issue

This PR add a new data for retrieve the list of repositories of a secret of an organisation.


Before the change?

  • It's not possible to have the repositories list for a secret.

After the change?

  • Data action for get the repositories list for a secret

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@kfcampbell
Copy link
Member

kfcampbell commented May 3, 2024

Can you talk to me a little more about the use case for this and why you want to add it to the provider? In general, I'm a little wary of adding new resources/data sources without issues and discussion beforehand.

@moutonjeremy
Copy link
Author

For my use case, we have a process to init/create (automatically) services (and repos) via a template but we must give access to a secret for each repo.
The service creation process is independent for each service, so we cannot manage access to the secret in centralized mode, so we must know the previous state before making a modification.
This is not an ideal situation.

@moutonjeremy
Copy link
Author

Hello, will the PR be reviewed or will it be refused?

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 this pull request may close these issues.

None yet

2 participants