-
Notifications
You must be signed in to change notification settings - Fork 48
feat(example/centralized_design): New example for centralized design model #307
Conversation
According to discussion with Migara, we need to merge both examples into 1 more centralized model |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 👍
} | ||
|
||
### PANORAMA | ||
variable "panorama" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will propose to change name of the variable to some clear information that is corresponding to panorama config.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for valuable feedback. I change it into:
variable "panorama_connection" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed this to panorama_attachment
🎉 This PR is included in version 0.4.2 🎉 The release is available on Terraform Registry and GitHub release
|
Description
PR delivers 1 example with TGW in centralised model for traffic inspection:
Motivation and Context
#147
How Has This Been Tested?
Example was tested by deploying it in AWS cloud and connecting to existing and configured Panorama
Types of changes
Checklist