Skip to content

Conversation

pleimer
Copy link
Contributor

@pleimer pleimer commented Oct 22, 2020

Add a section explaining how to enable the gathering of service container metrics in the overcloud.

@pleimer
Copy link
Contributor Author

pleimer commented Oct 22, 2020

I don't think my link to stf-connectors.yaml is working correctly at the moment

@leifmadsen
Copy link
Member

You also need to include your module in doc-Service-Telemetry-Framework/assemblies/assembly_advanced-features.adoc

@leifmadsen
Copy link
Member

diff --git a/doc-Service-Telemetry-Framework/assemblies/assembly_advanced-features.adoc b/doc-Service-Telemetry-Framework/assemblies/assembly_advanced-features.adoc
index 65c1486..6cd450c 100644
--- a/doc-Service-Telemetry-Framework/assemblies/assembly_advanced-features.adoc
+++ b/doc-Service-Telemetry-Framework/assemblies/assembly_advanced-features.adoc
@@ -69,3 +69,6 @@ include::../modules/proc_querying-metrics-data-from-multiple-clouds.adoc[levelof
 //Configuring ephemeral storage
 include::../modules/con_ephemeral-storage.adoc[leveloffset=+1]
 include::../modules/proc_configuring-ephemeral-storage.adoc[leveloffset=+2]
+
+//Service resource usage metrics
+include::../modules/con_service-resource-usage-metrics.adoc[leveloffset=+1]

@JoanneOFlynn2018 JoanneOFlynn2018 dismissed leifmadsen’s stale review October 28, 2020 12:24

changes implemented, enabling merge

@leifmadsen
Copy link
Member

I'm going to get this merged down and imported in to the STF 1.1 documentation updates. If for some reason this misses 16.1.3 we'll need to comment it out downstream, but that's not a huge deal for upstream master docs.

@leifmadsen
Copy link
Member

Did a local build and check, and this all loads fine. Merging!

@leifmadsen leifmadsen self-requested a review October 28, 2020 13:46
@leifmadsen leifmadsen merged commit 8247157 into master Oct 28, 2020
@leifmadsen leifmadsen deleted the pleimer-libpodstats-enable branch October 28, 2020 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants