[chore][receiver/sqlserver] provide one generic configuration in readme #40851
Conversation
github-actions
bot
added
receiver/sqlserver
Run Windows
Co-authored-by: Curtis Robert <crobert@splunk.com>
crobert-1
added
the
ready to merge
|
|
| server: sqlserver.address | ||
| port: 1433 | ||
| events: | ||
| db.server.query_sample: |
There was a problem hiding this comment.
Currently, to enable or disable a collection, we have to configure it under events rather than within the collection-related section. Personally, I find this a bit unintuitive.
Do you think it would be feasible to modify this on mdatagen side, so that enable/disable configuration could live in the same section as the rest of the collection settings?
@sincejune
There was a problem hiding this comment.
The solution I came up with is to add a parameters section to the metadata file, structured as follows:
parameters:
<param>:
type: int
default: <default_value>
events:
default_event:
attributes: [...]
parameters:
<param>: <value>
This approach works well for parameters that will be used within a single collection. However, it may not be ideal for parameters that need to be shared across multiple collections (e.g., metrics or events).
@dmitryax what do you think?
crobert-1
removed
the
ready to merge
Description
we now have many different configurations but the current readme does not have one generic readme that user can use directly.
Link to tracking issue
Fixes
Testing
Documentation
/cc @natalie-arsky