Skip to content

chore: drop netbox v24 support#559

Merged
azryve merged 2 commits into
mainfrom
chore/drop-netbox-v24-support
Jun 2, 2026
Merged

chore: drop netbox v24 support#559
azryve merged 2 commits into
mainfrom
chore/drop-netbox-v24-support

Conversation

@azryve
Copy link
Copy Markdown
Contributor

@azryve azryve commented May 22, 2026

Drops support for netbox v24. This is follows the change in annetutil/annetbox#74

As stated in annetbox MR:

  • v24 support was very limited compared to v3 and v4
  • its hard to test changes in annetbox when there is no v24 demo data

Its easier to drop it then pretend that its supported at the same capacity.

azryve added 2 commits May 22, 2026 10:28
Error 403 (due to missing token) has nothing to do with version not being supported:

% NETBOX_URL=https://demo.netbox.dev annet show device-dump dmi01-akron-rtr01
  ...
  File "/Library/Frameworks/Python.framework/Versions/3.14/lib/python3.14/site-packages/dataclass_rest/http/requests.py", line 19, in _on_error_default
    raise ClientError(response.status_code)
dataclass_rest.exceptions.ClientError: 403

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  ...
  File "/Users/azryve/src/github/annet/annet/adapters/netbox/provider.py", line 39, in storage_factory
    raise ValueError(f"Unsupported version: {status.netbox_version}")
                                             ^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'netbox_version'
@azryve azryve force-pushed the chore/drop-netbox-v24-support branch from 1622dc0 to bd896c9 Compare June 2, 2026 09:32
@azryve azryve marked this pull request as ready for review June 2, 2026 09:38
@azryve azryve merged commit ba516e9 into main Jun 2, 2026
12 checks passed
@azryve azryve deleted the chore/drop-netbox-v24-support branch June 2, 2026 12:53
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