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
[breaking] update results field of ResourceList to implement function spec v1 #4248
[breaking] update results field of ResourceList to implement function spec v1 #4248
Conversation
/hold This should be included in the same release as #4249. |
16b72c0
to
9a4e941
Compare
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.
/lgtm
9a4e941
to
f058047
Compare
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.
/lgtm
f058047
to
b386685
Compare
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: KnVerey, mengqiy, monopole, natasha41575 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The changes in this PR contains breaking changes. Is there a way to mark it in kustomize repo? |
I can rename the PR to draw our attention to it. The release is a manual process, and recently we have been combing through the commits when determining whether to bump the versions of modules. It's not the best system though and is prone to human error, but essentially what will happen is we will look through the PRs that are part of the release and bump the version accordingly. @KnVerey @monopole Let's try to remember that this PR means kyaml needs a version bump. |
/unhold |
@natasha41575: This PR has multiple commits, and the default merge method is: merge. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
bb1c833
to
3ea8b79
Compare
/lgtm |
Task 2 in #4024, implementing function spec v1.
There are small discrepancies between kyaml implementation and the current specification of the ResourceList.results field. This PR fixes that.
This may considered a backwards-incompatible change that requires a version bump for kyaml.