Merge pull request #305 from pikasTech/fix/hwlab-1119-postprocess-no-shortcircuit
fix: remove D601 metrics sidecars fully
This commit is contained in:
@@ -1452,7 +1452,10 @@ function nodeRuntimePipelinePostprocessScript(): string[] {
|
|||||||
" for (const item of listItems(doc).filter(Boolean)) {",
|
" for (const item of listItems(doc).filter(Boolean)) {",
|
||||||
" if (!isObject(item)) continue;",
|
" if (!isObject(item)) continue;",
|
||||||
" if (overlay.observability && overlay.observability.prometheusOperator === false) {",
|
" if (overlay.observability && overlay.observability.prometheusOperator === false) {",
|
||||||
" const monitoringChanged = stripMonitoringMetadata(item.metadata) || stripMonitoringMetadata(templateMetadataFor(item)) || removeMetricsSidecar(podSpecFor(item));",
|
" const metadataChanged = stripMonitoringMetadata(item.metadata);",
|
||||||
|
" const templateChanged = stripMonitoringMetadata(templateMetadataFor(item));",
|
||||||
|
" const sidecarChanged = removeMetricsSidecar(podSpecFor(item));",
|
||||||
|
" const monitoringChanged = metadataChanged || templateChanged || sidecarChanged;",
|
||||||
" changed = monitoringChanged || changed;",
|
" changed = monitoringChanged || changed;",
|
||||||
" observabilityChanged = observabilityChanged || monitoringChanged;",
|
" observabilityChanged = observabilityChanged || monitoringChanged;",
|
||||||
" }",
|
" }",
|
||||||
|
|||||||
Reference in New Issue
Block a user