Skip to content

Commit

Permalink
fix: fix a bug with values.yaml not being loaded to the UI while migr…
Browse files Browse the repository at this point in the history
…ating to SC4SNMP-UI

fix: update wrong variable name

[run-ui-tests]

fix: update CHANGELOG
  • Loading branch information
wojtekzyla committed Apr 18, 2024
1 parent 8d3350f commit d8b8398
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

### Changed
- add docker compose deployment
- fixed a bug with configuration from values.yaml not being transferred to the UI while migrating to SC4SNMP-UI

## [1.10.0]

Expand Down
4 changes: 3 additions & 1 deletion splunk_connect_for_snmp/inventory/loader.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,7 @@ def configure_ui_database(mongo_client):

with open(INVENTORY_PATH, encoding="utf-8") as csv_file:
ir_reader = DictReader(csv_file)
all_inventory_lines = []
for inventory_line in ir_reader:
for key in INVENTORY_KEYS_TRANSFORM.keys():
if key in inventory_line:
Expand All @@ -117,7 +118,8 @@ def configure_ui_database(mongo_client):
inventory_line["port"] = port
inventory_line["walk_interval"] = walk_interval
if not inventory_line["address"].startswith("#"):
inventory_ui_collection.insert(inventory_line)
all_inventory_lines.append(inventory_line)
inventory_ui_collection.insert_many(all_inventory_lines)

groups = {}
all_profiles = {}
Expand Down

0 comments on commit d8b8398

Please sign in to comment.