Skip to content

packages/prometheus/docs: Replace darasteam with datastream #7914

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
May 8, 2024
Merged

packages/prometheus/docs: Replace darasteam with datastream #7914

merged 17 commits into from
May 8, 2024

Conversation

limistah
Copy link
Contributor

What does this PR do?

The page https://docs.elastic.co/integrations/prometheus#dashboard states:

Prometheus integration is shipped including default overview dashboard. Default dashboard works only for remote_write datastream and collector darastream, if metrics are scraped from the Prometheus server metrics endpoint.

The darastream should be datastream and read like

Prometheus integration is shipped including default overview dashboard. Default dashboard works only for remote_write datastream and collector datastream, if metrics are scraped from the Prometheus server metrics endpoint.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Screenshots

image

@elasticmachine
Copy link

elasticmachine commented Sep 21, 2023

💔 Build Failed

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-11-18T05:32:25.354+0000

  • Duration: 12 min 37 sec

Steps errors 4

Expand to view the steps failures

Check integration: prometheus
  • Took 0 min 1 sec . View more details here
  • Description: ../../build/elastic-package check -v
Take down the Elastic stack
  • Took 0 min 0 sec . View more details here
  • Description: build/elastic-package stack down -v
Google Storage Download
  • Took 0 min 0 sec . View more details here
Google Storage Download
  • Took 0 min 0 sec . View more details here

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

@limistah
Copy link
Contributor Author

/test

@limistah
Copy link
Contributor Author

limistah commented Oct 5, 2023

The CI keeps failing, this is a typo fix. I'm not sure if I am doing anything wrong to cause the CI to fail. Can someone help?

@botelastic
Copy link

botelastic bot commented Nov 17, 2023

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Nov 17, 2023
@limistah
Copy link
Contributor Author

👍🏾

@botelastic botelastic bot removed the Stalled label Nov 18, 2023
@limistah
Copy link
Contributor Author

Hi @tetianakravchenko

Your review is pending on this PR

Thanks

@botelastic
Copy link

botelastic bot commented Dec 18, 2023

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Dec 18, 2023
@limistah
Copy link
Contributor Author

👍🏾

@botelastic botelastic bot removed the Stalled label Dec 18, 2023
@mrodm
Copy link
Collaborator

mrodm commented Dec 21, 2023

Hi @limistah, please update your branch with the latest contents from main branch. There was an important PR merged updating the CI pipelines. Thanks!

@ali786XI ali786XI self-requested a review March 7, 2024 12:47
@@ -1,4 +1,9 @@
# newer versions go on top
- version: "1.14.0"
Copy link
Contributor

@ali786XI ali786XI Mar 7, 2024

Choose a reason for hiding this comment

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

Suggested change
- version: "1.14.0"
- version: "1.13.2"

As this is a minor fix in documentation bumping the patch version would suffice.

@botelastic
Copy link

botelastic bot commented Apr 6, 2024

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added Stalled and removed Stalled labels Apr 6, 2024
@ishleenk17
Copy link
Member

@limistah : Please fix the comments. We can then retrigger the build and commit the change

@botelastic
Copy link

botelastic bot commented May 8, 2024

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label May 8, 2024
@ishleenk17
Copy link
Member

@limistah : Lets take this PR forward!

@botelastic botelastic bot removed the Stalled label May 8, 2024
limistah and others added 2 commits May 8, 2024 10:27
Co-authored-by: Aliabbas Attarwala <124054599+aliabbas-elastic@users.noreply.github.com>
@limistah limistah requested a review from ali786XI May 8, 2024 09:29
@ali786XI
Copy link
Contributor

ali786XI commented May 8, 2024

@limistah Please also bump up the manifest.yml version as well

Signed-off-by: Aleem Isiaka <aleemisiaka@gmail.com>
@limistah
Copy link
Contributor Author

limistah commented May 8, 2024

@limistah Please also bump up the manifest.yml version as well

Done with this commit 70c8895

@ali786XI
Copy link
Contributor

ali786XI commented May 8, 2024

/test

Copy link
Contributor

@ali786XI ali786XI left a comment

Choose a reason for hiding this comment

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

LGTM

@limistah limistah requested a review from ali786XI May 8, 2024 15:20
@ali786XI
Copy link
Contributor

ali786XI commented May 8, 2024

/test

Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @limistah

Copy link
Contributor

@ali786XI ali786XI left a comment

Choose a reason for hiding this comment

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

LGTM

@ali786XI ali786XI merged commit 96832aa into elastic:main May 8, 2024
@elasticmachine
Copy link

Package prometheus - 1.15.2 containing this change is available at https://epr.elastic.co/search?package=prometheus

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:prometheus Prometheus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants