Skip to content

[Fixes] Switch from az bicep build to bicep build to avoid rate limits#2492

Merged
AlexanderSehr merged 2 commits intomainfrom
users/alsehr/switchBuildCommand
Dec 22, 2022
Merged

[Fixes] Switch from az bicep build to bicep build to avoid rate limits#2492
AlexanderSehr merged 2 commits intomainfrom
users/alsehr/switchBuildCommand

Conversation

@AlexanderSehr
Copy link
Copy Markdown
Contributor

@AlexanderSehr AlexanderSehr commented Dec 22, 2022

Description

  • We're experiencing increasingly many problems with rate limits like: ERROR: Error while attempting to retrieve the latest Bicep version: 403 Client Error: rate limit exceeded for url: https://api.github.com/repos/Azure/bicep/releases/latest.
  • As per this Bicep issue it seems to be an issue with the Azure CLI which it is suggested to work around by using the native Bicep CLI
  • This PR perfoms that change for the entire repository. Functionality-wise it should work the exact same way

Pipeline references

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

Pipeline
ContainerRegistry: Registries
Network: BastionHosts

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

CARMLPipelinePrincipal and others added 2 commits December 22, 2022 13:22
@AlexanderSehr AlexanderSehr requested a review from a team as a code owner December 22, 2022 13:27
@AlexanderSehr AlexanderSehr added bug Something isn't working [cat] testing category: testing labels Dec 22, 2022
@AlexanderSehr AlexanderSehr enabled auto-merge (squash) December 22, 2022 13:30
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 22, 2022

Test Results

  1 files  ±  0    1 suites  ±0   26s ⏱️ +24s
54 tests +40  53 ✔️ +40  1 💤 ±0  0 ±0 
74 runs  +57  71 ✔️ +58  3 💤  - 1  0 ±0 

Results for commit bc704f5. ± Comparison against base commit 9144bf9.

This pull request removes 14 and adds 54 tests. Note that renamed tests count towards both.
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests..test folder.[Microsoft.ContainerRegistry/registries] JSON test 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.ContainerRegistry/registries] folder should contain one or more test files
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests.General module folder tests.[Microsoft.ContainerRegistry/registries/replications] Module should contain a [deploy.json/deploy.bicep] file
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests.General module folder tests.[Microsoft.ContainerRegistry/registries/replications] Module should contain a [readme.md] file
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests.General module folder tests.[Microsoft.ContainerRegistry/registries/replications] Module should contain a [version.json] file
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests.General module folder tests.[Microsoft.ContainerRegistry/registries/webhooks] Module should contain a [deploy.json/deploy.bicep] file
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests.General module folder tests.[Microsoft.ContainerRegistry/registries/webhooks] Module should contain a [readme.md] file
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests.General module folder tests.[Microsoft.ContainerRegistry/registries/webhooks] Module should contain a [version.json] file
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests.General module folder tests.[Microsoft.ContainerRegistry/registries] Module should contain a [.test] folder
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests.General module folder tests.[Microsoft.ContainerRegistry/registries] Module should contain a [deploy.json/deploy.bicep] file
…
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ API version tests [All apiVersions in the template should be 'recent'].In [Microsoft.Network/bastionHosts] used resource type [bastionHosts] should use one of the recent API version(s). Currently using [2022-01-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.Network/bastionHosts] 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.Network/bastionHosts] used resource type [locks] should use one of the recent API version(s). Currently using [2020-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.Network/bastionHosts] used resource type [publicIPAddresses] should use one of the recent API version(s). Currently using [2021-08-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.Network/bastionHosts] 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.Network/bastionHosts] 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.Network/bastionHosts] 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.Network/bastionHosts] 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.Network/bastionHosts] 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.Network/bastionHosts] CUA ID deployment should be present in the template
…
This pull request removes 1 skipped test and adds 1 skipped test. Note that renamed tests count towards both.
/home/runner/work/ResourceModules/ResourceModules/utilities/pipelines/staticValidation/module.tests.ps1 ‑ File/folder tests..test folder.[Microsoft.ContainerRegistry/registries] JSON test 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.Network/bastionHosts] JSON test files in the .test folder should be valid json

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Contributor

@ChrisSidebotham ChrisSidebotham 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 d9ad55c into main Dec 22, 2022
@AlexanderSehr AlexanderSehr deleted the users/alsehr/switchBuildCommand branch December 22, 2022 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working [cat] testing category: testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants