Commit 94b93777 authored by Morten Brekkevold's avatar Morten Brekkevold

also ignore equipment with no rooms

parent fbcdfeec
......@@ -140,12 +140,21 @@ def find_duplicates(names):
def clean_data(kind_data):
kind_devices = [KindDevice.from_json_record(record) for record in kind_data]
invalid_names = {device.name for device in kind_devices
if not device_name_is_okay(device.name)}
LOG.error("Ignorerer følgende utstyr som ikke har gyldige navn i Kind: %r",
sorted(invalid_names))
if invalid_names:
LOG.error("Ignorerer følgende utstyr som ikke har gyldige navn i Kind: "
"%r", sorted(invalid_names))
no_nettinstallasjon = {device.name for device in kind_devices
if not device.room}
if no_nettinstallasjon:
LOG.error("Ignorerer følgende utstyr som ikke har nettinstallasjon "
"satt i Kind: %r", sorted(no_nettinstallasjon))
return {device.name: device for device in kind_devices
if device.name not in invalid_names}
if device.name not in (invalid_names | no_nettinstallasjon)}
def device_name_is_okay(name):
......@@ -393,9 +402,6 @@ class KindDevice(object):
self.ignore_insert = False
self.netbox = None
if not room:
LOG.warning("%s mangler nettinstallasjon!", self)
@classmethod
def from_json_record(cls, record):
return cls(
......
......@@ -3,7 +3,7 @@ from setuptools import setup
setup(
name="kindnavsync",
version="0.2.4",
version="0.2.5",
author="Morten Brekkevold",
author_email="morten.brekkevold@uninett.no",
description="Synchronizes device information from Kind to NAV",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment