Commit a8b02d4
committed
Remove invalid kustomizeconfig from config/webhook
This change has no effect on the output of this kustomization, but fixes
a bug which can be triggered when using it as a base for another
kustomization.
kustomizeconfig contained 3 directives:
* nameReference
* namespace
* varReference
varReference is redundant because vars are no longer used in this
kustomize: they are deprecated and usage was removed some time ago.
nameReference is redundant because the specified configuration is
already in kustomize's defaults. However, nameReference is the only
important transformation here.
namespace is incorrect. It directs the namespace transformer to update
webhooks/clientConfig/service/namespace. However, this is not the
intended function of the namespace transformer: it should only set the
namespace directly on objects and allow references to be updated
automatically by nameReference. Configuring it to update a reference
directly leaves kustomize with inconsistent internal state. Depending on
execution order this can cause a subsequent transformation to fail to
update the reference when it makes further changes to the Service
object.1 parent 57ecea2 commit a8b02d4
2 files changed
+0
-29
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
6 | | - | |
7 | | - | |
8 | | - | |
This file was deleted.
0 commit comments