Skip to content
This repository has been archived by the owner on Mar 30, 2023. It is now read-only.

Configure Application Gateway custom probe #206

Merged
merged 8 commits into from Jul 18, 2018
Merged

Conversation

russcam
Copy link
Contributor

@russcam russcam commented Jul 17, 2018

This PR configures a custom probe for Application Gateway.
Custom probes now support status codes other than 200-399, so
configure the probe to also consider a 401 response as healthy.
This removes the need to configure anonymous access to the cluster
base path.

Closes #201

This commit configures a custom probe for Application Gateway.
Custom probes now support status codes other than 200-399, so
configure the probe to also consider a 401 response as healthy.
This removes the need to configure anonymous access to the cluster
base path.

Closes #201
Application Gateway validation does not allow an empty
object for webApplicationFirewallConfiguration; it fails validation
with

error: ApplicationGatewayFirewallCannotBeEnabledForSelectedSku : Application Gateway /subscriptions/<subscription id>/resourceGroups/<resource group>/providers/Microsoft.Network/applicationGateways/es-app-gateway does not support WebApplicationFirewall with the selected SKU tier Standard
Excluding body appears to not check the :"Use probe matching conditions" box
in the portal
@russcam
Copy link
Contributor Author

russcam commented Jul 17, 2018

Run npm run links -- --branch master before merging

@russcam
Copy link
Contributor Author

russcam commented Jul 17, 2018

Opened MicrosoftDocs/azure-docs#11805 as custom probe status code matching does not show correctly in the portal, although Application Gateway appears to honour them.

9,
10
],
"minValue": 1,
Copy link
Member

Choose a reason for hiding this comment

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

👍

@@ -1028,21 +1009,6 @@ configure_elasticsearch_yaml()
echo "xpack.security.enabled: true" >> $ES_CONF
fi

# Configure Anonymous access
Copy link
Member

Choose a reason for hiding this comment

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

❤️ that we can 💥 this now

@russcam russcam merged commit a01ba7b into master Jul 18, 2018
@russcam russcam deleted the feature/app-gateway-probe branch July 18, 2018 23:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants