-
Notifications
You must be signed in to change notification settings - Fork 474
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
packages/prometheus/docs: Replace darasteam
with datastream
#7914
Conversation
💔 Build Failed
Expand to view the summary
Build stats
Steps errors
Expand to view the steps failures
|
|
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? |
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
👍🏾 |
Your review is pending on this PR Thanks |
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
👍🏾 |
Hi @limistah, please update your branch with the latest contents from main branch. There was an important PR merged updating the CI pipelines. Thanks! |
packages/prometheus/changelog.yml
Outdated
@@ -1,4 +1,9 @@ | |||
# newer versions go on top | |||
- version: "1.14.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- version: "1.14.0" | |
- version: "1.13.2" |
As this is a minor fix in documentation bumping the patch version would suffice.
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
@limistah : Please fix the comments. We can then retrigger the build and commit the change |
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
@limistah : Lets take this PR forward! |
Co-authored-by: Aliabbas Attarwala <124054599+aliabbas-elastic@users.noreply.github.com>
@limistah Please also bump up the |
Signed-off-by: Aleem Isiaka <aleemisiaka@gmail.com>
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/test |
|
💚 Build Succeeded
History
cc @limistah |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Package prometheus - 1.15.2 containing this change is available at https://epr.elastic.co/search?package=prometheus |
What does this PR do?
The page https://docs.elastic.co/integrations/prometheus#dashboard states:
The
darastream
should bedatastream
and read likeChecklist
changelog.yml
file.Screenshots