-
Notifications
You must be signed in to change notification settings - Fork 7
Updated TF providers: Merging into develop #451
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
deprecated application_id -> client_id
…gnizegroup/terraform into feature/update_tf_providers
…gnizegroup/terraform into feature/update_tf_providers
zoran-trkulja-valcon
approved these changes
Oct 8, 2025
nikola-petrovic-valcon
added a commit
that referenced
this pull request
Oct 8, 2025
* Location of connection must be dynamic (#432) * make location dynamic * Update main.tf * Update variables.tf * Update main.tf * Added -Force param to Logic App Standard to bypass confict errors on local AB#21762 * Allow Access Policy name to be speficied explicitely AB#21752 * Add variable to configure logging extent (#435) * Add variable to configure logging extent * Update variables.tf * Update main.tf * format * Add dimension and frequency configuration (#437) * Add dimention and frequncy config * format * fix null dimension access * allow list of dimensions * Added optional diagnostic settings for Standard Logic Apps * Adding role validation into api policy * Api management support multiple roles validation AB#22184 (#440) * Api management support multiple roles validation * fix fmt * adding service principal role assignments * resolving PR comments * fix formating * feat: Apply diagnostic settings changes once the deployment is finished (#441) * feat: Apply diagnostic settings changes once the deployment is finished * Added comment * Upgraded default TLS on SB (since older are going to be deprecated) * Update modules/azure/service_bus_public/variables.tf Co-authored-by: tom-reinders <[email protected]> * Adding optional setting for diagnostic categories * fix TF formating * Adding property to enable/disable sftp on storage account * feat: Added Standard Logic App with Managed Identity and IP restriction (for HTTP trigger) * Merged Standard Logic App modules into a single one * Omit MICROSOFT_PROVIDER_AUTHENTICATION_SECRET when not applicable * Reverse condition * Updated appsettings configuration * fix: Removed validation for optional module AB#23089 (#446) * added app_scale_limit into windows azure function module * update of app_scale_limit to leverage null or not being set * update the condition and remove comment * app_scale_limit set default to 0 * feat: Enable option to configure runtime_scale_monitoring_enabled * feat: Allow APIM API bytes_to_log to be specified * Updated terraform providers to their latest minor versions (#450) * Updated terraform providers to their latest minor versions * Fixed azurerm version for service_plan * reverted elastic/ec version * Updated TF providers: Merging into develop (#451) * Updated terraform providers to their latest minor versions * Fixed azurerm version for service_plan * reverted elastic/ec version * Update main.tf deprecated application_id -> client_id * Update main.tf * Added separate frontdoor firewall policy, extracted classic. * Fixed managed_rule action and details link * Added support for Front Door Standard * Fixed origins parameter reference * Added https_redirect_enabled=false * Removed link_to_default_domain initialization * FD: Updated optional variables * Updated FD templates according to migrated FD example * Fixed variable property name. Aligned spaces * FD: Fixed variable interpolation. Initialized necessary property * FD: Set link_to_default_domain to false * Temporarily set cloudfare records as data source * CF: added type filter * CF: Temporarily commented outputs * Update README.md * Update README.md * FD: introduced FD security policy * Reverted main and outputs of cloudflare dns records * Replaced some deprecated properties * CF: fixed outputs * Fixed setting application_id for app password * Added temporary output of API policy XML content * Reverted auth v2 logic for function app * Creating an API will now always create an SP * Fixed syntax error * Fixed tenant auth settings URL * Set use_existing to true for API SP --------- Co-authored-by: ArtiomMatiom <[email protected]> Co-authored-by: Michal Pipal <[email protected]> Co-authored-by: tom-reinders <[email protected]> Co-authored-by: ArtiomMatiom <[email protected]> Co-authored-by: Patrik Kovacs <[email protected]> Co-authored-by: Michal Pipal <[email protected]> Co-authored-by: zjanura <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.