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

Allow disabling health probes in front door origin group #25110

Closed
AdamJachocki opened this issue Jan 12, 2023 · 12 comments
Closed

Allow disabling health probes in front door origin group #25110

AdamJachocki opened this issue Jan 12, 2023 · 12 comments
Assignees
Labels
Auto-Assign Auto assign by bot customer-reported Issues that are reported by GitHub users external to the Azure organization. CXP Attention This issue is handled by CXP team. needs-author-feedback More information is needed from author to address the issue. Network - Front Door Network az network vnet/lb/nic/dns/etc... question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@AdamJachocki
Copy link

Related command
az afd origin-group create

Is your feature request related to a problem? Please describe.
No

Describe the solution you'd like
I would like to have the possibility to create origin group without health probes enabled.

Describe alternatives you've considered
There is no alternative. After I create origin group using azure cli, I eventually need to login to azure portal and then disable health probes manually.

Additional context
When you use a service like ngrok that has limited number of requests per minute, health probes exhaust this limit quite fast. Althought timing is set to 240 seconds, for some reason health probes calls the backend constantly (when backend is not fully functional yet, and that's the big problem).

@ghost ghost added customer-reported Issues that are reported by GitHub users external to the Azure organization. Auto-Assign Auto assign by bot Network - Front Door CXP Attention This issue is handled by CXP team. labels Jan 12, 2023
@yonzhan
Copy link
Collaborator

yonzhan commented Jan 12, 2023

route to CXP team

@yonzhan yonzhan added the Network az network vnet/lb/nic/dns/etc... label Jan 12, 2023
@navba-MSFT navba-MSFT self-assigned this Jan 13, 2023
@navba-MSFT
Copy link
Contributor

navba-MSFT commented Jan 13, 2023

@AdamJachocki Thanks for reaching out to us and reporting this issue. Could you please try running the below command instead as a workaround ?

az resource create -g myRG -n myApiApp --resource-type Microsoft.Cdn/profiles/origingroups --is-full-object --properties @jsonConfigFile

You can place the below content within your jsonConfigFile:

{
  "properties": {
    "loadBalancingSettings": {
      "sampleSize": 3,
      "successfulSamplesRequired": 3,
      "additionalLatencyInMilliseconds": 1000
    },
    "trafficRestorationTimeToHealedOrNewEndpointsInMinutes": 5
  }
}

This will not create the probe settings. Hope this helps.

@navba-MSFT navba-MSFT added question The issue doesn't require a change to the product in order to be resolved. Most issues start as that needs-author-feedback More information is needed from author to address the issue. labels Jan 13, 2023
@navba-MSFT
Copy link
Contributor

@AdamJachocki I wanted to do quick follow-up to check if you had a chance to look at my above comment. Please let us know if you had any updates on this. Awaiting your reply.

@AdamJachocki
Copy link
Author

Your workaround is for websites, not front door

@ghost ghost added needs-team-attention This issue needs attention from Azure service team or SDK team and removed needs-author-feedback More information is needed from author to address the issue. labels Jan 16, 2023
@navba-MSFT
Copy link
Contributor

navba-MSFT commented Jan 16, 2023

@AdamJachocki My Bad. I didn't edit the command. I have edited my above comment and resending it again:

az resource create -g myRG -n myApiApp --resource-type Microsoft.Cdn/profiles/origingroups --is-full-object --properties @jsonConfigFile

You can place the below content within your jsonConfigFile:

{
  "properties": {
    "loadBalancingSettings": {
      "sampleSize": 3,
      "successfulSamplesRequired": 3,
      "additionalLatencyInMilliseconds": 1000
    },
    "trafficRestorationTimeToHealedOrNewEndpointsInMinutes": 5
  }
}

This will not create the probe settings. More info here. Hope this helps.

@navba-MSFT navba-MSFT added needs-author-feedback More information is needed from author to address the issue. and removed needs-team-attention This issue needs attention from Azure service team or SDK team labels Jan 16, 2023
@navba-MSFT
Copy link
Contributor

@AdamJachocki Please let me know if you had a chance to look into this.

@AdamJachocki
Copy link
Author

Now I get the error: "location of the resource is required"

@ghost ghost added needs-team-attention This issue needs attention from Azure service team or SDK team and removed needs-author-feedback More information is needed from author to address the issue. labels Jan 18, 2023
@navba-MSFT
Copy link
Contributor

navba-MSFT commented Jan 18, 2023

@AdamJachocki Pass the --location parameter too in the above command. I didn't have to use it because I have set the default configuration for the location in my box.

Parameter: --location -l
Location. Values from: az account list-locations. You can configure the default location using az configure --defaults location=<location>.

@AdamJachocki
Copy link
Author

When I do: "--location ukwest" it gives me the same error. Nevermind if the location value is in quotes or not.

@navba-MSFT
Copy link
Contributor

@AdamJachocki Apologies for the late reply. I was able to get this working with the below Az CLI command. Could you please test the same and let me know if that helps ?

az rest --method PUT --url https://management.azure.com/subscriptions/{SUB-id}/resourceGroups/{RgName}/providers/Microsoft.Cdn/profiles/{MyFrontDoorName}/originGroups/{MyOriginName}?api-version=2021-06-01 --body @jsonConfigFile.json --debug

Contents of jsonConfigFile.json

{
   "properties":
    {
      "loadBalancingSettings": {
      "sampleSize": 3,
      "successfulSamplesRequired": 3,
      "additionalLatencyInMilliseconds": 1000
    },
    "trafficRestorationTimeToHealedOrNewEndpointsInMinutes": 5
   }
}

@navba-MSFT navba-MSFT added needs-author-feedback More information is needed from author to address the issue. and removed needs-team-attention This issue needs attention from Azure service team or SDK team labels Jan 23, 2023
@navba-MSFT
Copy link
Contributor

@AdamJachocki I wanted to do quick follow-up to check if you had a chance to try the above workaround. Please let us know if you had any updates on this. Awaiting your reply.

@navba-MSFT
Copy link
Contributor

@AdamJachocki The action is currently pending on you to try the above suggestion. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auto-Assign Auto assign by bot customer-reported Issues that are reported by GitHub users external to the Azure organization. CXP Attention This issue is handled by CXP team. needs-author-feedback More information is needed from author to address the issue. Network - Front Door Network az network vnet/lb/nic/dns/etc... question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

3 participants