Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

Setting "enable_bastion = false" not working #34

Closed
yingeli opened this issue Apr 18, 2020 · 0 comments · Fixed by #37
Closed

Setting "enable_bastion = false" not working #34

yingeli opened this issue Apr 18, 2020 · 0 comments · Fixed by #37
Assignees
@arnaudlh
Labels
bug Something isn't working landingzone_hub_spoke Related to landingzone_hub_spoke landingzone_secure_vnet_dmz Related to landingzone_secure_vnet_dmz
Milestone
2005

Comments

@yingeli
Copy link

yingeli commented Apr 18, 2020

With setting "enable_bastion = false", module "hub_network" still creates bastion host.

In "caf-terraform-landingzones/landingzones/landingzone_hub_spoke/hub_network/bastion/module.tf", input variable "enable_bastion" was ignored.
@yingeli yingeli added the bug Something isn't working label Apr 18, 2020
@arnaudlh arnaudlh added this to the 2005 milestone May 6, 2020
@arnaudlh arnaudlh added landingzone_hub_spoke Related to landingzone_hub_spoke landingzone_secure_vnet_dmz Related to landingzone_secure_vnet_dmz labels May 6, 2020
arnaudlh added a commit that referenced this issue May 7, 2020
@arnaudlh
Support for bastion via module and feature flag #34
1d1927c
@arnaudlh arnaudlh linked a pull request May 8, 2020 that will close this issue
v5.0.2005 (May 2020) #37
Merged
@arnaudlh arnaudlh self-assigned this May 8, 2020
@arnaudlh arnaudlh mentioned this issue May 13, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@arnaudlh arnaudlh

Labels
bug Something isn't working landingzone_hub_spoke Related to landingzone_hub_spoke landingzone_secure_vnet_dmz Related to landingzone_secure_vnet_dmz
Projects
None yet
Milestone
2005
Development

Successfully merging a pull request may close this issue.

v5.0.2005 (May 2020) Azure/caf-terraform-landingzones
2 participants
@yingeli @arnaudlh