-
Notifications
You must be signed in to change notification settings - Fork 158
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
Added etcd2 config source #317
Conversation
ef27f73
to
15004ab
Compare
Codecov Report
@@ Coverage Diff @@
## main #317 +/- ##
==========================================
+ Coverage 79.10% 79.51% +0.40%
==========================================
Files 30 33 +3
Lines 2240 2328 +88
==========================================
+ Hits 1772 1851 +79
- Misses 423 431 +8
- Partials 45 46 +1
Continue to review full report at Codecov.
|
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: just one Q and two "empty" files :)
A good thing to add will be a README.md describing the config options and a few examples, nothing fancy: configuration information and examples, see https://github.com/signalfx/splunk-otel-collector/blob/main/internal/configsource/vaultconfigsource/README.md |
d67572e
to
d6a7968
Compare
d6a7968
to
acdbc54
Compare
endpoints: [http://localhost:2379] | ||
# auth is a optional section used to indicate the authentication method to be used. | ||
# currently only username and password is supported. | ||
auth: |
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.
Nice!
No description provided.