Bug 1845014

Summary: Grafana dashboard shows the listener IP addresses of controller nodes in the "Nodes" list.
Product: Service Telemetry Framework Reporter: Leonid Natapov <lnatapov>
Component: DocumentationAssignee: Paul Leimer <pleimer>
Status: CLOSED CURRENTRELEASE QA Contact: Leonid Natapov <lnatapov>
Severity: high Docs Contact: Joanne O'Flynn <joflynn>
Priority: medium    
Version: 1.2CC: augol, csibbitt, lmadsen, mburns, mmagr, mrunge, oblaut, pleimer
Target Milestone: z2Keywords: TestOnly, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2021-06-24 14:48:19 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1912895, 1943801    
Bug Blocks:    

Description Leonid Natapov 2020-06-08 09:44:59 UTC
In addition to the overlcoud nodes names,Grafana dashboard also shows listener IP addresses of controller nodes in the "Nodes" list. In the "Nodes" list shoule appear only overcloud nodes names.

Comment 3 Paul Leimer 2020-08-05 19:34:16 UTC
https://review.opendev.org/#/c/742685/

Comment 9 Leonid Natapov 2021-01-05 14:24:55 UTC
memcached is still reporting it by IP address. Moving back to Assign

Comment 10 Paul Leimer 2021-04-28 17:32:49 UTC
This can be fixed by adding the following to the stf-connectors.yml:

```
    ExtraConfig:
      collectd::plugin::memcached::instances:
        local:
          host: "%{hiera('fqdn_canonical')}
          port: 11211
```

Comment 12 Paul Leimer 2021-05-05 12:52:22 UTC
Hi Joanne,

Originally I figured this would go in the dashboard section, but I think it would be better to be upfront about the change. The section you pointed out would be better IMO.

We can add a bullet point that goes as follows:

- In the ExtraConfig, add the following to tell the collectd memcached to report the hostname by fqdn rather than ip address in accordance with the rest of the collectd plugins:

>       collectd::plugin::memcached::instances:
>         local:
>           host: "%{hiera('fqdn_canonical')}
>           port: 11211

And then below, where the whole stf-connectors.yaml is laid out, also include the changes:

> parameter_defaults: 
>     EventPipelinePublishers: []
>     PipelinePublishers: []
>     CeilometerQdrPublishEvents: true
>     CeilometerQdrPublishMetrics: true
>     MetricsQdrConnectors:
>     - host: default-interconnect-5671-service-telemetry.apps.infra.watch
>       port: 443
>       role: edge
>       sslProfile: sslProfile
>       verifyHostname: false
>     ExtraConfig:
>       ceilometer::agent::polling::polling_interval: 5
>       collectd::plugin::memcached::instances:
>         local:
>           host: "%{hiera('fqdn_canonical')}
>           port: 11211

Comment 15 Paul Leimer 2021-06-10 19:11:05 UTC
This issue will be solved by updates to our enable-stf.yaml and will no longer need a documentation update: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/795608/4/environments/enable-stf.yaml

Comment 16 Leif Madsen 2021-06-10 19:20:24 UTC
(In reply to Paul Leimer from comment #15)
> This issue will be solved by updates to our enable-stf.yaml and will no
> longer need a documentation update:
> https://review.opendev.org/c/openstack/tripleo-heat-templates/+/795608/4/
> environments/enable-stf.yaml

Actually I think we'll need to include this in the docs in the short term. I think the way this will go will be:

1) change as you've described here

2) migrate to documenting the enable-stf.yaml and changing the installation procedure slightly, with a copy/pasta of the new enable-stf.yaml

3) once changes are available in the downstream product, refer to the in-tree enable-stf.yaml and remove the documented copy/pasta method