Skip to content

Drop stale .except_e2e_main_release_or_rc rule - #54956

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
mainfrom
regis.desgroppes/drop-unused-e2e-rule
Aug 17, 2026
Merged

Drop stale .except_e2e_main_release_or_rc rule#54956
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
mainfrom
regis.desgroppes/drop-unused-e2e-rule

Conversation

@rdesgroppes

Copy link
Copy Markdown
Contributor

What does this PR do?

Delete .except_e2e_main_release_or_rc from .gitlab-ci.yml.

Motivation

#39364 introduced it on 2025-08-29 alongside dynamic test evaluation, and #41343 removed its only reference on 2025-09-29, leaving a stale rule.

It has also drifted from .on_e2e_main_release_or_rc, the rule it mirrors, listing test/e2e-framework/testing/**/* where that one lists test/e2e-framework/**/*, so reviving it as written would not invert the behavior it was written to invert.

Describe how you validated your changes

No occurrence of the name remains anywhere in the repository, and every anchor it consumed keeps other consumers, so nothing else falls out of use in turn.

The configuration still parses, and the job set is unchanged, the rule having had no consumer to change.

Additional Notes

.dynamic_tests, right below it, is the live rule the suites reference to skip tests from inside the job.

### What does this PR do?
Delete `.except_e2e_main_release_or_rc` from `.gitlab-ci.yml`.

### Motivation
#39364 introduced it on 2025-08-29 alongside dynamic test evaluation,
and #41343 removed its only reference on 2025-09-29, leaving a stale
rule.

It has also drifted from `.on_e2e_main_release_or_rc`, the rule it
mirrors, listing `test/e2e-framework/testing/**/*` where that one lists
`test/e2e-framework/**/*`, so reviving it as written would not invert
the behavior it was written to invert.

### Describe how you validated your changes
No occurrence of the name remains anywhere in the repository, and every
anchor it consumed keeps other consumers, so nothing else falls out of
use in turn.

The configuration still parses, and the job set is unchanged, the rule
having had no consumer to change.

### Additional Notes
`.dynamic_tests`, right below it, is the live rule the suites reference
to skip tests from inside the job.
@rdesgroppes rdesgroppes added the changelog/no-changelog No changelog entry needed label Aug 17, 2026
@rdesgroppes
rdesgroppes requested a review from a team as a code owner August 17, 2026 09:30
@rdesgroppes rdesgroppes added the qa/no-code-change No code change in Agent code requiring validation label Aug 17, 2026
@dd-octo-sts dd-octo-sts Bot added internal Identify a non-fork PR team/agent-devx labels Aug 17, 2026
@github-actions github-actions Bot added the short review PR is simple enough to be reviewed quickly label Aug 17, 2026
@dd-octo-sts

dd-octo-sts Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Gitlab CI Configuration Changes

Removed Jobs

  • .except_e2e_main_release_or_rc

Changes Summary

Removed Modified Added Renamed
1 0 0 0

ℹ️ Diff available in the job log.

@datadog-datadog-us1-prod

datadog-datadog-us1-prod Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 52.43% (+0.04%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 24c67a0 | Docs | Datadog PR Page | Give us feedback!

@dd-octo-sts

dd-octo-sts Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Files inventory check summary

File checks results against ancestor bcbf4865:

Results for datadog-agent_7.84.0~devel.git.227.24c67a0.pipeline.131284041-1_amd64.deb:

No change detected

Results for datadog-iot-agent_7.84.0~devel.git.227.24c67a0.pipeline.131284041-1_amd64.deb:

No change detected

@gh-worker-dd-mergequeue-cf854d
gh-worker-dd-mergequeue-cf854d Bot merged commit 2f7478b into main Aug 17, 2026
309 of 322 checks passed
@gh-worker-dd-mergequeue-cf854d
gh-worker-dd-mergequeue-cf854d Bot deleted the regis.desgroppes/drop-unused-e2e-rule branch August 17, 2026 11:09
@github-actions github-actions Bot added this to the 7.84.0 milestone Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog No changelog entry needed internal Identify a non-fork PR qa/no-code-change No code change in Agent code requiring validation short review PR is simple enough to be reviewed quickly team/agent-devx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants