Commit 3c1da8e
authored
OTA-740: ci-operator/step-registry/ipi/conf/telemetry: Disable Telemetry (#32153)
* ci-operator/step-registry/ipi/conf/telemetry: Disable Telemetry
The OCM team is concerned about the volume of data submitted by
ephemeral CI clusters. This commit disables Telemetry by default (for
the bulk of CI, which flows through this step), while still allowing
Telemetry to be enabled for jobs that have important data to report,
or jobs that want to excersise the Telemetry or Insights reporting
logic.
From [1]:
> You can modify your existing global cluster pull secret to disable
> remote health reporting. This disables both Telemetry and the
> Insights Operator.
In 4.12, the Insights operator is growing a new configuration
structure that allows disabling Insights [1,2] without affecting
Telemetry. And it turns out that the monitoring operator has a
similar option to disable Telemetry [3], although the only documented
use for telemeterClient is for setting nodeSelector [4].
There are a number of existing steps poking at the
cluster-monitoring-config ConfigMap, including
ipi-conf-inframachineset setting telemeterClient's nodeSelector, so I
am following ipi-conf-user-workload-monitoring's use of yq patching
and manifest naming to fit in with the other steps.
I'm following 8a4696c (using yq installed from upi-installer image,
2022-08-25, #31256) to get yq from the upi-installer image instead of
curling it down dynamically. With the 4.12 pin to get `yq`, we're
safe using `upi-installer`, even for CI jobs that are otherwise
looking at different 4.y.
[1]: https://github.com/openshift/enhancements/blob/ef85659d01738b9f89958d5f0da31cff05bb1182/enhancements/insights/insights-config-api.md
[2]: https://docs.openshift.com/container-platform/4.11/support/remote_health_monitoring/opting-out-of-remote-health-reporting.html
[3]: https://github.com/openshift/cluster-monitoring-operator/blob/8d331d78b22948d36c20da0552763ddd8a4e2093/pkg/manifests/config.go#L337
[4]: https://docs.openshift.com/container-platform/4.11/monitoring/configuring-the-monitoring-stack.html#moving-monitoring-components-to-different-nodes_configuring-the-monitoring-stack
* ci-operator/step-registry: Inject ipi-conf-telemetry after ipi-conf
Generated with:
$ sed -i 's/^\( *- ref: ipi-conf\)$/\1\n\1-telemetry/' $(git grep -l '^ *- ref: ipi-conf$' ci-operator/step-registry)
to slot the new step in after the generic ipi-conf, so we can
configure Telemetry in all of the existing chains and workflows that
were flowing through ipi-conf.
* ci-operator/config/openshift/telemeter: TELEMETER_ENABLED=true
When making changes to the Telemeter client, we want to ensure that
we're still excercising uploads.1 parent 990374d commit 3c1da8e
73 files changed
Lines changed: 146 additions & 0 deletions
File tree
- ci-operator
- config/openshift/telemeter
- step-registry
- cucushift/installer/rehearse
- aws
- ipi
- byo-iam-role/provision
- byo-route53/provision
- disconnected/private
- provision
- sdn/provision
- sno/provision
- techpreview/provision
- private
- cco-manual-security-token-service/provision
- ovn/ipsec/provision
- provision
- proxy
- cco-manual-security-token-service/provision
- provision
- sdn/provision
- techpreview/provision
- usgov/ipi/private/workers-marketplace/provision
- azure
- ipi
- disconnected
- fullyprivate
- provision
- techpreview/provision
- provision
- disk-encryption-set/provision
- fullyprivate/provision
- proxy
- provision
- techpreview/provision
- vmgenv1/provision
- workers-marketimage/provision
- workers-rhel8/provision
- upi
- disconnected/provision
- proxy/provision
- gcp
- ipi
- disconnected
- private
- ovn/provision
- provision
- provision
- private/provision
- proxy
- provision
- techpreview/provision
- workers-rhel8/provision
- upi
- disconnected/provision
- proxy/provision
- vsphere
- ipi/baselinecaps/none/additionalenabledcaps/provision
- upi
- disconnected
- provision
- sdn/provision
- ovn
- sdn2ovn-rollback/provision
- sdn2ovn/provision
- provision
- proxy/provision
- ipi
- azure/arcconformance
- conf
- alibabacloud
- aws
- blackholenetwork
- azuremag
- azurestack
- azure
- resourcegroup
- gcp
- ibmcloud
- nutanix
- powervs
- telemetry
- vsphere
- zones
- openshift/e2e
- aws
- capi
- ccm
- ovn-hybrid
- gcp
- capi
- ccm
- ovn
- upi/conf/vsphere
- platform-none/ovn
- proxy/https
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
| 58 | + | |
| 59 | + | |
58 | 60 | | |
59 | 61 | | |
60 | 62 | | |
61 | 63 | | |
| 64 | + | |
| 65 | + | |
62 | 66 | | |
63 | 67 | | |
64 | 68 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
| 6 | + | |
6 | 7 | | |
7 | 8 | | |
8 | 9 | | |
| |||
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
7 | 8 | | |
8 | 9 | | |
9 | 10 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| |||
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| |||
0 commit comments