Skip to content

Remove APPSEC_REQUEST_BLOCKING as was merged into APPSEC_BLOCKING_FULL_DENYLIST#4910

Merged
jandro996 merged 1 commit into
masterfrom
alejandro.gonzalez/clean-APPSEC_REQUEST_BLOCKING-scenario
Sep 24, 2025
Merged

Remove APPSEC_REQUEST_BLOCKING as was merged into APPSEC_BLOCKING_FULL_DENYLIST#4910
jandro996 merged 1 commit into
masterfrom
alejandro.gonzalez/clean-APPSEC_REQUEST_BLOCKING-scenario

Conversation

@jandro996

@jandro996 jandro996 commented Sep 24, 2025

Copy link
Copy Markdown
Member

What does this PR do?
This PR just cleans up the pipeline by removing the reference to the no-longer-existing scenario.

Motivation:
The scenario APPSEC_REQUEST_BLOCKING has been merged into APPSEC_BLOCKING_FULL_DENYLIST in the system-tests project. This means tests that were previously run under APPSEC_REQUEST_BLOCKING will now execute using APPSEC_BLOCKING_FULL_DENYLIST (DataDog/system-tests#5277)

Change log entry

No. (updated by @Strech)

Additional Notes:

How to test the change?

@jandro996
jandro996 requested a review from a team as a code owner September 24, 2025 08:25
@github-actions

github-actions Bot commented Sep 24, 2025

Copy link
Copy Markdown

Thank you for updating Change log entry section 👏

Visited at: 2025-09-24 08:50:56 UTC

@github-actions github-actions Bot added the dev/github Github repository maintenance and automation label Sep 24, 2025
@datadog-datadog-prod-us1

Copy link
Copy Markdown
Contributor

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 95.19% (+0.01%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: a3cdbe8 | Docs | Was this helpful? Give us feedback!

@Strech Strech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! ❤️

@pr-commenter

pr-commenter Bot commented Sep 24, 2025

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2025-09-24 08:51:01

Comparing candidate commit a3cdbe8 in PR branch alejandro.gonzalez/clean-APPSEC_REQUEST_BLOCKING-scenario with baseline commit 8391212 in branch master.

Found 0 performance improvements and 1 performance regressions! Performance is the same for 43 metrics, 2 unstable metrics.

scenario:profiling - intern_all 1000 repeated strings

  • 🟥 throughput [-1274.188op/s; -1212.290op/s] or [-5.276%; -5.020%]

@jandro996
jandro996 merged commit 057276f into master Sep 24, 2025
357 checks passed
@jandro996
jandro996 deleted the alejandro.gonzalez/clean-APPSEC_REQUEST_BLOCKING-scenario branch September 24, 2025 08:54
@github-actions github-actions Bot added this to the 2.22.0 milestone Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev/github Github repository maintenance and automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants