airshipit / treasuremap

Reference Airship manifests, CICD, and reference architecture.
http://openstack.org
Apache License 2.0
52 stars 39 forks source link

Deploy Kibana Dashboards in airship-core/multi-tenant types #150

Closed lb4368 closed 3 years ago

lb4368 commented 3 years ago

Proposed change Provide a demonstration of a Kibana dashboard that includes logs from elasticsearch as part of an airship-core and multi-tenant site deployment.

MVP requirements:

  1. Functioning lma-stack composite
    • Create LMA-stack composite in airshipit/treasuremap · Issue #61 · airshipit/treasuremap (github.com)
  2. Functioning elastic-stack composite
    • Create airshipit/treasuremap Function for Elasticsearch · Issue #52 · airshipit/treasuremap (github.com)
    • Includes basic pod / system logs
    • All composite overrides
  3. Elasticsearch function
    • Create airshipit/treasuremap Function for Elasticsearch · Issue #52 · airshipit/treasuremap (github.com)
  4. Kibana Function
    • Create airshipit/treasuremap Function for Kibana · Issue #53 · airshipit/treasuremap (github.com)
    • Dashboard is accessible via nodeport
    • Dashboard showcases data from Elasticsearch
StevTheDev commented 3 years ago

Hey @lb4368, please assign this one to me!

lb4368 commented 3 years ago

Done. Thanks @StevTheDev !

StevTheDev commented 3 years ago

These PS have been merged, which brings us closer to completing this epic.

Treasuremap Elasticsearch & Kibana Add elasticsearch-exporter to our elastic-stack

I have a few PS still outstanding which will help close this out:

StevTheDev commented 3 years ago

Those last two patchsets were getting too big to be easily reviewed, so I broke them down into smaller pieces:

These are all pending review now. T-Shirt size for this item could still be considered M, because most of the development is complete, or bumped up to L due to the scope involved.

StevTheDev commented 3 years ago

Most of the patchsets associated with this feature have merged, but some things are not quite right in the implementation of the replacement transformers:

eak13 commented 3 years ago

@StevTheDev looks like those 2 patchsets merged (799090 & 799749). Can we close this issue now? Thanks.

eak13 commented 3 years ago

Closing per code merges.