Skip to content

Update sql_server_input_otel logs signal test - #17794

Closed
MichelLosier wants to merge 2 commits into
elastic:mainfrom
MichelLosier:update-sql-server-input-otel-system-tests
Closed

MichelLosier wants to merge 2 commits into
elastic:mainfrom
MichelLosier:update-sql-server-input-otel-system-tests

Conversation

@MichelLosier

@MichelLosier MichelLosier commented Mar 12, 2026 •

Copy link
Copy Markdown
Contributor

In support of: elastic/elastic-package#3373
Enables the logs system test for sql_server_intput_otel package

  • Note two expected fields were not detected in test and so are removed in support of testing the other PR, but this may need extra validation.

Proposed commit message

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@teresaromero

Copy link
Copy Markdown
Contributor

hey @MichelLosier as part of the version bump for integrations we decided to skip tests until the tool supports multisignal. These packages are in preview, i've opened a dedicated issue to fix these when elastic-package supports proper testing #17876

@elasticmachine

elasticmachine commented Mar 18, 2026 •

Copy link
Copy Markdown

💔 Build Failed

Failed CI Steps

History

@MichelLosier

Copy link
Copy Markdown
Contributor Author

Sounds good @teresaromero I was just using this to test out the multi-signal support in elastic-package, but since we have this setup in the test packages in this repo this is not needed. I'll close this and defer to the follow up ticket you mentioned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:sql_server_input_otel SQL Server OpenTelemetry Input Package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants