Skip to content

Nomos CLI: replace APIGROUP with APIVERSION#868

Merged
google-oss-prow[bot] merged 3 commits into
GoogleContainerTools:mainfrom
victorpras:apigroup
Sep 26, 2023
Merged

Nomos CLI: replace APIGROUP with APIVERSION#868
google-oss-prow[bot] merged 3 commits into
GoogleContainerTools:mainfrom
victorpras:apigroup

Conversation

@victorpras

@victorpras victorpras commented Sep 12, 2023

Copy link
Copy Markdown
Contributor

APIGROUP column name has been replaced from Kubernetes v1.20 onwards with APIVERSION. This has broken the usage of api-resources.txt with Nomos command for users using Kubernetes v1.20 or newer.

b/299337751

Comment thread pkg/vet/api_resources_test.go Outdated
Comment thread pkg/vet/api_resources_test.go Outdated
Comment thread pkg/vet/api_resources.go
Comment thread pkg/vet/api_resources.go Outdated
Comment thread e2e/testcases/cli_test.go Outdated
Comment thread pkg/vet/api_resources.go
@nan-yu

nan-yu commented Sep 26, 2023

Copy link
Copy Markdown
Contributor

only one minor nit, otherwise looks good.

@victorpras victorpras requested a review from nan-yu September 26, 2023 20:36

@nan-yu nan-yu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@google-oss-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nan-yu

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow Bot merged commit 1c33b05 into GoogleContainerTools:main Sep 26, 2023
victorpras added a commit to victorpras/kpt-config-sync that referenced this pull request Sep 26, 2023
* Nomos CLI: replace APIGROUP with APIVERSION

* NIT CR feedback

* CR feedback
victorpras added a commit to victorpras/kpt-config-sync that referenced this pull request Sep 26, 2023
* Nomos CLI: replace APIGROUP with APIVERSION

* NIT CR feedback

* CR feedback
google-oss-prow Bot pushed a commit that referenced this pull request Sep 26, 2023
* Nomos CLI: replace APIGROUP with APIVERSION

* NIT CR feedback

* CR feedback
@victorpras victorpras deleted the apigroup branch November 16, 2023 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants