Skip to content

[kibana] Mention elasticsearch data dependency in readme - #5215

Merged
jennypavlova merged 4 commits into
elastic:mainfrom
jennypavlova:3974-kibana-mention-elasticsearch-data-dependency-in-readme
Feb 9, 2023
Merged

[kibana] Mention elasticsearch data dependency in readme #5215
jennypavlova merged 4 commits into
elastic:mainfrom
jennypavlova:3974-kibana-mention-elasticsearch-data-dependency-in-readme

Conversation

@jennypavlova

Copy link
Copy Markdown
Member

What does this PR do?

This PR adds an explanation to the readme file that the elasticsearch data should be monitored in order to see Kibana data in Stack monitoring UI

Related issues

Screenshots

image

@jennypavlova jennypavlova added the enhancement New feature or request label Feb 8, 2023
@jennypavlova jennypavlova self-assigned this Feb 8, 2023
@jennypavlova jennypavlova added the Team:Infra Monitoring UI - DEPRECATED Label for the Infrastructure Monitoring UI team. - DEPRECATED - Use Team:obs-ux-infra_services label Feb 8, 2023
@jennypavlova
jennypavlova marked this pull request as ready for review February 8, 2023 15:33
@jennypavlova
jennypavlova requested a review from a team as a code owner February 8, 2023 15:33
@elasticmachine

elasticmachine commented Feb 8, 2023

Copy link
Copy Markdown

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-02-08T16:56:19.054+0000

  • Duration: 28 min 31 sec

Test stats 🧪

Test Results
Failed 0
Passed 28
Skipped 0
Total 28

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

Comment thread packages/kibana/manifest.yml Outdated
name: kibana
title: Kibana
version: 2.2.0-preview1
version: 2.3.0-preview1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just for a better understanding.
Is there a way to go with the versioning here, like why 2.3.0 and not 2.2.1 since its a documentation update for example

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point! I was thinking about it but I wasn't sure if we usually use patch versions here. I changed it to 2.2.1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes same here 😄 I just looked up previous PRs on the same file and saw that they usually increase the patch version when it comes to documentation updates 👍

@elasticmachine

Copy link
Copy Markdown

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (2/2) 💚
Files 100.0% (4/4) 💚
Classes 100.0% (4/4) 💚
Methods 94.872% (37/39) 👍 0.532
Lines 87.234% (82/94) 👎 -5.279
Conditionals 100.0% (0/0) 💚

@mohamedhamed-ahmed mohamedhamed-ahmed left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jennypavlova
jennypavlova merged commit f0f67e5 into elastic:main Feb 9, 2023
@jennypavlova
jennypavlova deleted the 3974-kibana-mention-elasticsearch-data-dependency-in-readme branch February 9, 2023 10:38
orestisfl pushed a commit to orestisfl/integrations that referenced this pull request May 15, 2026
* Add elastic data dependency in kibana README.md

* Change version to 2.3.0-preview1

* Change version to 2.2.1-preview1

* Update build docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Team:Infra Monitoring UI - DEPRECATED Label for the Infrastructure Monitoring UI team. - DEPRECATED - Use Team:obs-ux-infra_services

3 participants