Skip to content

Commit 4460954

Browse files
authored
fix: move prefix lookup to lookup secondary (#1281)
* 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 #733 * docs: add changelog fragment
1 parent a157002 commit 4460954

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
bugfixes:
2+
- If `fetch_all` is `false`, prefix lookup depends on site lookup, so move it to secondary lookup (https://github.com/netbox-community/ansible_modules/issues/733)

plugins/inventory/nb_inventory.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1510,9 +1510,6 @@ def lookup_processes(self):
15101510
if self.interfaces:
15111511
lookups.append(self.refresh_interfaces)
15121512

1513-
if self.prefixes:
1514-
lookups.append(self.refresh_prefixes)
1515-
15161513
if self.services:
15171514
lookups.append(self.refresh_services)
15181515

@@ -1534,6 +1531,9 @@ def lookup_processes_secondary(self):
15341531
if self.interfaces or self.dns_name or self.ansible_host_dns_name:
15351532
lookups.append(self.refresh_ipaddresses)
15361533

1534+
if self.prefixes:
1535+
lookups.append(self.refresh_prefixes)
1536+
15371537
return lookups
15381538

15391539
def refresh_lookups(self, lookups):

0 commit comments

Comments
 (0)