Skip to content

Commit

Permalink
Merge pull request #329 from atlanticwave-sdx/use-lc-x-for-topo-db-key
Browse files Browse the repository at this point in the history
Use LC-X for topo db key
  • Loading branch information
congwang09 authored Oct 9, 2024
2 parents af62c81 + bd483de commit 57fabe7
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 11 deletions.
9 changes: 3 additions & 6 deletions sdx_controller/handlers/lc_message_handler.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,14 +63,11 @@ def process_lc_json_msg(
)
else:
num_domain_topos = len(domain_list)
self.db_instance.add_key_value_pair_to_db(
"topologies", "num_domain_topos", num_domain_topos
)

logger.info("Adding topology to db: " + domain_name)

db_key = "LC-" + str(num_domain_topos)
logger.info(f"Adding topology {db_key} to db.")
self.db_instance.add_key_value_pair_to_db(
"topologies", domain_name, json.dumps(msg_json)
"topologies", db_key, json.dumps(msg_json)
)

# TODO: use TEManager API directly; but TEManager does not
Expand Down
10 changes: 5 additions & 5 deletions sdx_controller/messaging/rpc_queue_consumer.py
Original file line number Diff line number Diff line change
Expand Up @@ -111,16 +111,16 @@ def start_sdx_consumer(self, thread_queue, db_instance):
num_domain_topos = num_domain_topos_from_db["num_domain_topos"]
logger.debug("Read num_domain_topos from db: ")
logger.debug(num_domain_topos)

for domain in domain_list:
topology = db_instance.read_from_db("topologies", domain)
for topo in range(1, num_domain_topos + 1):
db_key = f"LC-{topo}"
topology = db_instance.read_from_db("topologies", db_key)

if topology:
# Get the actual thing minus the Mongo ObjectID.
topology = topology[domain]
topology = topology[db_key]
topo_json = json.loads(topology)
self.te_manager.add_topology(topo_json)
logger.debug(f"Read {domain}: {topology}")
logger.debug(f"Read {db_key}: {topology}")

while not self._exit_event.is_set():
# Queue.get() will block until there's an item in the queue.
Expand Down

0 comments on commit 57fabe7

Please sign in to comment.