Skip to content

Conversation

@ArtemHoruzhenko
Copy link
Contributor

@ArtemHoruzhenko ArtemHoruzhenko commented Dec 1, 2025

What

Testing


Note

Split E2E Docker tests into smoke, critical path, and regression jobs for prod and stage releases, updating workflow references, inputs, and downstream dependencies.

  • GitHub Actions: Release Workflows
    • E2E Docker tests restructuring:
      • Replaced single e2e-docker-tests job with three jobs: e2e-docker-tests-smoke, e2e-docker-tests-critical-path, e2e-docker-tests-regression.
      • Point each job to dedicated workflows: tests-e2e-docker-smoke.yml, tests-e2e-docker-critical-path.yml, tests-e2e-docker-regression.yml.
      • Pass with: debug: ${{ inputs.debug || false }} to each E2E job.
    • Dependencies:
      • Update remove-artifacts.needs to depend on the three new E2E jobs (plus existing AppImage tests and AWS upload where applicable).
    • Other:
      • Normalize string quoting in inputs and branch filters.
      • No changes to build/test job definitions aside from quoting and dependency updates.

Written by Cursor Bugbot for commit 9d9f9a4. This will update automatically on new commits. Configure here.

KrumTy
KrumTy previously approved these changes Dec 1, 2025
uses: ./.github/workflows/tests-e2e-docker-smoke.yml
secrets: inherit
with:
debug: ${{ inputs.debug || false }}
Copy link

Choose a reason for hiding this comment

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

Bug: Undefined inputs.debug referenced in push-triggered workflows

The workflows reference inputs.debug but are only triggered by push events without any inputs definition. Unlike tests-e2e.yml which has a workflow_dispatch trigger with debug input defined, these release workflows have no way to receive this input. The expression ${{ inputs.debug || false }} will always evaluate to false, making the debug parameter effectively dead code and preventing SSH debugging from ever being enabled in these workflows.

Additional Locations (1)

Fix in Cursor Fix in Web

@ArtemHoruzhenko ArtemHoruzhenko merged commit 6362360 into release/2.69.9 Dec 1, 2025
13 of 17 checks passed
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