-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[docs] Add status header for humio exporter #10319
[docs] Add status header for humio exporter #10319
Conversation
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.
/cc @Xitric as code owner to verify beta stability
exporter/humioexporter/README.md
Outdated
| Status | | | ||
| ------------------------ |-----------| | ||
| Stability | [beta] | | ||
| Supported pipeline types | trace | |
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.
| Supported pipeline types | trace | | |
| Supported pipeline types | traces | |
Yes, I can verify that this exporter only has beta stability. To my knowledge, it has only been used by a small handful of people, including myself. |
|
Am I mistaken, or do those links work perfectly fine? There seems to be a discussion on the markdown-link-check repository and some open issues stating that a regression on fragment anchors has been introduced recently. See tcort/markdown-link-check#91. |
The Core Repo (and our workflow as well) stopped using the action and started using the npm package directly so we should have the latest version. I think the issue you provided is more relevant. |
Core is able to get anchors to work tho: https://github.com/open-telemetry/opentelemetry-collector/runs/6492891036?check_suite_focus=true. |
exporter/humioexporter/README.md
Outdated
| Supported pipeline types | traces | | ||
| Distributions | [contrib] | | ||
|
||
Exports data to Humio using JSON over the HTTP [Ingest API](https://docs.humio.com/reference/api/ingest/). | ||
|
||
> :construction: This exporter is currently intended for evaluation purposes only! It has yet to be enabled in the build. |
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.
Should this be removed? The exporter is in the contrib distribution
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.
Yes, we forgot to remove that comment way back when the exporter was enabled. My former colleague and I have evaluated it internally (we are not affiliated with Humio), so we do consider it sufficiently stable for beta.
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.
ok ,I will remove the comment.
@Frapschen @Xitric the anchor links need to be lowercase: https://github.com/TylerHelmuth/opentelemetry-collector-contrib/runs/6612328113?check_suite_focus=true |
@TylerHelmuth Alright, that is good to know. I learned something new today. Is this something you or @Frapschen can handle? I am currently on vacation and only on my phone. |
@Frapschen as the PR creator please update the anchor links |
done:) |
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.
Thank you, @Frapschen
* Add status header for humio * update for reviewer's suggestion * apply reviewer's suggestion
Description:
Add status header for humio exporter.