-
Notifications
You must be signed in to change notification settings - Fork 62
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
Fix: using deprecated field AddOnConfiguiration
causing proxy health
not work.
#348
Fix: using deprecated field AddOnConfiguiration
causing proxy health
not work.
#348
Conversation
4483ff0
to
3a126b8
Compare
pkg/cmd/proxy/health/exec.go
Outdated
proxyConfig, err := proxyClient.ProxyV1alpha1().ManagedProxyConfigurations(). | ||
Get(context.TODO(), clusterAddon.Spec.AddOnConfiguration.CRName, metav1.GetOptions{}) | ||
Get(context.TODO(), "cluster-proxy", metav1.GetOptions{}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name of managedproxyconfiguration is always cluster-proxy
if we deploy addon via clusteradm: https://github.com/open-cluster-management-io/cluster-proxy/blob/2a197b382114b674b76a362146a7259aed164424/charts/cluster-proxy/templates/managedproxyconfiguration.yaml#L4
/assign @qiujian16 |
0dc91b7
to
02188d8
Compare
pkg/cmd/proxy/health/exec.go
Outdated
proxyConfig, err := proxyClient.ProxyV1alpha1().ManagedProxyConfigurations(). | ||
Get(context.TODO(), clusterAddon.Spec.AddOnConfiguration.CRName, metav1.GetOptions{}) | ||
Get(context.TODO(), "cluster-proxy", metav1.GetOptions{}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set it as a const
02188d8
to
56ead55
Compare
…th` not work. Signed-off-by: xuezhaojun <zxue@redhat.com>
56ead55
to
5a69366
Compare
/assign @qiujian16 Rebase, PTAL, thanks! |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: qiujian16, xuezhaojun 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 |
b254f69
into
open-cluster-management-io:main
Fixes: #347