-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Update Namespace and Name simultaneously #1379
Conversation
/assign @Liujingfang1 |
Confirmed this fixes the issue described in #1377. Thanks for fixing this before I had a chance to look into what was going on. 😄 |
- Removed RoleBinding and Webhook specific code in the namespacetransformer. That code was attempting to perform the task of the namereference - Updated namereference transformer configuration to suppport the Webhooks. - Prevent the namereference from wiping out the namespace value if no referral candidate was selected - Added unit tests.
Add tests combining prefixsuffix and namespace transformers.
@Liujingfang1 Gentle bump of the PR. I did update the code according to the review and rebased the PR. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jbrette, Liujingfang1 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 |
Update Namespace and Name simultaneously
Namespace, namesuffix and nameprefix transformation should be handled in same way.
Removed RoleBinding and Webhook specific code in the namespacetransformer.
That code was attempting to perform the task of the namereference
Updated namereference transformer configuration to suppport the
Webhooks.
Prevent the namereference from wiping out the namespace value if
no referral candidate was selected
Added unit tests.
This PR is a fix for: