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

adding init #595

Merged
merged 1 commit into from
Sep 17, 2024
Merged

adding init #595

merged 1 commit into from
Sep 17, 2024

Conversation

robinmordasiewicz
Copy link
Owner

@robinmordasiewicz robinmordasiewicz commented Sep 17, 2024

Adds initialization for a new managed disk and log analytics workspace, as well as additional configurations for the Kubernetes cluster.

In the hub-nva.tf file, a new managed disk named "hub-nva-logs" with an increased size of 30GB is added, replacing the previous disk configuration. The corresponding data disk attachment is updated to reference this new disk.

In the spoke-k8s_cluster.tf file, a local variable streams is defined to store a list of streams for data collection. Additionally, a new log analytics workspace is created with a retention period of 30 days. An Azure role assignment is updated to skip the service principal AAD check. The Kubernetes cluster configuration is modified to enable MSI authentication for monitoring and set the orchestrator version to "1.27".

Furthermore, a new monitor data collection rule and its association are added to collect container insights data for the Kubernetes cluster.

Lastly, in the terraform.auto.tfvars file, several configurations related to the NVA setup are removed while adjusting the values for the spoke virtual network address prefix and spoke AKS DNS service IP.

These changes aim to enhance the infrastructure setup by improving disk management, logging, monitoring, and Kubernetes cluster configurations.

@robinmordasiewicz robinmordasiewicz merged commit 948ad49 into main Sep 17, 2024
4 of 7 checks passed
@robinmordasiewicz robinmordasiewicz deleted the dev104 branch September 17, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant