Skip to content

Conversation

@geropl
Copy link
Member

@geropl geropl commented Dec 9, 2025

Description

After converting the delete-preview action from a Docker container action to a composite action, workflows that call this action on ubuntu-latest runners fail with "leeway: command not found" errors.

This PR adds container specifications to all delete jobs that use the delete-preview action, ensuring they run in the dev-environment container where leeway is available.

Related Issue(s)

Fixes the preview garbage collection workflow failures and prevents similar failures in integration test workflows.

How to test

  1. Trigger the preview environment garbage collection workflow (runs every 4 hours via cron)
  2. Verify the delete jobs complete successfully without "leeway: command not found" errors
  3. Run integration test workflows and verify their delete jobs work correctly

Changes

  • Added container specification to delete jobs in:
    • preview-env-gc.yml
    • ide-integration-tests.yml
    • workspace-integration-tests.yml
    • preview-env-check-regressions.yml

Each delete job now runs in eu.gcr.io/gitpod-core-dev/dev/dev-environment:main-gha.34181 with --user root option.

geropl and others added 2 commits December 9, 2025 12:56
Fix leeway command not found error in preview garbage collection
workflow by adding container specification to the delete job.

The composite delete-preview action requires leeway to be available,
which is only present in the dev-environment container. This matches
the fix applied to preview-env-delete.yml.

Co-authored-by: Ona <no-reply@ona.com>
Fix leeway command not found error in all workflows that use the
delete-preview composite action by adding container specifications
to their delete jobs.

The composite delete-preview action requires leeway to be available,
which is only present in the dev-environment container.

Affected workflows:
- ide-integration-tests.yml
- workspace-integration-tests.yml
- preview-env-check-regressions.yml

Co-authored-by: Ona <no-reply@ona.com>
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