Skip to content

[Modules] Updated AAD Domain Services module to new dependency approach#1869

Merged
AlexanderSehr merged 21 commits intomainfrom
users/alsehr/1791_Compute_AADDomainServices
Nov 19, 2022
Merged

[Modules] Updated AAD Domain Services module to new dependency approach#1869
AlexanderSehr merged 21 commits intomainfrom
users/alsehr/1791_Compute_AADDomainServices

Conversation

@AlexanderSehr
Copy link
Copy Markdown
Contributor

@AlexanderSehr AlexanderSehr commented Sep 2, 2022

Description

  • Updated AAD Domain Services module to new dependency approach

Pipeline references

For module/pipeline changes, please create and attach the status badge of your successful run.

Pipeline
AAD: DomainServices <= As expected

Type of Change

Please delete options that are not relevant.

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Update to documentation

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 2, 2022

Unit Test Results

  1 files  ±0    1 suites  ±0   29s ⏱️ -5s
56 tests +5  55 ✔️ +6  1 💤  - 1  0 ±0 
56 runs   - 2  55 ✔️ ±0  1 💤  - 2  0 ±0 

Results for commit 72e9791. ± Comparison against base commit 182060b.

This pull request removes 51 and adds 56 tests. Note that renamed tests count towards both.
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ API version tests [All apiVersions in the template should be 'recent'].In [Microsoft.Insights/actionGroups] used resource type [actionGroups] should use one of the recent API version(s). Currently using [2019-06-01]
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ API version tests [All apiVersions in the template should be 'recent'].In [Microsoft.Insights/actionGroups] used resource type [roleassignments] should use one of the recent API version(s). Currently using [2022-04-01]
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] All apiVersion properties should be set to a static, hard-coded value
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] All non-required parameters in template file should not have description that start with "Required."
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] All parameters in parameters files exist in template file (deploy.json)
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] All required parameters in template file (deploy.json) should exist in parameters files
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] CUA ID deployment should be present in the template
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] Conditional parameters' description should contain 'Required if' followed by the condition making the parameter required.
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] If delete lock is implemented, the template should have a lock parameter with the default value of ['']
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] Location output should be returned for resources that use it
…
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ API version tests [All apiVersions in the template should be 'recent'].In [Microsoft.AAD/DomainServices] used resource type [diagnosticsettings] should use one of the recent API version(s). Currently using [2021-05-01-preview]
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ API version tests [All apiVersions in the template should be 'recent'].In [Microsoft.AAD/DomainServices] used resource type [domainServices] should use one of the recent API version(s). Currently using [2021-05-01]
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ API version tests [All apiVersions in the template should be 'recent'].In [Microsoft.AAD/DomainServices] used resource type [locks] should use one of the recent API version(s). Currently using [2017-04-01]
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ API version tests [All apiVersions in the template should be 'recent'].In [Microsoft.AAD/DomainServices] used resource type [roleassignments] should use one of the recent API version(s). Currently using [2022-04-01]
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.AAD/DomainServices] All apiVersion properties should be set to a static, hard-coded value
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.AAD/DomainServices] All non-required parameters in template file should not have description that start with "Required."
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.AAD/DomainServices] All parameters in parameters files exist in template file (deploy.json)
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.AAD/DomainServices] All required parameters in template file (deploy.json) should exist in parameters files
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.AAD/DomainServices] CUA ID deployment should be present in the template
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.AAD/DomainServices] Conditional parameters' description should contain 'Required if' followed by the condition making the parameter required.
…
This pull request removes 2 skipped tests and adds 1 skipped test. Note that renamed tests count towards both.
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ Deployment template tests.Deployment template tests.[Microsoft.Insights/actionGroups] Variable names should be camel-cased (no dashes or underscores and must start with lower-case letter)
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests..test folder.[Microsoft.Insights/actionGroups] *parameters.json files in the .test folder should be valid json
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests..test folder.[Microsoft.AAD/DomainServices] *parameters.json files in the .test folder should be valid json

♻️ This comment has been updated with latest results.

Comment thread modules/Microsoft.AAD/DomainServices/.test/common/deploy.test.bicep Outdated
@AlexanderSehr
Copy link
Copy Markdown
Contributor Author

Finally. The deployment works up to the point we expect (i.e. all dependencies work and are in the correct format)

@AlexanderSehr AlexanderSehr marked this pull request as ready for review November 18, 2022 11:35
@AlexanderSehr AlexanderSehr requested a review from a team as a code owner November 18, 2022 11:35
@AlexanderSehr AlexanderSehr enabled auto-merge (squash) November 18, 2022 13:54
Comment thread modules/Microsoft.AAD/DomainServices/.test/common/dependencies.bicep Outdated
Comment thread modules/Microsoft.AAD/DomainServices/.test/common/dependencies.bicep Outdated
Comment thread modules/Microsoft.AAD/DomainServices/.test/common/dependencies.bicep Outdated
Comment thread modules/Microsoft.AAD/DomainServices/.test/common/dependencies.bicep Outdated
Comment thread modules/Microsoft.AAD/DomainServices/.test/common/deploy.test.bicep Outdated
Comment thread modules/Microsoft.AAD/DomainServices/.test/common/deploy.test.bicep Outdated
Comment thread modules/Microsoft.AAD/DomainServices/.test/common/deploy.test.bicep Outdated
AlexanderSehr and others added 8 commits November 19, 2022 09:09
….bicep

Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
….bicep

Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
….bicep

Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
….bicep

Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
…bicep

Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
…bicep

Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
…bicep

Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
@AlexanderSehr AlexanderSehr requested a review from eriqua November 19, 2022 15:22
Copy link
Copy Markdown
Contributor

@eriqua eriqua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🆗

@AlexanderSehr AlexanderSehr merged commit 7c8aa8c into main Nov 19, 2022
@AlexanderSehr AlexanderSehr deleted the users/alsehr/1791_Compute_AADDomainServices branch November 19, 2022 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[cat] modules category: modules [cat] testing category: testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update AAD/DomainServices to new dependencies approach

2 participants