You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.
Simplify the definition of the application host name for a chart deployer.
What is the current behavior?
The chart deployer must specify the application host using either global.host or global.clusterRouterBase.
The latter parameter is basically requesting the user to specify a cluster configuration that we can also extract from the cluster itself with a lookup function.
What is the new behavior?
In the janus-idp.hostname template function, extend the exit condition with a further attempt to lookup the cluster domain from the configuration, as in: oc get ingress.config.openshift.io/cluster -oyaml | yq '.spec.domain'
If the configuration is found, then we can use it as a last resort option.
The text was updated successfully, but these errors were encountered:
@masayag I tested on openshift local and verified that the relevant hostname information can be extracted from the cluster with a lookup function. I'll add the PR for it right now.
What do you want to improve?
Simplify the definition of the application host name for a chart deployer.
What is the current behavior?
The chart deployer must specify the application host using either
global.host
orglobal.clusterRouterBase
.The latter parameter is basically requesting the user to specify a cluster configuration that we can also extract from the cluster itself with a lookup function.
What is the new behavior?
In the
janus-idp.hostname
template function, extend the exit condition with a further attempt to lookup the cluster domain from the configuration, as in:oc get ingress.config.openshift.io/cluster -oyaml | yq '.spec.domain'
If the configuration is found, then we can use it as a last resort option.
The text was updated successfully, but these errors were encountered: