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.

Sign up for GitHub

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

Jump to bottom

Create a terraform module for dex-auth charm #226

Closed
NohaIhab opened this issue Aug 26, 2024 · 2 comments
Closed

Create a terraform module for dex-auth charm #226

NohaIhab opened this issue Aug 26, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@NohaIhab
Copy link
Contributor

Context

According to the terraform modules organization spec, this is the first step to provide a terraform solution module. We need to create a terraform module for each charm.

What needs to get done

Create a terraform module for this charm that complies with the spec.

Definition of Done

Charm has a terraform module
@NohaIhab NohaIhab added the enhancement New feature or request label Aug 26, 2024
@syncronize-issues-to-jira Syncronize issues to Jira
Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/KF-6162.

This message was autogenerated

DnPlas added a commit that referenced this issue Sep 18, 2024
@DnPlas
feat: add Terraform module for dex-auth
6fe023d 
This commit adds the terraform/ directory to the root of the repository to host
the Terraform module of this charm. This follows the standard set in CC006.
For more information please also refer to canonical/argo-operators/pull/198.

Fixes #226
@DnPlas DnPlas mentioned this issue Sep 18, 2024
Merged
DnPlas added a commit that referenced this issue Sep 24, 2024
@DnPlas
feat: add Terraform module for dex-auth (#228)
b923697 
* feat: add Terraform module for dex-auth

This commit adds the terraform/ directory to the root of the repository to host
the Terraform module of this charm. This follows the standard set in CC006.
For more information please also refer to canonical/argo-operators/pull/198.

Part of #226
DnPlas added a commit that referenced this issue Sep 24, 2024
@DnPlas
feat: add Terraform module for dex-auth (#228)
64f5881 
* feat: add Terraform module for dex-auth

This commit adds the terraform/ directory to the root of the repository to host
the Terraform module of this charm. This follows the standard set in CC006.
For more information please also refer to canonical/argo-operators/pull/198.

Part of #226
@DnPlas DnPlas mentioned this issue Sep 24, 2024
Merged
@DnPlas
Copy link
Contributor

DnPlas commented Sep 24, 2024

Fixed by #228 in main and #229 in track/2.39

@DnPlas DnPlas closed this as completed Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@DnPlas @NohaIhab