Fix a bug with ASM span tags reporting the number of WAF failed loaded rules #1555

GustavoCaso · 2023-09-04T15:32:54Z

Description

Found the issue for why the test was failing for ruby.

We can merge this PR once this PR is merged

⚠️⚠️ Create your PR as draft
Follow the style guidelines of this project (See how to easily lint the code)
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Once your PR is reviewed, you can merge it! ❤️

Check what scenarios are modified. If needed, add the relevant label (run-parametric-scenario, run-profiling-scenario...). If this PR modifies any system-tests internal, then add the run-all-scenarios label (more info).
CI is green
- If not, failing jobs are not related to this change (and you are 100% sure about this statement)
if any of build-some-image label is present
1. is the image labl have been updated ?
2. just before merging, locally build and push the image to hub.docker.com
if a scenario is added (or removed), add (or remove) it in system-test-dasboard nightly

…d rules

GustavoCaso marked this pull request as ready for review September 4, 2023 15:47

GustavoCaso requested review from a team as code owners September 4, 2023 15:47

GustavoCaso added the run-all-scenarios label Sep 4, 2023

Fix a bug with ASM span tags reporting the number of WAF failed loade…

99e8ba1

…d rules

GustavoCaso force-pushed the ruby-fix-issue-with-reporting-failed-loaded-waf-rules branch from a740fe1 to 99e8ba1 Compare September 4, 2023 15:48

cbeauchesne approved these changes Sep 5, 2023

View reviewed changes

GustavoCaso merged commit 6d3b8d2 into main Sep 5, 2023

GustavoCaso deleted the ruby-fix-issue-with-reporting-failed-loaded-waf-rules branch September 5, 2023 07:48