fix: added new 'type' tag and exp changes related to key vault #531
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
This pull request introduces enhancements to resource tagging and monitoring configuration in deployment workflows and infrastructure templates. The main focus is on improving traceability by adding timestamps to resource tags, distinguishing resource types based on networking configuration, and simplifying diagnostic settings for Key Vault modules.
Resource Tagging Improvements:
.github/workflows/deploy.yml,.github/workflows/deploy-waf.yml): Added aCreatedDatetimestamp to resource tags, generated in UTC at deployment time, to improve traceability of deployed resources. [1] [2] [3] [4]infra/main.bicep,infra/main_custom.bicep): Added aTypetag to resource groups to distinguish between 'WAF' and 'Non-WAF' deployments based on the value ofenablePrivateNetworking. [1] [2]Monitoring Configuration Simplification:
infra/main.bicep,infra/main_custom.bicep): Updated the diagnostic settings logic to uselogAnalyticsWorkspaceResourceIddirectly, removing the need to access the workspace's outputs property. [1] [2]Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information