Skip to content
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

Mysterious values referenced in helloword template #270

Open
MikeSpreitzer opened this issue May 18, 2024 · 1 comment
Open

Mysterious values referenced in helloword template #270

MikeSpreitzer opened this issue May 18, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@MikeSpreitzer
Copy link

MikeSpreitzer commented May 18, 2024

https://github.com/open-cluster-management-io/addon-framework/blob/main/examples/helloworld/manifests/templates/deployment.yaml references some values that are not explained in the documentation and, I suspect, may be outdated.

In that template I see the following mysterious value references.

  • .Tolerations
  • .NodeSelector
  • .HTTPProxy
  • .HTTPSProxy
  • .NoProxy

Looking in https://github.com/open-cluster-management-io/addon-framework/blob/main/pkg/addonfactory/addondeploymentconfig.go I found ToAddOnNodePlacementValues and ToAddOnProxyConfigValues. These support the following references from a template.

  • .tolerations
  • .global.nodeSelector
  • .global.proxyConfig.HTTP_PROXY
  • .global.proxyConfig.HTTPS_PROXY
  • .global.proxyConfig.NO_PROXY

They also support .global.proxyConfig.PROXY_CA_BUNDLE but the example template does not use anything like that.

I also found ToAddOnDeloymentConfigValues, which provides values named Tolerations and NodeSelector but nothing for proxy config. This function is not mentioned in https://open-cluster-management.io/developer-guides/addon/ .

@qiujian16
Copy link
Member

cc @zhujian7

@qiujian16 qiujian16 added the documentation Improvements or additions to documentation label May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants