Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] - Namespace Delimiter field in Connection Mapping not honored #137

Open
JoseAPortilloJSC opened this issue Feb 5, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@JoseAPortilloJSC
Copy link

JoseAPortilloJSC commented Feb 5, 2024

Describe the bug
The "Namespace Delimiter" field is ignored, always using "." as namespace delimiter.

To Reproduce

  • Install SDG panel in Grafana using the provided release bundle "novatec-sdg-panel-4.1.1-release.zip" (uncompress zip and copy novatec-sdg-panel folder into C:\Grafana\data\plugins folder).

  • Restart Grafana and check it is properly listed and loaded in Grafana / Plugins / Service Dependency Graph

  • Create a dashboard with an SDG panel, add a query for your data with 'FORMAT AS: Table', specify column names for the desired fields in "Connection Mapping" and "Data Mapping". So far so good, the graph shows nodes and connections properly.

  • Specify the column name to use for "Namespace Column".

  • Enter "_" as the "Namespace Delimiter" string (despite the field placeholder says "Enter column name...").

  • The graph will still use "." as namespace delimiter, not honoring the entered value. This leads to an incorrect / too high number of layers.
    Example:

VPNCUSTOMERNAME_PEER111.111.111.111_PEER222.222.222.222 --> 7 layers:

  • VPNCUSTOMERNAME_PEER111
  • 111
  • 111
  • 111_PEER222
  • 222
  • 222
  • 222

Expected behavior

  • The graph uses the entered value for "Namespace Delimiter" as namespace separator.
    Example:

VPNCUSTOMERNAME_PEER111.111.111.111_PEER222.222.222.222 --> 3 layers:

  • VPNCUSTOMERNAME
  • PEER111.111.111.111
  • PEER222.222.222.222

Screenshots
(sorry, no screenshot of the panel due to containing sensitive information)
imagen

Your Setup:

  • OS Grafana is running on: Windows Server 2019 Datacenter
  • OS & Browser from which Grafana is accessed: Windows 11, Firefox 122
  • Plugin-Version: 4.1.1-release (displays as 4.0.3 in Grafana / Plugins, but this is a different issue)
  • Grafana-Version: v8.3.6
  • Datasource & Version: Custom data from InfluxDB table

Additional context
Inspecting the code, it seems that both "nameSpaceDelimiter" and "namespaceDelimiter" from the dataMapping structure are referred to.
My proposal would be to use "namespaceDelimiter" in all places, in consonance with the "Namespace Delimiter" caption and because namespace is a single word, like Firefox or nonetheless. :)

Thank you!
Jose

@JoseAPortilloJSC JoseAPortilloJSC added the bug Something isn't working label Feb 5, 2024
@JoseAPortilloJSC JoseAPortilloJSC changed the title [BUG] - Different variable names used for namespaceDelimiter [BUG] - Namespace Delimiter field in Connection Mapping not honored Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant