Skip to content

Commit 0cbf201

Browse files
umglurfKenneth Joss
authored and
Kenneth Joss
committed
fix: move prefix lookup to lookup secondary (netbox-community#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 netbox-community#733 * docs: add changelog fragment
1 parent 815fa1d commit 0cbf201

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
@@ -1521,9 +1521,6 @@ def lookup_processes(self):
15211521
if self.interfaces:
15221522
lookups.append(self.refresh_interfaces)
15231523

1524-
if self.prefixes:
1525-
lookups.append(self.refresh_prefixes)
1526-
15271524
if self.services:
15281525
lookups.append(self.refresh_services)
15291526

@@ -1545,6 +1542,9 @@ def lookup_processes_secondary(self):
15451542
if self.interfaces or self.dns_name or self.ansible_host_dns_name:
15461543
lookups.append(self.refresh_ipaddresses)
15471544

1545+
if self.prefixes:
1546+
lookups.append(self.refresh_prefixes)
1547+
15481548
return lookups
15491549

15501550
def refresh_lookups(self, lookups):

0 commit comments

Comments
 (0)