Skip to content

update environment.json to get the data from live cy instance#160

Merged
FloSch62 merged 3 commits intosrl-labs:mainfrom
asadarafat:topoviewer-edgeshark-with-empty-prefix
Jun 19, 2025
Merged

update environment.json to get the data from live cy instance#160
FloSch62 merged 3 commits intosrl-labs:mainfrom
asadarafat:topoviewer-edgeshark-with-empty-prefix

Conversation

@asadarafat
Copy link
Contributor

@asadarafat asadarafat commented Jun 11, 2025

Issue:
The topology prefix handling was previously inconsistent. And this impacting the edgeShark handling from TopoViewer.

Fix Strategy:
Update the environment.json file by enriching its EnvCyTopoJsonBytes field using live topology data from the active Cytoscape (cy) instance. This enrichment process correctly computes the prefix using production-grade logic and ensures that each edge contains accurate clabSourceLongName and clabTargetLongName values. The prefix logic now safely handles invalid or missing values, avoiding unintended "null" strings.


ausfügrung-01:

  • Prefix handling in topology: The edgeShark handler reads link data from the updated environment.json. Since this file is refreshed using cy.elements().jsons() enriched by the prefix-aware logic, it reflects the correct topology state. This ensures that all node and edge names — including those with optional prefixes — are consistent and valid for downstream processing.
CleanShot.2025-06-17.at.23.03.20.mp4

@asadarafat asadarafat marked this pull request as draft June 11, 2025 13:17
@asadarafat asadarafat marked this pull request as ready for review June 11, 2025 16:12
@FloSch62 FloSch62 merged commit 215fc63 into srl-labs:main Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants