kustomize icon indicating copy to clipboard operation
kustomize copied to clipboard

[WIP] feat: support labels key in transformer configuration

Open stormqueen1990 opened this issue 1 year ago • 2 comments

Allow the usage of a separate transformer configuration for the labels key, similar to what is currently available for commonLabels and commonAnnotations. This aims to provide the same functionality that commonLabels currently provide for labels, since commonLabels is deprecated and slated for removal in a future release.

stormqueen1990 avatar Feb 28 '24 03:02 stormqueen1990

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

k8s-ci-robot avatar Feb 28 '24 03:02 k8s-ci-robot

/lgtm

antoooks avatar Mar 27 '24 15:03 antoooks

/remove-lgtm

stormqueen1990 avatar Mar 27 '24 15:03 stormqueen1990

/unassign @antoooks /cc @koba1t

stormqueen1990 avatar Apr 07 '24 22:04 stormqueen1990

This PR has multiple commits, and the default merge method is: merge. You can request commits to be squashed using the label: tide/merge-method-squash

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.

k8s-ci-robot avatar Apr 09 '24 02:04 k8s-ci-robot

/assign koba1t

koba1t avatar Apr 17 '24 15:04 koba1t

Hi @stormqueen1990 I think this PR looks good! I added a few minor comments. Could you check those and fix the conflict?

koba1t avatar Apr 18 '24 18:04 koba1t

Hi @stormqueen1990 I think this PR looks good! I added a few minor comments. Could you check those and fix the conflict?

Done! Let me know if there's anything else that needs to be addressed 😄

stormqueen1990 avatar Apr 19 '24 02:04 stormqueen1990

/label tide/merge-method-squash

koba1t avatar Apr 25 '24 07:04 koba1t

Thanks for your work! /approve

koba1t avatar Apr 25 '24 07:04 koba1t

/lgtm

koba1t avatar Apr 25 '24 07:04 koba1t

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: koba1t, stormqueen1990

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 /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Apr 25 '24 07:04 k8s-ci-robot