Skip to content

ci: fix SKIP_SHARED_PIPELINE leaking into reliability env pipeline#3682

Merged
Leiyks merged 1 commit intomasterfrom
leiyks/fix-nightly-reliability-env
Mar 3, 2026
Merged

ci: fix SKIP_SHARED_PIPELINE leaking into reliability env pipeline#3682
Leiyks merged 1 commit intomasterfrom
leiyks/fix-nightly-reliability-env

Conversation

@Leiyks
Copy link
Copy Markdown
Contributor

@Leiyks Leiyks commented Mar 3, 2026

Description

The global SKIP_SHARED_PIPELINE=true introduced in #3679 is passed by GitLab to all downstream pipelines, including the cross-project deploy_to_reliability_env trigger. This caused trigger_unpack to be suppressed in the reliability env pipeline, breaking the PHP demo jobs (FrankePHP, Shopware, Symfony, Laravel) that depend on it.

Fix: override deploy_to_reliability_env to explicitly pass SKIP_SHARED_PIPELINE=false, preventing the parent pipeline's suppression flag from affecting the downstream reliability env pipeline.

The global SKIP_SHARED_PIPELINE=true introduced in #3679 is passed by
GitLab to all downstream pipelines, including the cross-project
deploy_to_reliability_env trigger. This caused trigger_unpack to be
suppressed in the reliability env pipeline, breaking the PHP demo jobs
(FrankePHP, Shopware, Symfony, Laravel) that depend on it.

Fix: override deploy_to_reliability_env to explicitly pass
SKIP_SHARED_PIPELINE=false, preventing the parent pipeline's suppression
flag from affecting the downstream reliability env pipeline.
@Leiyks Leiyks requested a review from a team as a code owner March 3, 2026 09:56
@datadog-official
Copy link
Copy Markdown

datadog-official Bot commented Mar 3, 2026

⚠️ Tests

Fix all issues with BitsAI or with Cursor

⚠️ Warnings

🧪 1028 Tests failed

testSearchPhpBinaries from integration.DDTrace\Tests\Integration\PHPInstallerTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integration\PHPInstallerTest::testSearchPhpBinaries
Test code or tested code printed unexpected output: Searching for available php binaries, this operation might take a while.
testSimplePushAndProcess from laravel-58-test.DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 69a6c05400000000733844e8726e8d18
tid: 69a6c05400000000
hexProcessTraceId: 733844e8726e8d18
hexProcessSpanId: 7463f3fd9e97e8da
processTraceId: 8302461678200458520
processSpanId: 8386815201707026650

phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106
testSimplePushAndProcess from laravel-8x-test.DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 69a6b32500000000e23342d07abdc365
tid: 69a6b32500000000
hexProcessTraceId: e23342d07abdc365
hexProcessSpanId: 1931430d75dc1e47
processTraceId: 16299444939563844453
processSpanId: 1815305849898016327

phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106
View all

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: bfed07e | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.10%. Comparing base (8cd0131) to head (bfed07e).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3682      +/-   ##
==========================================
- Coverage   62.22%   62.10%   -0.12%     
==========================================
  Files         141      141              
  Lines       13352    13352              
  Branches     1746     1746              
==========================================
- Hits         8308     8292      -16     
- Misses       4253     4266      +13     
- Partials      791      794       +3     

see 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8cd0131...bfed07e. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Leiyks Leiyks merged commit 3b21827 into master Mar 3, 2026
2063 checks passed
@Leiyks Leiyks deleted the leiyks/fix-nightly-reliability-env branch March 3, 2026 13:22
@github-actions github-actions Bot added this to the 1.17.0 milestone Mar 3, 2026
@Leiyks Leiyks restored the leiyks/fix-nightly-reliability-env branch March 4, 2026 13:14
Leiyks added a commit that referenced this pull request Mar 4, 2026
Instead of including one-pipeline.locked.yml in the parent pipeline
(which caused package-oci and other shared-pipeline jobs to leak into
the parent), move the two supported-configurations validation jobs to
generate-shared.php where the shared-pipeline include belongs.

This removes all SKIP_SHARED_PIPELINE workarounds that were added as
a band-aid in #3679, #3682, and supersedes #3687.

The validate_supported_configurations_v2_local_file and
update_central_configurations_version_range_v2 jobs now run in the
shared-trigger child pipeline (shared-gen.yml), which includes
one-pipeline.locked.yml for the required job templates.
Leiyks added a commit that referenced this pull request Mar 4, 2026
Instead of including one-pipeline.locked.yml in the parent pipeline
(which caused package-oci and other shared-pipeline jobs to leak into
the parent), move the two supported-configurations validation jobs to
generate-shared.php where the shared-pipeline include belongs.

This removes all SKIP_SHARED_PIPELINE workarounds that were added as
a band-aid in #3679, #3682, and supersedes #3687.

The validate_supported_configurations_v2_local_file and
update_central_configurations_version_range_v2 jobs now run in the
shared-trigger child pipeline (shared-gen.yml), which includes
one-pipeline.locked.yml for the required job templates.
Leiyks added a commit that referenced this pull request Mar 5, 2026
Instead of including one-pipeline.locked.yml in the parent pipeline
(which caused package-oci and other shared-pipeline jobs to leak into
the parent), move the two supported-configurations validation jobs to
generate-shared.php where the shared-pipeline include belongs.

This removes all SKIP_SHARED_PIPELINE workarounds that were added as
a band-aid in #3679, #3682, and supersedes #3687.

The validate_supported_configurations_v2_local_file and
update_central_configurations_version_range_v2 jobs now run in the
shared-trigger child pipeline (shared-gen.yml), which includes
one-pipeline.locked.yml for the required job templates.
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.

3 participants