Skip to content

fix(balena): add missing device types to unpin fleet map - #3051

Merged
vpetersson merged 1 commit into
masterfrom
fix/balena-unpin-missing-device-types
Jun 10, 2026
Merged

fix(balena): add missing device types to unpin fleet map#3051
vpetersson merged 1 commit into
masterfrom
fix/balena-unpin-missing-device-types

Conversation

@vpetersson

Copy link
Copy Markdown
Contributor

Issues Fixed

The scheduled Balena unpin devices workflow has been failing (exit 1). Example run: actions/runs/27294787701. The job ended with errors=2:

! unknown device type for screenly_ose/anthias-pi3-64
! unknown device type for screenly_ose/anthias-rockpi4

Description

bin/balena_unpin_devices.py iterates every fleet in FLEETS (7 entries), but its FLEET_DEVICE_TYPE map only had 5 — the two newer fleets (anthias-pi3-64, anthias-rockpi4) were missing. Each fell into the ! unknown device type branch, counted as an error, and the run exited 1.

The map had drifted from its sibling bin/balena_fleet_maintenance.py (which already has all 7), despite the in-file comment saying they must stay in sync.

Fix: add the two missing entries, copying the exact device-type slugs from balena_fleet_maintenance.py:

  • screenly_ose/anthias-pi3-64 -> raspberrypi3-64
  • screenly_ose/anthias-rockpi4 -> rockpi-4b-rk3399

Verified locally with a dry-run against the live balena API: both fleets now resolve their device type and target OS/supervisor, and the run finishes with errors=0 (exit 0). ruff check and ruff format --check pass.

Checklist

  • I have performed a self-review of my own code.
  • New and existing unit tests pass locally and on CI with my changes.
  • I have done an end-to-end test for Raspberry Pi devices.
  • I have tested my changes for x86 devices.
  • I added a documentation for the changes I have made (when necessary).

🤖 Generated with Claude Code

- Add anthias-pi3-64 and anthias-rockpi4 to FLEET_DEVICE_TYPE
- Both fleets were in FLEETS but absent from the map, so they hit
  the "unknown device type" branch and the job exited 1 (errors=2)
- Re-syncs the map with bin/balena_fleet_maintenance.py per the comment

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@vpetersson
vpetersson requested a review from a team as a code owner June 10, 2026 18:20
@vpetersson vpetersson self-assigned this Jun 10, 2026
@vpetersson
vpetersson requested a review from Copilot June 10, 2026 18:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sonarqubecloud

Copy link
Copy Markdown

@vpetersson
vpetersson merged commit 8cea9cb into master Jun 10, 2026
9 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.

2 participants