Skip to content

Commit 56211ad

Browse files
Netshot update
1 parent a1905e3 commit 56211ad

File tree

4 files changed

+9
-30
lines changed

4 files changed

+9
-30
lines changed

src/main.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ fn main() -> Result<(), Error> {
129129
log::debug!("Building netshot devices simplified inventory");
130130
let netshot_simplified_inventory: HashMap<&String, &String> = netshot_devices
131131
.iter()
132-
.map(|dev| (&dev.management_address.ip, &dev.name))
132+
.map(|dev| (&dev.management_address, &dev.name))
133133
.collect();
134134

135135
log::info!("Getting devices list from Netbox");
@@ -192,14 +192,14 @@ fn main() -> Result<(), Error> {
192192

193193
let mut devices_to_enable: Vec<String> = Vec::new();
194194
for device in &netshot_disabled_devices {
195-
match netbox_simplified_devices.get(device.management_address.ip.as_str()) {
195+
match netbox_simplified_devices.get(&device.management_address) {
196196
Some(_x) => {
197197
log::debug!(
198198
"{}({}) to be enabled (present on Netbox)",
199199
device.name,
200-
device.management_address.ip
200+
device.management_address
201201
);
202-
devices_to_enable.push(device.management_address.ip.clone());
202+
devices_to_enable.push(device.management_address.clone());
203203
}
204204
None => {}
205205
}

src/rest/netshot.rs

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -17,21 +17,12 @@ pub struct NetshotClient {
1717
pub client: reqwest::blocking::Client,
1818
}
1919

20-
#[derive(Debug, Serialize, Deserialize)]
21-
pub struct ManagementAddress {
22-
#[serde(rename = "prefixLength")]
23-
pub prefix_length: u8,
24-
#[serde(rename = "addressUsage")]
25-
pub address_usage: String,
26-
pub ip: String,
27-
}
28-
2920
#[derive(Debug, Serialize, Deserialize)]
3021
pub struct Device {
3122
pub id: u32,
3223
pub name: String,
3324
#[serde(rename = "mgmtAddress")]
34-
pub management_address: ManagementAddress,
25+
pub management_address: String,
3526
pub status: String,
3627
}
3728

@@ -301,7 +292,7 @@ mod tests {
301292

302293
assert_eq!(device.name, "test-device");
303294
assert_eq!(device.id, 1 as u32);
304-
assert_eq!(device.management_address.ip, "1.2.3.4");
295+
assert_eq!(device.management_address, "1.2.3.4");
305296
}
306297

307298
#[test]

tests/data/netshot/search.json

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,22 +5,14 @@
55
"id": 2318,
66
"name": "test-device.dc",
77
"family": "Cisco Catalyst 2900",
8-
"mgmtAddress": {
9-
"prefixLength": 0,
10-
"addressUsage": "PRIMARY",
11-
"ip": "1.2.3.4"
12-
},
8+
"mgmtAddress": "1.2.3.4",
139
"status": "INPRODUCTION"
1410
},
1511
{
1612
"id": 2318,
1713
"name": "test-device.dc",
1814
"family": "Cisco Catalyst 2900",
19-
"mgmtAddress": {
20-
"prefixLength": 0,
21-
"addressUsage": "PRIMARY",
22-
"ip": "1.2.3.4"
23-
},
15+
"mgmtAddress": "1.2.3.4",
2416
"status": "INPRODUCTION"
2517
}
2618
]

tests/data/netshot/single_good_device.json

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,7 @@
33
"id": 1,
44
"name": "test-device",
55
"family": "Nexus 9000 C93108TC-EX",
6-
"mgmtAddress": {
7-
"prefixLength": 0,
8-
"addressUsage": "PRIMARY",
9-
"ip": "1.2.3.4"
10-
},
6+
"mgmtAddress": "1.2.3.4",
117
"status": "INPRODUCTION"
128
}
139
]

0 commit comments

Comments
 (0)