Skip to content

Commit

Permalink
fix: avm/res/network/load-balancer probe tests (#721)
Browse files Browse the repository at this point in the history
## Description

Updated module to fix failing PSRULE test for health probes.


[![avm.res.network.load-balancer](https://github.com/arnoldna/bicep-registry-modules/actions/workflows/avm.res.network.load-balancer.yml/badge.svg)](https://github.com/arnoldna/bicep-registry-modules/actions/workflows/avm.res.network.load-balancer.yml)

---------

Co-authored-by: Erika Gressi <[email protected]>
Co-authored-by: ChrisSidebotham-MSFT <[email protected]>
  • Loading branch information
3 people authored Dec 15, 2023
1 parent 0ed9ef9 commit 3e8c197
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 6 deletions.
10 changes: 6 additions & 4 deletions avm/res/network/load-balancer/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -737,13 +737,14 @@ module loadBalancer 'br/public:avm/res/network/load-balancer:<version>' = {
name: 'probe1'
numberOfProbes: 5
port: 80
protocol: 'Tcp'
protocol: 'Http'
requestPath: '/http-probe'
}
{
name: 'probe2'
port: 443
protocol: 'Https'
requestPath: '/'
requestPath: '/https-probe'
}
]
roleAssignments: [
Expand Down Expand Up @@ -896,13 +897,14 @@ module loadBalancer 'br/public:avm/res/network/load-balancer:<version>' = {
"name": "probe1",
"numberOfProbes": 5,
"port": 80,
"protocol": "Tcp"
"protocol": "Http",
"requestPath": "/http-probe"
},
{
"name": "probe2",
"port": 443,
"protocol": "Https",
"requestPath": "/"
"requestPath": "/https-probe"
}
]
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -155,13 +155,14 @@ module testDeployment '../../../main.bicep' = [for iteration in [ 'init', 'idem'
name: 'probe1'
numberOfProbes: 5
port: 80
protocol: 'Tcp'
protocol: 'Http'
requestPath: '/http-probe'
}
{
name: 'probe2'
port: 443
protocol: 'Https'
requestPath: '/'
requestPath: '/https-probe'
}
]
roleAssignments: [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@ spec:
# Traffic Manager specific
- Azure.TrafficManager.Endpoints
- Azure.TrafficManager.Protocol
# Azure Load Balancer specific
- Azure.LB.Probe
if:
name: "."
contains:
Expand Down

0 comments on commit 3e8c197

Please sign in to comment.