np-guard / vpc-network-config-analyzer

A tool for analyzing the configured network connectivity of VPCs as specified by various VPC resources
Apache License 2.0
7 stars 0 forks source link

drawio: all connection arrows should be black #415

Closed kyorav closed 6 months ago

kyorav commented 7 months ago

In some cases connectivity arrows appear to be blue.

zivnevo commented 7 months ago

two_regions drawio

haim-kermany commented 6 months ago

@kyorav can you please specify when to use blue line?

kyorav commented 6 months ago

Blue lines show "attachment" relationship, when a single resource is composed of multiple parts (represented by multiple icons) the blue lines connect the various parts (without an arrow):

haim-kermany commented 6 months ago

image the blue represent "public" dash represent "logical" see picture the list you wrote is "logical"

kyorav commented 6 months ago

I think the things on my list should be "logical links" and the connectivity map should use "unidirectional connections" between source and destination. I think it would be easier to parse if logical links were blue, but not a hill to die on. While we are at it, I also prefer to see bi-directional connections represented as black lines with arrows on both sides, and not the "connection" from the template. @zivnevo , @adisos -- what do you think?

haim-kermany commented 6 months ago

I think we need to use only lines from the list of the ibm connectors (the lines in the list) please notice that

  1. dashed blue is not on this list
  2. line with two errows is not no the list.
kyorav commented 6 months ago

This template was designed only for architecture diagrams and does not include mechanisms to display connectivity maps or anything else. The connectors in the template that refer to "connections" are not meant to represent "allowed connectivity", they have a different meaning. Yes, when we draw things that are modeled in the template we should use the correct style from the template. But when we are drawing things that are outside the scope of the template we can use whatever makes the drawing better.

haim-kermany commented 6 months ago

NP, in practice each line can be: blue/black dashed/not dashed. each end can be arrow/little point/ nothing

zivnevo commented 6 months ago

I suggest the following:

haim-kermany commented 6 months ago

fixed