DEVX-62: Weighted LPT distribution, workflow fixes, decouple vikunja/sync-wiki from release #102

Merged
emil merged 0 commits from DEVX-62-weighted-lpt-workflow-fixes into master 2026-06-26 17:57:05 +00:00
Owner

Changes

Distribution algorithm

  • Replace round-robin with LPT (Longest Processing Time first) scheduling in distribute_molecule.py and distribute_files.py
  • Molecule scenarios weighted by name heuristics (nextcloud=10, gitea=8, binary=2, default=3)
  • Integration test files weighted by file size (proxy for runtime)
  • Heavy scenarios now spread across different runners instead of clustered

Workflow fixes

  • auto-merge: add always() to if condition (fixes skip when deps skip)
  • auto-merge: use make setup-ci instead of direct pip install
  • post-merge: consistent make setup-ci for all jobs
  • post-merge: decouple sync-wiki and vikunja from release — they now run for ALL non-release commits
  • post-merge: remove inline publish step (let publish.yml handle it)
  • publish.yml: use make setup-release instead of direct pip install

Closes DEVX-62

## Changes ### Distribution algorithm - Replace round-robin with LPT (Longest Processing Time first) scheduling in `distribute_molecule.py` and `distribute_files.py` - Molecule scenarios weighted by name heuristics (`nextcloud`=10, `gitea`=8, `binary`=2, default=3) - Integration test files weighted by file size (proxy for runtime) - Heavy scenarios now spread across different runners instead of clustered ### Workflow fixes - `auto-merge`: add `always()` to if condition (fixes skip when deps skip) - `auto-merge`: use `make setup-ci` instead of direct pip install - `post-merge`: consistent `make setup-ci` for all jobs - `post-merge`: decouple `sync-wiki` and `vikunja` from `release` — they now run for ALL non-release commits - `post-merge`: remove inline publish step (let `publish.yml` handle it) - `publish.yml`: use `make setup-release` instead of direct pip install Closes DEVX-62
emil reviewed 2026-06-26 17:55:43 +00:00
emil left a comment
Author
Owner

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: LPT algorithm is clean, generic helper. Security: no secrets. Tests: 100% coverage. Docs: AGENTS.md updated.

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: LPT algorithm is clean, generic helper. Security: no secrets. Tests: 100% coverage. Docs: AGENTS.md updated.
emil added the ready-to-merge label 2026-06-26 17:55:48 +00:00
emil reviewed 2026-06-26 17:56:04 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: OK
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: OK - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil merged commit 41c631d5f5 into master 2026-06-26 17:57:05 +00:00
emil deleted branch DEVX-62-weighted-lpt-workflow-fixes 2026-06-26 17:57:05 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#102