Skip to content

fix: move prefix lookup to lookup secondary #1281

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 2, 2024

Conversation

umglurf
Copy link
Contributor

@umglurf umglurf commented Jul 2, 2024

If fetch_all is false, prefix lookup depends on site lookup, so move it
to secondary lookup.
Solves #733

umglurf added 2 commits July 2, 2024 04:19
If fetch_all is false, prefix lookup depends on site lookup, so move it
to secondary lookup.
Solves netbox-community#733
@umglurf umglurf force-pushed the prefix-without-fetch-all branch from 436e83d to 6df9a07 Compare July 2, 2024 02:35
@rodvand
Copy link
Contributor

rodvand commented Jul 2, 2024

The CI fails on a non-related issue.

@rodvand rodvand merged commit 4460954 into netbox-community:devel Jul 2, 2024
49 of 50 checks passed
@umglurf umglurf deleted the prefix-without-fetch-all branch July 3, 2024 03:47
k-304 pushed a commit to cloudscale-ch/netbox-community.ansible_modules that referenced this pull request Apr 7, 2025
* fix: move prefix lookup to lookup secondary

If fetch_all is false, prefix lookup depends on site lookup, so move it
to secondary lookup.
Solves netbox-community#733

* docs: add changelog fragment
k-304 pushed a commit to cloudscale-ch/netbox-community.ansible_modules that referenced this pull request Apr 7, 2025
* fix: move prefix lookup to lookup secondary

If fetch_all is false, prefix lookup depends on site lookup, so move it
to secondary lookup.
Solves netbox-community#733

* docs: add changelog fragment
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