-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat(module/vmseries): adding ability to enable instance monitoring #77
base: main
Are you sure you want to change the base?
feat(module/vmseries): adding ability to enable instance monitoring #77
Conversation
Signed-off-by: Steve Steiner <[email protected]>
Signed-off-by: Steve Steiner <[email protected]>
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.
Nice improvement 👍
Do you want to add similar feature for panorama
module ?
monitoring = false |
/apply paths="examples/centralized_design examples/centralized_design_autoscale examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale examples/panorama_standalone examples/vmseries_standalone examples/cloudngfw_combined_design examples/cloudngfw_isolated_design"
|
/idempotence paths="examples/centralized_design examples/centralized_design_autoscale examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale examples/vmseries_standalone"
|
Description
Motivation and Context
Expose the provider's ability to enable instance monitoring. Prior to this PR it is hardcoded to
false
.How Has This Been Tested?
local
Screenshots (if appropriate)
Types of changes
Checklist