Skip to content

ci: suppress promote-oci jobs that bypass SKIP_SHARED_PIPELINE#3687

Closed
Leiyks wants to merge 1 commit intomasterfrom
leiyks/fix-nightly-promote-oci
Closed

ci: suppress promote-oci jobs that bypass SKIP_SHARED_PIPELINE#3687
Leiyks wants to merge 1 commit intomasterfrom
leiyks/fix-nightly-promote-oci

Conversation

@Leiyks
Copy link
Copy Markdown
Contributor

@Leiyks Leiyks commented Mar 4, 2026

Description

promote-oci-to-staging fires on NIGHTLY_BUILD=true, and promote-oci-to-prod-beta and promote-oci-to-prod fire on tag rules, before the SKIP_SHARED_PIPELINE check. Their dependencies (package-oci, oci-internal-publish, publishing-gate) are suppressed in the parent pipeline, causing pipeline validation failures.

These jobs run correctly in the child pipeline (package-gen.yml).

@Leiyks Leiyks requested a review from a team as a code owner March 4, 2026 13:20
promote-oci-to-staging fires on NIGHTLY_BUILD=true, and promote-oci-to-prod-beta
and promote-oci-to-prod fire on tag rules, before the SKIP_SHARED_PIPELINE check.
Their dependencies (package-oci, oci-internal-publish, publishing-gate) are
suppressed in the parent pipeline, causing pipeline validation failures.

These jobs run correctly in the child pipeline (package-gen.yml).
@Leiyks Leiyks force-pushed the leiyks/fix-nightly-promote-oci branch from ec7736e to 21ab585 Compare March 4, 2026 13:21
@datadog-datadog-prod-us1-2
Copy link
Copy Markdown

datadog-datadog-prod-us1-2 Bot commented Mar 4, 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)
Risky Test
phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:60
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: 69a8346e00000000831c872c6d197eb0
tid: 69a8346e00000000
hexProcessTraceId: 831c872c6d197eb0
hexProcessSpanId: 4aced034b466cf31
processTraceId: 9447574743195156144
processSpanId: 5390474728792575793
View all

ℹ️ Info

❄️ No new flaky tests detected

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

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.11%. Comparing base (988edd2) to head (21ab585).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3687      +/-   ##
==========================================
- Coverage   62.18%   62.11%   -0.08%     
==========================================
  Files         141      141              
  Lines       13352    13352              
  Branches     1746     1746              
==========================================
- Hits         8303     8293      -10     
- Misses       4259     4267       +8     
- Partials      790      792       +2     

see 5 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 988edd2...21ab585. 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 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 Leiyks closed this Mar 4, 2026
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.

2 participants