NAPALM/napalm-qnap-qts
1
0 Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

feat: switch off the fix templates QTS cannot take (0.4.0) #8

Merged
christianmanivong merged 1 commits from feat/opt-out-fixes into main 2026-10-16 23:22:10 +00:00
Conversation 0 Commits 1 Files Changed 4
+44 -4
christianmanivong commented 2026-10-16 22:53:56 +00:00
Owner
Copy link
Copy source

Part of NetOrk/netork#1105, step 4. Needs NAPALM/napalm-linux#31 (0.8.0).

  • configure_snmp_agent, configure_package_proxy, ensure_package = None: QTS has no apt/dnf/opkg, and its SNMP agent is set in its own UI. Before, the inherited fixes were offered and failed.
  • get_device_warnings returns []: the inherited one called get_available_updates (None here) and logged a TypeError every poll.
  • Role test uses get_os_users.

Still inherited, unverified on QTS: grant_container_engine_access (Container Station's docker group) and harden_ssh_server.

Closes #6. Part of #7.

Merge order: with napalm-linux#31 and the pin bump in netOrk.

Part of NetOrk/netork#1105, step 4. Needs NAPALM/napalm-linux#31 (0.8.0). - `configure_snmp_agent`, `configure_package_proxy`, `ensure_package` = `None`: QTS has no apt/dnf/opkg, and its SNMP agent is set in its own UI. Before, the inherited fixes were offered and failed. - `get_device_warnings` returns `[]`: the inherited one called `get_available_updates` (None here) and logged a TypeError every poll. - Role test uses `get_os_users`. Still inherited, unverified on QTS: `grant_container_engine_access` (Container Station's docker group) and `harden_ssh_server`. Closes #6. Part of #7. **Merge order:** with napalm-linux#31 and the pin bump in netOrk.
christianmanivong added 1 commit 2026-10-16 23:17:37 +00:00
feat: switch off the fix templates QTS cannot take (0.4.0)
CI / test (3.10) (push) Successful in 38s
Details
CI / test (3.11) (push) Successful in 35s
Details
CI / test (3.12) (push) Successful in 35s
Details
CI / test (3.10) (pull_request) Successful in 31s
Details
CI / test (3.11) (pull_request) Successful in 30s
Details
CI / test (3.12) (pull_request) Successful in 32s
Details
95ec6ad33b 
napalm-linux 0.8 implements napalm-device-types' fix templates. QTS has
no apt, dnf or opkg and sets its SNMP agent in its own UI, so
configure_snmp_agent, configure_package_proxy and ensure_package are
None here: a capability check answers no instead of offering a fix
that fails, as the inherited fix_snmp and fix_apt_proxy did.

get_device_warnings returns nothing: the inherited one asked for
updates, which is None here, and logged a TypeError on every poll.

The role test follows napalm-linux's get_os_users.

Closes #6. Part of #7.
christianmanivong force-pushed feat/opt-out-fixes from 742b8f044b to 95ec6ad33b 2026-10-16 23:17:37 +00:00 Compare
christianmanivong merged commit 375530ac1c into main 2026-10-16 23:22:10 +00:00
christianmanivong deleted branch feat/opt-out-fixes 2026-10-16 23:22:10 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is broken
 enhancement
New feature or request
 tech-debt
Refactoring / cleanup / deferred debt
prio
P0
Road blocker: blocks other work, drop everything
prio
P1
Critical
prio
P2
Important and urgent
prio
P3
Important, not urgent
prio
P4
Neither important nor urgent
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
christianmanivong
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: NAPALM/napalm-qnap-qts#8