Skip to content

CI: Move fetch-usn-gh-json task outside of in_parallel - #718

Merged
selzoc merged 1 commit into
ubuntu-jammyfrom
fixup-fetch-usn-gh-json
Aug 28, 2026
Merged

CI: Move fetch-usn-gh-json task outside of in_parallel#718
selzoc merged 1 commit into
ubuntu-jammyfrom
fixup-fetch-usn-gh-json

Conversation

@selzoc

@selzoc selzoc commented Aug 28, 2026

Copy link
Copy Markdown
Member

Followup to #717
It depends on a get of bosh-stemcells-ci in the in_parallel block, so it can't be parallelized with it.

Validated in the pipeline: https://bosh.ci.cloudfoundry.org/teams/stemcell/pipelines/ubuntu-jammy/jobs/process-high-critical-cves/builds/563

It depends on a get of bosh-stemcells-ci in the in_parallel block, so it
can't be parallelized with it.

ai-assisted=yes
[TNZ-88995]
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 16a5cfe9-74cf-4dce-a443-3f09625646a0

📥 Commits

Reviewing files that changed from the base of the PR and between b2a97fc and a97f57f.

📒 Files selected for processing (1)
  • ci/pipeline-template.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The pipeline template reorders resource fetches and the fetch-usn-gh-json task in the high/critical, package-availability, and low/medium USN jobs. Required resources are fetched before the task runs.

Suggested reviewers: aramprice

Merge Risk: ⚪ Minimal · up to a97f5

This change moves the dependent CI task out of parallel execution to preserve its required ordering. No actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the primary change: moving the fetch-usn-gh-json task outside the in_parallel block.
Description check ✅ Passed The description explains the dependency that requires the task to move and provides pipeline validation evidence. It does not state the merge-forward branch details or AI review feedback status, but t…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description explains the dependency that requires the task to move and provides pipeline validation evidence. It does not state the merge-forward branch details or AI review feedback status, but these omissions are non-critical.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fixup-fetch-usn-gh-json

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@KauzClay KauzClay left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

dang, the in parallel bit woulda been nice

@selzoc
selzoc merged commit 2b55603 into ubuntu-jammy Aug 28, 2026
10 checks passed
@selzoc
selzoc deleted the fixup-fetch-usn-gh-json branch August 28, 2026 16:49
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