Skip to content
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

Create EXAMPLES.md #63

Merged
merged 9 commits into from
Mar 14, 2021
Merged

Create EXAMPLES.md #63

merged 9 commits into from
Mar 14, 2021

Conversation

bluecmd
Copy link
Owner

@bluecmd bluecmd commented Mar 14, 2021

Add examples how to use the Fortigate Exporter

Fixes #39 but of course more examples will be added over time.

Add examples how to use the Fortigate Exporter
@bluecmd bluecmd requested a review from secustor March 14, 2021 13:22
Copy link
Collaborator

@secustor secustor left a comment

Choose a reason for hiding this comment

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

What do you think about moving this in a docs folder?

EXAMPLES.md Show resolved Hide resolved
@bluecmd
Copy link
Owner Author

bluecmd commented Mar 14, 2021

What do you think about moving this in a docs folder?

Hm, will we have many more?
I am thinking README.md, CONTRIBUTING.md, and EXAMPLES.md.
The first two will need to be in the root because of well, that's where people expect them to be.

I'd say we do not create a docs/ for now but I am 100% with you to move it if we end up having a website or something.

@bluecmd
Copy link
Owner Author

bluecmd commented Mar 14, 2021

Ready for review. Had a friend (thanks @ventris) look over it quite quickly as well to see that it reads OK.

EXAMPLES.md Outdated
# PromQL usage examples for Fortigate Exporter

PromQL is an advanced language and proper usage of it requires one to both understand
the language itself and how the metrics one has are constructed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

English is my second language so could be me, but that sounds kinda weird.

WDYT about PromQL is an advanced language and proper usage of it requires first to understand the language itself and how the metrics one are constructed.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Yeah, it is a hard sentence to grok. What do you think of this new one?

Copy link
Collaborator

Choose a reason for hiding this comment

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

What are trying to bring across with organized? I'm not sure what you mean here

Copy link
Owner Author

Choose a reason for hiding this comment

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

I want to state that even if you are the world's best champion on PromQL, you still need to learn how each exporter works; as in what metrics are available to you, and how their labels associate between the metrics.

E.g. that we have a fortigate_version_info that can be used to get the build info (as opposed to adding the build label to all metrics, as another exporter might have had done).

Copy link
Collaborator

Choose a reason for hiding this comment

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

I see. :)

EXAMPLES.md Outdated Show resolved Hide resolved
bluecmd and others added 2 commits March 14, 2021 23:58
Co-authored-by: Sebastian Poxhofer <secustor@users.noreply.github.com>
@secustor secustor merged commit 2b7dcda into master Mar 14, 2021
@secustor secustor deleted the bluecmd/document-promql branch March 14, 2021 23:13
@bluecmd
Copy link
Owner Author

bluecmd commented Mar 17, 2021

🎉 This PR is included in version 1.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add example PromQl queries to README.md
2 participants