-
Notifications
You must be signed in to change notification settings - Fork 8
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(entities): support terraform in config card [khcp-12445] #1549
Conversation
...ges/entities/entities-shared/src/components/entity-base-config-card/EntityBaseConfigCard.vue
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good.
However, id field is missing in the Terraform format for any entity.
I am not familiar with Terraform, but
- why is it my_** for every resource ? eg: resource "konnect_gateway_service" "my_service" {
- same with my_** in control_plane_id = konnect_gateway_control_plane.my_konnect_cp.id
packages/entities/entities-upstreams-targets/src/components/TargetForm.vue
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CaCertificate = 'ca_certificate', | ||
Certificate = 'certificate', | ||
Consumer = 'consumer', | ||
ConsumerGroup = 'consumer_group', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kaiarrowood We need an entry for ConsumerCredential
to display Terraform since we do display JSON/YAML Configuration in the ConsumerCredentialForm
ConsumerGroup = 'consumer_group', | |
ConsumerGroup = 'consumer_group', | |
ConsumerCredential = 'consumer_credential', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consumer credentials are plugins 🤔 I need to look into this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will fix in a future update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kaiarrowood We need to display Terraform in the ConsumerCredentialForm since we do display JSON/YAML Configuration
packages/entities/entities-certificates/src/components/CACertificateConfigCard.vue
Show resolved
Hide resolved
Co-authored-by: Manjiri Tapaswi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! Great work ⭐
Summary
Add support to view terraform configuration for entities.
For KHCP-12445.
This work is behind a FF:
khcp-12445-terraform-config-details
Adoption PR: https://github.com/Kong/konnect-ui-apps/pull/3935
Resources