elastic / elastic/integrations

[Microsoft SQLServer] Add support for List of Dynamic counters

Open
#14,083 2 comments 0 reactions 1 assignee Claimed by @muthu-mps View on GitHub
Integration:microsoft_sqlserver Team:Obs-InfraObs
Dominant language
Handlebars
Stars
333
Forks
647
Avg merge
3d 4h
Merged PRs (30d)
209

Description

### Summary

The integration currently supports dynamic counter value using LIKE operator which provides capability to add more counters based on the input provided.

Based on the recent research we try to use the Regex operator to add multiple counter values as input.

### Challenges in using Regex:

- As a user knowing the exact metrics, Do we need to build a regex pattern?
- Using regex doesn't offer the exact match for a metric and pulls additional metrics.

### Solution

Is it possible to add list of counter values as an input?
- Yes its possible to pass list of values as input and generate query out of it.
- I have made the counter value as array and that which generates me the query to match the given input.

### Integration Upgrade

Will changing the input parameter from text to an array ensure seamless integration upgrade experience for the end user?
- Yes / No
- Yes, If the dynamic counter is blank. This may not be the case as we have default value for this config.
- No, The integration fails to upgrade and reports issue with the field type when the counter value exists.

#### How to fix the upgrade failure?

- This can be achieved by removing the existing input value from the config and save the integration then upgrade the integration by providing the dynamic counter values.
- We can check whether its possible to upgrade integration without modifying the input value.

#### Short video on integration upgrade scenario,

https://github.com/user-attachments/assets/e2a3f791-ddcd-4ec1-ada8-14d3f32d815e

cc: @lalit-satapathy

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.