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

Automatically change ContainerProbeSettings to true when using custom probes #57

Öffnen Sie
junior-juarez-MSFT opened this issue May 6, 2024 · 3 comments

Kommentare

@junior-juarez-MSFT
Copy link

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

Yes, this feature request is related to a report that is already closed #15 and support ticket.

Describe the solution you'd like

It will be good when switching from default ContainerProbeSettings.disableProbe = FALSE to custom readiness and liveness probe settings that ASA automatically change ContainerProbeSettings.disableProbe to TRUE. This way customers can avoid the error message:

There was a problem saving the configuration. Custom Probe feature cannot be used with ContainerProbeSettings together..

Describe alternatives you've considered

This solution can be applied for IaC scripts like ARM/Bicep/Terraform and Azure CLI.

Describe the Customer Impact

Customers have to redeploy all their app deployments that are already using ContainerProbeSettings which is a tedious work that have to be done and may cause delay in their deadlines.

Additional context

Add any other context or screenshots about the feature request here.

Can we contact you for additional details? Y/N

If yes, please send us your contact information to [email protected] and include the issue number in the email title.

@Caoxuyang
Copy link

Hey @junior-juarez-MSFT, thx for the suggestion.

But I don't quite understand the issue.
The background is that we have a scenario, the customers used to use "ContainerProbeSettings.disableProbe" and now they want to turn around to use the custom prove feature, right?

No matter what deployment tools they are using, they have to edit all their apps with "ContainerProbeSettings.disableProbe", at least they need to add properties like liveness probes/readiness probes, is that right?

So they just need to delete the "ContainerProbeSettings.disableProbe" property at the same time, no additional redeploy at all. They always have to do one redeploy when switching the feature.

@junior-juarez-MSFT
Copy link
Author

junior-juarez-MSFT commented May 7, 2024

@Caoxuyang

Yes, you are right they have to redeploy everything again if custom probe settings are going to be set.

What we would like is that ContainerProbeSettings.disableProbe can be omitted during the Azure CLI or IaC deployment and be automatically changed to TRUE when custom settings are set and the other way around, as initially ContainerProbeSettings.disableProbe are going to be either TRUE oder FALSE depending on what settings are originally used.

@Caoxuyang
Copy link

@junior-juarez-MSFT I was looped in the email thread. I will address the question in email thread and update the result later back here in github.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants