weave-gitops
weave-gitops copied to clipboard
Add analytics to helm chart on dashboard installation
Closes #2771
When creating the Helm Chart for the dashboard, we will now check the user's config file and set analytics accordingly
:no_entry: This version of this doesn't work because it's passing nil as the logger. However when rebased on top of #2868 so the logger isn't needed, it seems to work :+1:
@ozamosi I've
- added the global
sessionConfigvar to be able to set it for unit testing. - fixed the helm release and values unit tests.
- fixed text messages and prompt (which was worse because it just hang the terminal) being forwarded to generated dashboard manifests if the
create dashboardwith--exportflag on was the first command (before the analytics config was created) when running the GitOps CLI.
Tested it, seems to work as expected.
This should be merged before the enterprise PRs.