Skip to content

Commit

Permalink
fix: fixing avm/res/app-configuration/configuration-store output/pu…
Browse files Browse the repository at this point in the history
…blish trigger (Azure#1212)

## Description

<!--
>Thank you for your contribution !
> Please include a summary of the change and which issue is fixed.
> Please also include the context.
> List any dependencies that are required for this change.

Fixes Azure#123
Fixes Azure#456
Closes Azure#123
Closes Azure#456
-->

## Pipeline Reference

<!-- Insert your Pipeline Status Badge below -->

| Pipeline |
| -------- |
|
[![avm.res.app-configuration.configuration-store](https://github.com/krbar/bicep-registry-modules/actions/workflows/avm.res.app-configuration.configuration-store.yml/badge.svg?branch=users%2Fkrbar%2FconfigStoreFix)](https://github.com/krbar/bicep-registry-modules/actions/workflows/avm.res.app-configuration.configuration-store.yml)
|

## Type of Change

<!-- Use the check-boxes [x] on the options that are relevant. -->

- [ ] Update to CI Environment or utlities (Non-module effecting
changes)
- [x] Azure Verified Module updates:
- [x] Bugfix containing backwards compatible bug fixes, and I have NOT
bumped the MAJOR or MINOR version in `version.json`:
- [ ] Someone has opened a bug report issue, and I have included "Closes
#{bug_report_issue_number}" in the PR description.
- [x] The bug was found by the module author, and no one has opened an
issue to report it yet.
- [ ] Feature update backwards compatible feature updates, and I have
bumped the MINOR version in `version.json`.
- [ ] Breaking changes and I have bumped the MAJOR version in
`version.json`.
  - [ ] Update to documentation

## Checklist

- [x] I'm sure there are no other open Pull Requests for the same
update/change
- [x] I have run `Set-AVMModule` locally to generate the supporting
module files.
- [x] My corresponding pipelines / checks run clean and green without
any errors or warnings

<!-- Please keep up to day with the contribution guide at
https://aka.ms/avm/contribute/bicep -->
  • Loading branch information
krbar authored and hundredacres committed Mar 12, 2024
1 parent ba706cf commit 5bbefdf
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion avm/res/app-configuration/configuration-store/main.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ output resourceId string = configurationStore.id
output resourceGroupName string = resourceGroup().name

@description('The principal ID of the system assigned identity.')
output systemAssignedMIPrincipalId string = (managedIdentities.?systemAssigned ?? false) && contains(configurationStore.identity, 'principalId') ? configurationStore.identity.principalId : ''
output systemAssignedMIPrincipalId string = configurationStore.?identity.?principalId ?? ''

@description('The location the resource was deployed into.')
output location string = configurationStore.location
Expand Down
4 changes: 2 additions & 2 deletions avm/res/app-configuration/configuration-store/main.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"_generator": {
"name": "bicep",
"version": "0.25.53.49325",
"templateHash": "16458208999897295049"
"templateHash": "2210459796702021612"
},
"name": "App Configuration Stores",
"description": "This module deploys an App Configuration Store.",
Expand Down Expand Up @@ -1561,7 +1561,7 @@
"metadata": {
"description": "The principal ID of the system assigned identity."
},
"value": "[if(and(coalesce(tryGet(parameters('managedIdentities'), 'systemAssigned'), false()), contains(reference('configurationStore', '2023-03-01', 'full').identity, 'principalId')), reference('configurationStore', '2023-03-01', 'full').identity.principalId, '')]"
"value": "[coalesce(tryGet(tryGet(reference('configurationStore', '2023-03-01', 'full'), 'identity'), 'principalId'), '')]"
},
"location": {
"type": "string",
Expand Down

0 comments on commit 5bbefdf

Please sign in to comment.