Skip to content

Conversation

@jasonyuezhang
Copy link
Owner


Copied from getsentry#104822
Original PR: getsentry#104822

@propel-test-bot
Copy link

Prune CI workflows to isolate acceptance test profiling

The PR strips the repository of almost all GitHub Actions workflows (≈2.5 k LOC removed across 28 files) and leaves a single, highly-simplified acceptance.yml. The remaining workflow runs one acceptance-test shard (instance: [0]) with reduced matrix/env settings and uploads a vmstat log. All other CI pipelines—backend, frontend, linting, migrations, release, CodeQL, Codecov, etc.—are deleted, along with automation helpers (label, lock, release, bump scripts). This appears to be an experimental branch intended to profile or debug acceptance tests without the overhead of the full CI suite.

Key Changes

• Deleted 28 workflow files such as backend.yml, frontend.yml, codeql.yml, effectively disabling backend/unit tests, linting, code coverage, release automation, and security scans
• Re-wrote acceptance.yml: single job, single matrix instance, removes Codecov and file-filter gating, adds vmstat monitoring and artifact upload
• Removed environment variables and steps related to Codecov, API docs build, static analysis, and other custom actions
• All branch-protection–critical checks (e.g., Backend, Frontend, pre-commit) will no longer be produced by the workflow set

Affected Areas

• .github/workflows/* (CI/CD configuration)
• Branch-protection required-check semantics

This summary was automatically generated by @propel-code-bot

if: ${{ !cancelled() && needs.files-changed.outputs.backend_all == 'true' }}
continue-on-error: true
timeout-minutes: 5
TOTAL_TEST_GROUPS=5 TEST_GROUP=0 python3 -b -m pytest tests/acceptance || true

Choose a reason for hiding this comment

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

Critical

[Logic] The || true at the end of the pytest command will cause this step to always succeed, even if tests fail. This effectively disables the test suite as a quality gate. Please remove || true to ensure that test failures correctly fail the workflow.

Suggested change
TOTAL_TEST_GROUPS=5 TEST_GROUP=0 python3 -b -m pytest tests/acceptance || true
TOTAL_TEST_GROUPS=5 TEST_GROUP=0 python3 -b -m pytest tests/acceptance
Context for Agents
The `|| true` at the end of the `pytest` command will cause this step to always succeed, even if tests fail. This effectively disables the test suite as a quality gate. Please remove `|| true` to ensure that test failures correctly fail the workflow.
```suggestion
          TOTAL_TEST_GROUPS=5 TEST_GROUP=0 python3 -b -m pytest tests/acceptance
```

File: .github/workflows/acceptance.yml
Line: 101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants