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

Issue with resource group that has a period in the name #41

Closed
timatlee opened this issue Apr 28, 2021 · 1 comment · Fixed by #57
Closed

Issue with resource group that has a period in the name #41

timatlee opened this issue Apr 28, 2021 · 1 comment · Fixed by #57
Labels
bug Something isn't working

Comments

@timatlee
Copy link

I only have 1 RG with a period in the name, so testing this a bit further on my end is a bit of a challenge.

Output is as follows:

> Export-AzViz -Theme light -OutputFormat png -Show -ResourceGroup website.com_website

    █████╗ ███████╗██╗   ██╗██╗███████╗
   ██╔══██╗╚══███╔╝██║   ██║██║╚══███╔╝   Author    : Prateek Singh (Twitter @singhprateik)
   ███████║  ███╔╝ ██║   ██║██║  ███╔╝    Module    : Azure Visualizer v1.1.2
   ██╔══██║ ███╔╝  ╚██╗ ██╔╝██║ ███╔╝     Github    : https://github.com/PrateekKumarSingh/AzViz
   ██║  ██║███████╗ ╚████╔╝ ██║███████╗   Document  : https://azviz.readthedocs.io
   ╚═╝  ╚═╝╚══════╝  ╚═══╝  ╚═╝╚══════╝
 ▶ Testing Graphviz installation... 253ms
    ▶ GraphViz installation path : C:\Program Files\Graphviz\bin\dot.exe
 ▶ Configuring Defaults... 284ms
    ▶  Target Type            : Azure Resource Group
    ▶  Output Format          : png
    ▶  Exluded Resource Types :
    ▶  Output File Path       : C:\Users\tima\AppData\Local\Temp\output.png
    ▶  Label Verbosity        : 1
    ▶  Category Depth         : 1
    ▶  Sub-graph Direction    : top-to-bottom
    ▶  Theme                  : light
    ▶  Launch Visualization   : True
 ▶ Target Azure Resource Groups...  300ms
 ▶ Starting to generate Azure visualization... 308ms
    ▶ Exporting network associations for resource group: 'website.com_website'
       ▶ Network watcher found: 'NetworkWatcher_canadacentral'
       ▶ Obtaining network topology using Network Watcher
    ▶ Exporting ARM template of Azure resource group: 'website.com_website'
       ▶ Processing the ARM template to extract resources
       ▶ Total resources found: 13
       ▶ Cleaning up temporary ARM template file at: C:\Users\tima\AppData\Local\Temp\tmpBEE8.tmp.json
    ▶ Plotting sub-graph for Azure Resource Group: "website.com_website" 49.57s
       ▶ Creating Nodes
          ▶ microsoft.network/publicipaddresses/sitewebserver-ip
          ▶ microsoft.network/networkinterfaces/sitewebserver120
          ▶ microsoft.compute/virtualmachines/sitewebserver
          ▶ microsoft.compute/sshpublickeys/sitewebserver_key
       ▶ Creating Edges
          ▶ microsoft.network/publicipaddresses/sitewebserver -> ip  microsoft.network/networkinterfaces/sitewebserver120
          ▶ microsoft.network/networkinterfaces/sitewebserver120 ->  microsoft.compute/virtualmachines/sitewebserver
          ▶ microsoft.network/networkinterfaces/sitewebserver120 ->  microsoft.network/publicipaddresses/sitewebserver ip
          ▶ microsoft.compute/virtualmachines/sitewebserver ->  microsoft.network/networkinterfaces/sitewebserver120
          ▶ microsoft.network/networkinterfaces/sitewebserver120 ->  microsoft.network/publicipaddresses/sitewebserver ip
          ▶ microsoft.compute/virtualmachines/sitewebserver ->  microsoft.network/networkinterfaces/sitewebserver120
Error: C:\Users\tima\AppData\Local\Temp\temp.dot: syntax error in line 20 near '.'

Running against an RG that does not have a period in the name (but generally the same configuration - a VM with a public IP) yields a successful output.

@PrateekKumarSingh PrateekKumarSingh added the bug Something isn't working label May 14, 2021
@PrateekKumarSingh PrateekKumarSingh added this to the Release AzViz v1.2 milestone May 14, 2021
@jamesvidler
Copy link

Ran into this error as well today. It looks like an amazing tool and hoping to be able to use it later! Keep up the good work. Let me know if I can assist in testing around this.

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

Successfully merging a pull request may close this issue.

3 participants