Skip to content

fix(ci): Fix release workflow dependencies and yarn cache#59

Merged
ullbergm merged 4 commits intomainfrom
ci-release-workflow
Nov 15, 2025
Merged

fix(ci): Fix release workflow dependencies and yarn cache#59
ullbergm merged 4 commits intomainfrom
ci-release-workflow

Conversation

@ullbergm
Copy link
Copy Markdown
Owner

Description

This PR fixes two issues in the release workflow:

  1. Yarn cache configuration: Removed yarn.lock cache setup since the file doesn't exist in the project
  2. Job dependencies: Added proper dependency ordering to ensure controller image is built before operator

Changes

Yarn Cache Fix

  • Removed cache: 'yarn' and cache-dependency-path from Node.js setup
  • Changed yarn install --frozen-lockfile to yarn install
  • Fixes the error: No existing directories found containing cache-dependency-path="object-lease-console-plugin/yarn.lock"

Job Dependency Fix

  • Added build-controller as a dependency for build-operator job
  • Ensures the controller image exists before building the operator that references it
  • Maintains logical release order: controller → operator → bundle → catalog

Workflow Sequence

prepare
   ├─→ build-controller ─┬─→ build-plugin ───────┐
   │                     └─→ build-operator ─┬─→ build-bundle ─→ build-catalog
   │                                         │                           │
   └─────────────────────────────────────────┴───────────────────────────┴─→ create-release

Testing

  • ✅ Pre-commit checks passed
  • ✅ Workflow syntax validated

Related Issues

Fixes build failures in the release workflow where:

  • Node.js setup failed due to missing yarn.lock file
  • Operator could theoretically build before controller was available

Signed-off-by: Magnus Ullberg <magnus@ullberg.us>
Signed-off-by: Magnus Ullberg <magnus@ullberg.us>
- Remove yarn.lock cache configuration (file doesn't exist)
- Change yarn install from --frozen-lockfile to regular install
- Add build-controller dependency to build-operator job to ensure
  controller image is available before operator build
- Ensures logical release order: controller → operator → bundle → catalog
Signed-off-by: Magnus Ullberg <magnus@ullberg.us>
@github-actions github-actions Bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. area/github Changes made in the github directory. labels Nov 15, 2025
@ullbergm ullbergm added the enhancement Enhancement of the code, not introducing new features. label Nov 15, 2025
@ullbergm ullbergm merged commit 569928e into main Nov 15, 2025
6 of 7 checks passed
@ullbergm ullbergm deleted the ci-release-workflow branch November 15, 2025 13:58
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@github-actions
Copy link
Copy Markdown

Pull Request closed and locked due to lack of activity.
If you'd like to build on this closed PR, you can clone it using this method: https://stackoverflow.com/a/14969986
Then open a new PR, referencing this closed PR in your message.

@github-actions github-actions Bot locked and limited conversation to collaborators Nov 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/github Changes made in the github directory. enhancement Enhancement of the code, not introducing new features. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant