[CHORE] adding agents.md#3699
Conversation
Signed-off-by: Nicolas Takashi <nicolas.tcs@hotmail.com>
|
I would love to have this, but I think we should think about a system to sync these between all the exporters. I don't want to hand-maintain 70+ of these across the community. |
Thanks for raising this @SuperQ How it would workEach exporter repo gets two files:
@AGENTS_EXPORTERS.md
## Project-specific guidelines
... exporter-specific content ...
Why this approach
What would need to change
The content split would be something like:
WDYT? Happy to send a PR to the sync script if this direction makes sense. |
|
Yeah, I think we should have most of these guidelines on the website, and in |
One of the things I've been testing and I can't make sure yet is, we don't have any guarantee that agents will follow external links, so pointing into the AGENTS.md will be a best effort |
|
I see a mix of things that apply only to node_exporter, apply to a few exporters, and apply to all exporters. The guidance I read about AGENTS.md is that it's best to keep it light and hyper focused on unique aspects of the project. Linking out to docs (that the agent is hopefully trained on 🤞) or providing guidance the agent can glean easily from looking at surrounding code might end up bloating token use for minimal gain. I would be happy to start with just node_exporter as we figure out what's valuable and when we get another repo or two look at doing a multi-file approach with syncing for the clear overlapping guidance. This can be captured in issues on an exporter board (working out the details on that ATM). |
|
Could we start with something that says metric names should follow Prometheus naming conventions, and things that are very specific to node exporter? I'm not really familiar with this codebase, so I can't really tell what kind of instructions only apply here 😅. |
|
Maybe the test fixtures, if it's not already documented somewhere else? I don't see this in other exporters 🤔. |
|
ah, just remembered that we could document something about the procfs dependency being responsible for parsing proc files, instead of adding this logic in the exporter |
No description provided.