Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

chore: update k8s libs to 1.24#577

Merged
k8s-ci-robot merged 1 commit into
kubernetes-sigs:masterkubernetes-sigs/cli-utils:masterfrom
mengqiy:depbumpmengqiy/cli-utils:depbumpCopy head branch name to clipboard
May 12, 2022
Merged

chore: update k8s libs to 1.24#577
k8s-ci-robot merged 1 commit into
kubernetes-sigs:masterkubernetes-sigs/cli-utils:masterfrom
mengqiy:depbumpmengqiy/cli-utils:depbumpCopy head branch name to clipboard

Conversation

@mengqiy

@mengqiy mengqiy commented Apr 20, 2022

Copy link
Copy Markdown
Member

We want to bump the k8s dependencies, since when want to import a newer kyaml version in kpt. This kyaml version is only compatible with k8s v0.24+ due import path change in kubernetes/kube-openapi#285.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Apr 20, 2022
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 20, 2022
Comment thread go.mod Outdated
Comment thread cmd/diff/cmddiff.go
}

o.DryRunVerifier = resource.NewDryRunVerifier(o.DynamicClient, o.DiscoveryClient)
o.DryRunVerifier = resource.NewQueryParamVerifier(o.DynamicClient, f.OpenAPIGetter(), resource.QueryParamDryRun)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This change is similar to kubernetes/kubectl@7e68149

@mengqiy

mengqiy commented Apr 20, 2022

Copy link
Copy Markdown
Member Author

/retest

}
if t.SelfLink != "" {
u.SetSelfLink(t.SelfLink)
if t.SelfLink != "" { // nolint:staticcheck

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

SA1019: t.SelfLink is deprecated: selfLink is a legacy read-only field that is no longer populated by the system. +optional (staticcheck)

This field is deprecated. Skipping it.

@mengqiy

mengqiy commented Apr 20, 2022

Copy link
Copy Markdown
Member Author

/cc @mortent

@k8s-ci-robot
k8s-ci-robot requested a review from mortent April 20, 2022 21:24
@mengqiy

mengqiy commented Apr 20, 2022

Copy link
Copy Markdown
Member Author

/cc @karlkfi

@yuwenma

yuwenma commented Apr 20, 2022

Copy link
Copy Markdown

This PR will fix the go mod conflicts we had in kpt and porch, and it is critical for all the users who use both KRM functions (new sdk) and cli-utils!

@karlkfi

karlkfi commented Apr 20, 2022

Copy link
Copy Markdown
Contributor

Historically we haven't taken new k8s versions until they're fully released and can be picked up by the known downstream clients (kpt, Config Sync, GitLab Agent). I'm not sure how to handle a release candidate. My best advice would be to temporarily vendor cli-utils with your changes into your downstream client, until cli-util is updated with the change and a new release is cut.

@karlkfi
karlkfi requested a review from Liujingfang1 April 20, 2022 23:55
@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 23, 2022
@karlkfi karlkfi changed the title bump to k8s 1.24 feat: update k8s libs to 1.24 May 12, 2022
@karlkfi karlkfi changed the title feat: update k8s libs to 1.24 chore: update k8s libs to 1.24 May 12, 2022
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 12, 2022
@mengqiy

mengqiy commented May 12, 2022

Copy link
Copy Markdown
Member Author

Rebased. PTAL

@karlkfi

karlkfi commented May 12, 2022

Copy link
Copy Markdown
Contributor

/hold

Blocking until #572 is merged.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 12, 2022

@karlkfi karlkfi 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

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 12, 2022

@mortent mortent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/approve

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: karlkfi, mengqiy, mortent

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 12, 2022
@karlkfi

karlkfi commented May 12, 2022

Copy link
Copy Markdown
Contributor

/unhold

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 12, 2022
@k8s-ci-robot
k8s-ci-robot merged commit 2d68222 into kubernetes-sigs:master May 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.