fix: Replace deprecated kubernetes_service_account with kubernetes_service_account_v1#2560
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request addresses a bug by updating a deprecated resource in the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request updates the deprecated kubernetes_service_account resource to kubernetes_service_account_v1. To ensure the module remains consistent and functional, the documentation and examples need to be updated. A comment has been added to highlight the specific files requiring updates.
| } | ||
|
|
||
| resource "kubernetes_service_account" "main" { | ||
| resource "kubernetes_service_account_v1" "main" { |
There was a problem hiding this comment.
The change to kubernetes_service_account_v1 requires updates to the module's documentation and examples to maintain consistency and functionality. Specifically:
modules/workload-identity/README.md: Update examples and error message snippets to reflect the use ofkubernetes_service_account_v1.examples/workload_identity/main.tf: Update to usekubernetes_service_account_v1and remove thesecretblock, as it's not supported and deprecated in recent Kubernetes versions.
f80aea1 to
09503aa
Compare
|
@gcp-terraform-admins Could you please trigger the Cloud Build check (/gcbrun)? I've addressed all review feedback. |
|
@gcp-terraform-admins Could you please trigger the Cloud Build check (/gcbrun)? I've addressed all review feedback. |
apeabody
left a comment
There was a problem hiding this comment.
Thanks for the contribution @nacholiya!
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request updates the kubernetes_service_account resource to kubernetes_service_account_v1 across the examples, README, and the main module. A logical error was identified where the output_k8s_namespace local variable in modules/workload-identity/main.tf was incorrectly changed to reference the service account's name instead of its namespace.
|
Hi maintainers, I've addressed the requested changes and triggered /gcbrun. |
|
/gemini review |
|
/gcbrun |
There was a problem hiding this comment.
Code Review
This pull request updates the Terraform configuration to use the kubernetes_service_account_v1 resource instead of the deprecated kubernetes_service_account resource across the main module and examples. I have no feedback to provide as the changes correctly implement the required resource migration.
|
Hi maintainers, could you please trigger the CI by running |
|
/gcbrun |
|
Hi maintainers, I’ve addressed the remaining changes including removing version pinning in the examples. Could you please trigger the CI again with |
Description
Replaced deprecated kubernetes_service_account with kubernetes_service_account_v1 in workload-identity module.
Type of Change