Skip to content

Commit 8aea5c2

Browse files
committed
USHIFT-6951: move otelcol metrics drop-in to metrics-server package
metrics-server installs its own otelcol drop-in config and creates the otelcol.d directory, removing the hard coupling to observability Signed-off-by: Jonathan H. Cope <jcope@redhat.com>
1 parent 47da5ea commit 8aea5c2

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

packaging/rpm/microshift.spec

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,6 @@ and can be used to embed those images into osbuilder blueprints or bootc contain
236236
Summary: OpenTelemetry-Collector configured for MicroShift
237237
BuildArch: noarch
238238
Requires: microshift = %{version}
239-
Requires: microshift-metrics-server = %{version}
240239
Requires: opentelemetry-collector
241240

242241
%description observability
@@ -584,7 +583,6 @@ install -p -m644 assets/optional/ai-model-serving/release-ai-model-serving-x86_6
584583
install -d -m755 %{buildroot}/%{_sysconfdir}/microshift/observability
585584
install -d -m755 %{buildroot}/%{_sysconfdir}/microshift/observability/otelcol.d
586585
install -p -m644 packaging/observability/*.yaml -D %{buildroot}%{_sysconfdir}/microshift/observability/
587-
install -p -m644 packaging/observability/otelcol.d/microshift-metrics-server.yaml %{buildroot}%{_sysconfdir}/microshift/observability/otelcol.d/
588586
# Explicit copy of large config as default. Not using symlink to avoid accidental package upgrade overwriting user config if the user edits the config without copying (i.e. edits the target of symlink).
589587
install -p -m644 packaging/observability/opentelemetry-collector-large.yaml -D %{buildroot}%{_sysconfdir}/microshift/observability/opentelemetry-collector.yaml
590588
install -p -m644 packaging/observability/microshift-observability.service %{buildroot}%{_unitdir}/
@@ -642,6 +640,10 @@ cat assets/optional/metrics-server/kustomization.aarch64.yaml >> %{buildroot}/%{
642640
cat assets/optional/metrics-server/kustomization.x86_64.yaml >> %{buildroot}/%{_prefix}/lib/microshift/manifests.d/080-microshift-metrics-server/kustomization.yaml
643641
%endif
644642

643+
# metrics-server otelcol drop-in (creates the config path if microshift-observability is not installed)
644+
install -d -m755 %{buildroot}/%{_sysconfdir}/microshift/observability/otelcol.d
645+
install -p -m644 packaging/observability/otelcol.d/microshift-metrics-server.yaml %{buildroot}%{_sysconfdir}/microshift/observability/otelcol.d/
646+
645647
# metrics-server-release-info
646648
mkdir -p -m755 %{buildroot}%{_datadir}/microshift/release
647649
install -p -m644 assets/optional/metrics-server/release-metrics-server-{x86_64,aarch64}.json %{buildroot}%{_datadir}/microshift/release/
@@ -842,7 +844,6 @@ fi
842844
%config(noreplace) %{_sysconfdir}/microshift/observability/opentelemetry-collector.yaml
843845
%{_sysconfdir}/microshift/observability/opentelemetry-collector-*.yaml
844846
%{_prefix}/lib/microshift/manifests.d/003-microshift-observability/*
845-
%config(noreplace) %{_sysconfdir}/microshift/observability/otelcol.d/microshift-metrics-server.yaml
846847

847848
%files cert-manager
848849
%dir %{_prefix}/lib/microshift/manifests.d/060-microshift-cert-manager
@@ -854,6 +855,8 @@ fi
854855
%files metrics-server
855856
%dir %{_prefix}/lib/microshift/manifests.d/080-microshift-metrics-server
856857
%{_prefix}/lib/microshift/manifests.d/080-microshift-metrics-server/*
858+
%dir %{_sysconfdir}/microshift/observability/otelcol.d
859+
%config(noreplace) %{_sysconfdir}/microshift/observability/otelcol.d/microshift-metrics-server.yaml
857860

858861
%files metrics-server-release-info
859862
%{_datadir}/microshift/release/release-metrics-server-{x86_64,aarch64}.json

0 commit comments

Comments
 (0)