Skip to content

Remove kustomize#436

Merged
rmoioliveira merged 1 commit into
rancher:mainfrom
rmoioliveira:remove-kustomize
Apr 27, 2026
Merged

Remove kustomize#436
rmoioliveira merged 1 commit into
rancher:mainfrom
rmoioliveira:remove-kustomize

Conversation

@rmoioliveira
Copy link
Copy Markdown
Contributor

This PR is related to rancher/rancher#54750.

Comment thread package/helm-cmd
@mallardduck
Copy link
Copy Markdown
Member

mallardduck commented Apr 24, 2026

Per feedback from slack - we might want to add a kustomize shell script to be a wrapper.


kustomize wrapper script

A minimal shell script installed at /usr/local/bin/kustomize that proxies the build subcommand to kubectl kustomize and rejects everything else.

Behaviour:

  • kustomize build [args...]exec kubectl kustomize [args...], passing all remaining flags and arguments through unchanged
  • kustomize (no subcommand) → error: no subcommand given
  • kustomize <anything-else> → error: subcommand not supported, only build is available
  • Exit code mirrors kubectl kustomize on success paths; non-zero on error paths

Non-goals:

  • No emulation of create, edit, cfg, fn, localize, version, or completion
  • No flag parsing beyond identifying the first positional argument as the subcommand

@mallardduck mallardduck self-requested a review April 24, 2026 17:26
@rmoioliveira
Copy link
Copy Markdown
Contributor Author

Per feedback from slack - we might want to add a kustomize shell script to be a wrapper.
....

@mallardduck, I implemented the script in this commit. Can you review it?

Comment thread package/kustomize Outdated
Copy link
Copy Markdown
Member

@mallardduck mallardduck left a comment

Choose a reason for hiding this comment

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

lgtm

@mallardduck mallardduck marked this pull request as ready for review April 24, 2026 22:05
Replaces `kustomize` with a minimal shell script that proxies the
`build` subcommand to `kubectl kustomize` and rejects everything else.

* Remove kustomize.sh script
* Remove kustomize binary
* Remove kustomize from helm-cmd
* Add kustomize wrapper script

See rancher/rancher#54676

Signed-off-by: Rodolfo Mói de Oliveira <rodolfo.oliveira@suse.com>
@rmoioliveira rmoioliveira merged commit 4126162 into rancher:main Apr 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants