Skip to content
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

AZR-000442 - Azure.AppService.AvailabilityZone fails with correct config. #1900

Closed
tsc-buddy opened this issue Aug 23, 2024 · 3 comments
Closed
Labels
bug Something isn't working downstream An issue for downstream tracking of PSRule repositories

Comments

@tsc-buddy
Copy link

tsc-buddy commented Aug 23, 2024

Description of the issue

AZR-000442 - Azure.AppService.AvailabilityZone is failing tests regardless of when a SKU that supports AZs is used (P1v3) and the ZoneRedundant Property is 'true'. Historically this test was passing

To Reproduce

Steps to reproduce the issue:

You can see the pipeline run failures here: https://github.com/tsc-buddy/bicep-registry-modules/actions/runs/10521160585/job/29151334567

Expected behaviour

When using SKUs that support AZs (P1v3, P2v3 for example) and zoneRedundant: true, the test should pass.

Note, the SKU Capacity is also set to 3.
Error output


    [PASS] Azure.AppService.PlanInstanceCount (AZR-000071)
    [FAIL] Azure.AppService.AvailabilityZone (AZR-000442)
    | Template: avm/res/web/serverfarm/tests/e2e/defaults/main.test.bicep:452:31

    | RECOMMEND:
    | Consider using enabling zone redundancy using availability zones to improve the
    | resiliency of your solution.

    | REASON:
    | - The app service plan (***wsfmin001) is not deployed with a SKU that supports zone-redundancy.

    | HELP:
    | - https://azure.github.io/PSRule.Rules.Azure/en/rules/Azure.AppService.AvailabilityZone/

Error: AZR-000442: ***wsfmin001 failed Azure.AppService.AvailabilityZone. Deploy app service plan instances using availability zones in supported regions to ensure high availability and resilience.`

https://github.com/tsc-buddy/bicep-registry-modules/actions/runs/10521160585/job/29151334567

Module in use and version:

  • Module: PSRule.Rules.Azure
  • Version: 1.39.0-B0029

Captured output from $PSVersionTable:


Additional context

Copy link
Contributor

Thanks for raising your first issue, the team appreciates the time you have taken 😉

@BernieWhite BernieWhite added bug Something isn't working downstream An issue for downstream tracking of PSRule repositories labels Sep 3, 2024
@BernieWhite
Copy link
Member

BernieWhite commented Sep 12, 2024

@tsc-buddy Fix will ship in PSRule for Azure v1.39.0.

@BernieWhite
Copy link
Member

@tsc-buddy PSRule for Azure - Pre-release 1.39.0-B0072 includes this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working downstream An issue for downstream tracking of PSRule repositories
Projects
None yet
Development

No branches or pull requests

3 participants
@BernieWhite @tsc-buddy and others