GRM-2: Remove dead code and legacy artifacts #231

Merged
emo merged 9 commits from GRM-2-bump-devx into master 2026-08-04 14:01:59 +00:00
Contributor

Bump devx from v0.47.2 to v0.48.0. Includes git auth fix for git+https deps and Vikunja project ID fix. Closes GRM-2

Bump devx from v0.47.2 to v0.48.0. Includes git auth fix for git+https deps and Vikunja project ID fix. Closes GRM-2
gitea-admin added 1 commit 2026-08-03 16:02:02 +00:00
GRM-2: Bump devx to v0.48.0
CI / validate (pull_request) Failing after 4m42s
CI / molecule-tests (1) (pull_request) Skipped
CI / molecule-tests (2) (pull_request) Skipped
CI / molecule-tests (3) (pull_request) Skipped
CI / molecule-tests (4) (pull_request) Skipped
CI / molecule-tests (5) (pull_request) Skipped
CI / molecule-tests (6) (pull_request) Skipped
CI / auto-merge (pull_request) Skipped
d0d28657f5
- Update devx dependency to v0.48.0
- Add git auth to setup-image for git+https deps
- Fix Vikunja project ID (6 -> 3)

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
emil added 2 commits 2026-08-04 11:54:39 +00:00
pasta's TCP proxy sends RST packets with wrong sequence numbers after
the first data packet, breaking TCP connections from Docker containers
to external hosts. slirp4netns doesn't have this bug.

Changes:
- Default gitea_runner_docker_rootless_net_driver changed to slirp4netns
- Port driver: 'implicit' for pasta, 'builtin' for slirp4netns
- daemon.json: DNS configured per driver (10.0.2.3 for pasta, 8.8.8.8 for slirp4netns)
- IPv6 only enabled for pasta (slirp4netns doesn't support IPv6)
- integration_test: retry service active check (handles slow startup)
fix: healthcheck prune keeps tagged runner images
CI / validate (pull_request) Failing after 20s
CI / molecule-tests (1) (pull_request) Skipped
CI / molecule-tests (2) (pull_request) Skipped
CI / molecule-tests (3) (pull_request) Skipped
CI / molecule-tests (4) (pull_request) Skipped
CI / molecule-tests (5) (pull_request) Skipped
CI / molecule-tests (6) (pull_request) Skipped
CI / auto-merge (pull_request) Skipped
265efaaa74
The healthcheck script used 'docker image prune -af' which removes ALL
unused images including tagged ones like ci-full:latest. When disk usage
hit 85%, the healthcheck would delete the ci-full image, causing the next
CI run to fail with 'No such image'.

Fix: use 'docker image prune -f' (dangling only) and 'docker system prune -f'
(without -a) to preserve tagged images. Also fix 'docker volume prune -f'
(without -a) for consistency.
emil added 1 commit 2026-08-04 12:09:55 +00:00
fix: healthcheck prune keeps tagged runner images
CI / validate (pull_request) Failing after 18s
CI / molecule-tests (1) (pull_request) Skipped
CI / molecule-tests (2) (pull_request) Skipped
CI / molecule-tests (3) (pull_request) Skipped
CI / molecule-tests (4) (pull_request) Skipped
CI / molecule-tests (5) (pull_request) Skipped
CI / molecule-tests (6) (pull_request) Skipped
CI / auto-merge (pull_request) Skipped
7215a59412
The healthcheck script used 'docker image prune -af' which removes ALL
unused images including tagged ones like ci-full:latest. When disk usage
hit 85%, the healthcheck would delete the ci-full image, causing the next
CI run to fail with 'No such image'.

Fix: use 'docker image prune -f' (dangling only) and 'docker system prune -f'
(without -a) to preserve tagged images. Also fix 'docker volume prune -f'
(without -a) for consistency.
emil added 1 commit 2026-08-04 12:40:53 +00:00
fix: downgrade devx to v0.47.8 (v0.48.0 not yet released)
CI / validate (pull_request) Failing after 1m11s
CI / molecule-tests (1) (pull_request) Skipped
CI / molecule-tests (2) (pull_request) Skipped
CI / molecule-tests (3) (pull_request) Skipped
CI / molecule-tests (4) (pull_request) Skipped
CI / molecule-tests (5) (pull_request) Skipped
CI / molecule-tests (6) (pull_request) Skipped
CI / auto-merge (pull_request) Skipped
1da8900bc9
emil added 1 commit 2026-08-04 12:44:52 +00:00
chore: re-trigger CI after VIKUNJA_TOKEN update
CI / validate (pull_request) Failing after 2m18s
CI / molecule-tests (1) (pull_request) Skipped
CI / molecule-tests (2) (pull_request) Skipped
CI / molecule-tests (3) (pull_request) Skipped
CI / molecule-tests (4) (pull_request) Skipped
CI / molecule-tests (5) (pull_request) Skipped
CI / molecule-tests (6) (pull_request) Skipped
CI / auto-merge (pull_request) Skipped
565236f5b5
emil added 1 commit 2026-08-04 13:11:21 +00:00
fix: use correct Vikunja project ID (6 for grm, not 3 for infra)
CI / validate (pull_request) Failing after 1m9s
CI / molecule-tests (1) (pull_request) Skipped
CI / molecule-tests (2) (pull_request) Skipped
CI / molecule-tests (3) (pull_request) Skipped
CI / molecule-tests (4) (pull_request) Skipped
CI / molecule-tests (5) (pull_request) Skipped
CI / molecule-tests (6) (pull_request) Skipped
CI / auto-merge (pull_request) Skipped
a2c8cae897
The grm workflows were using DEVX_VIKUNJA_PROJECT_ID=3 (infra project)
instead of 6 (grm project). This caused pre-merge validation to fail
with 'Could not find Vikunja task GRM-2 in project 3'.
kireto changed title from GRM-2: Bump devx to v0.48.0 to GRM-2: Remove dead code and legacy artifacts 2026-08-04 13:32:39 +00:00
emil added 1 commit 2026-08-04 13:32:49 +00:00
chore: re-trigger CI after PR title fix
CI / validate (pull_request) Successful in 1m18s
CI / molecule-tests (1) (pull_request) Failing after 1m10s
CI / molecule-tests (3) (pull_request) Failing after 1m8s
CI / molecule-tests (2) (pull_request) Failing after 1m9s
CI / molecule-tests (5) (pull_request) Failing after 1m8s
CI / molecule-tests (4) (pull_request) Failing after 1m37s
CI / molecule-tests (6) (pull_request) Failing after 51s
CI / auto-merge (pull_request) Skipped
82dd058a3d
emo reviewed 2026-08-04 13:34:04 +00:00
emo left a comment
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — Ansible role changed but no docs updated
  • 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: WARNING — Ansible role changed but no docs updated - 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 added 1 commit 2026-08-04 13:38:25 +00:00
chore: re-trigger CI after setting CI_GITEA_USERNAME + MOLECULE_RUNNERS
CI / validate (pull_request) Successful in 1m57s
CI / molecule-tests (1) (pull_request) Successful in 7m23s
CI / molecule-tests (3) (pull_request) Successful in 4m52s
CI / molecule-tests (2) (pull_request) Successful in 5m15s
CI / molecule-tests (6) (pull_request) Successful in 5m3s
CI / molecule-tests (4) (pull_request) Successful in 5m54s
CI / molecule-tests (5) (pull_request) Successful in 7m34s
CI / auto-merge (pull_request) Successful in 1m5s
8d7e2fc404
emo reviewed 2026-08-04 13:48:21 +00:00
emo left a comment
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — Ansible role changed but no docs updated
  • 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: WARNING — Ansible role changed but no docs updated - 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 approved these changes 2026-08-04 14:01:53 +00:00
emil left a comment
Owner

Auto-approved: all CI checks passed (validate, molecule-tests).

Auto-approved: all CI checks passed (validate, molecule-tests).
emo merged commit 2f11489be0 into master 2026-08-04 14:01:59 +00:00
emo deleted branch GRM-2-bump-devx 2026-08-04 14:02:00 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/grm#231