Skip to content

feat(nrdot-mssql): Added the automation cli for mssql otel - #1403

Open
RamanaReddy8801 wants to merge 32 commits into
mainfrom
mssql-otel-cli
Open

feat(nrdot-mssql): Added the automation cli for mssql otel#1403
RamanaReddy8801 wants to merge 32 commits into
mainfrom
mssql-otel-cli

Conversation

@RamanaReddy8801

@RamanaReddy8801 RamanaReddy8801 commented Jul 20, 2026

Copy link
Copy Markdown

Jira Ticket: https://new-relic.atlassian.net/browse/NR-595317
Description: Implementation of the New Relic CLI installer for MSSQL OpenTelemetry (OTel) monitoring.

I’ve attached the test [documentation] covering several key integration scenarios for your review.

CI test coverage (Windows)

Added test/definitions/nrdot/mssql-otel/mssql-windows2019.json to test
nrdot-collector-mssql on Windows (SQL-auth mode) — this is the first
NRDOT recipe with Windows CI coverage in the repo.

Two supporting changes were needed to make this possible:

  • New role test/deploy/windows/nrdot-mssql/install/roles/configure:
    the only SQL Server AMI available in CI ships with sa disabled and
    Windows-Auth-only mode. This role switches SQL Server to Mixed Mode,
    restarts it, then enables and sets a known sa password so the
    recipe's SQL-auth path (NR_CLI_MSSQL_AUTH_MODE=1) can be tested.
  • Modified test/deploy/windows/newrelic-cli/install-recipe/roles/onafterstart/tasks/main.yml:
    added a generic env_var passthrough (PowerShell $env: assignments),
    matching the mechanism the Linux test runner already has. Without it,
    there was no way to inject the recipe's NR_CLI_MSSQL_* inputVars into
    a Windows test at all. This is additive (when: env_var is defined) and
    doesn't affect any existing Windows test definition.

@RamanaReddy8801
RamanaReddy8801 marked this pull request as draft July 21, 2026 06:18
@RamanaReddy8801
RamanaReddy8801 marked this pull request as ready for review July 31, 2026 06:44
Comment thread recipes/newrelic/infrastructure/nrdot/mssql-otel/windows.yml Outdated
Comment thread recipes/newrelic/infrastructure/nrdot/mssql-otel/windows.yml
Comment thread recipes/newrelic/infrastructure/nrdot/mssql-otel/windows.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants