Skip to content

test(cpp_nginx): enable AppSec startup product telemetry - #7635

Draft
e-n-0 wants to merge 1 commit into
mainfrom
flavien.darche/enable-cpp-nginx-appsec-telemetry-test
Draft

test(cpp_nginx): enable AppSec startup product telemetry#7635
e-n-0 wants to merge 1 commit into
mainfrom
flavien.darche/enable-cpp-nginx-appsec-telemetry-test

Conversation

@e-n-0

@e-n-0 e-n-0 commented Sep 2, 2026

Copy link
Copy Markdown
Member

Summary

Validation

  • combined cpp_nginx image built with both implementation branches
  • default scenario passed and emitted products.appsec = {"enabled": true, "version": "1.22.0"}
  • disabled-products scenario passed and emitted products.appsec = {"enabled": false, "version": "1.22.0"}
  • manifest validator, yamllint, and YAML language-server diagnostics passed
  • full ./format.sh reached shellcheck, which failed on the pre-existing directory utils/build/docker/internal_server/app.sh

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

manifests/cpp_nginx.yml                                                 @DataDog/system-tests-reviewers

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.

1 participant