-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Add nameref for openshift routes? #97
Comments
This would be solved also by #91 probably |
@schweikert Adding name prefix should work for Route kind since kustomize doesn't really check the kind name for prefixing part. |
The problem is spec.to.name, which is the name of a service that gets renamed by kustomize |
#105 can solve the problem.
Get the open API definition of it and save it to a file Then in
Then |
Awesome, thanks! Looking forward to trying it out. |
#42 merged. Here is an example showing how it works https://github.com/kubernetes-sigs/kustomize/tree/master/pkg/commands/testdata/testcase-crds/crd |
For the records: it works for this use case :) Thanks @Liujingfang1 ! I have written here detailed instructions on how to use it for OpenShift routes: Maybe we can extend that repo in the future to contain more tuning tips for using kustomize with openshift. |
@Liujingfang1 I realise this request has been addressed/closed but I was wondering if there is an easier way to create JSON CRD files for OpenShift extensions rather than to add the properties by hand? Like, would it be possible to simply generate a Kustomize JSON CRD for the entire DeploymentConfig struct for example? |
Hi,
I understand that this is likely going to be rejected, but I'll try anyway :) Something that is really needed to make the nameprefix mechanism work in a OpenShift context, is replacing the name of the target service in Route objects (apiVersion: route.openshift.io/v1). Would you maybe consider doing the nameref subtitution for that too?
A typical route definition looks as follows:
See: https://docs.openshift.org/latest/dev_guide/routes.html
The text was updated successfully, but these errors were encountered: