-
Notifications
You must be signed in to change notification settings - Fork 447
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
LeaseLostException & WebException on Scale-Out #3212
Comments
7 days, any acknowledgement? :( |
@m1nkeh we will be fixing this in a future release. In the meantime, kindly treat these errors as informational messages. |
+1 for fixing |
guys, is there an ETA on this please... is it in a backlog somewhere? or perhaps already resolved in V2?? cc: @tohling |
I am experiencing this too. Any ETA on a fix for this? cc: @tohling |
Sorry about the delayed response here. To confirm the impact, this is primarily generating noise in the logs, correct? Those exceptions are normal, and handled by the runtime, but currently still tracked/logged. Some improvements were made in 2.0, so it would be interesting to understand if the behavior there is satisfactory. @tohling were there any other open work items related to this? |
Actually, the issue linked to the SO question referenced in the original issue was resolved with PRs linked to Azure/azure-webjobs-sdk#1760 @m1nkeh can you confirm? @garygsscott if you're seeing similar exceptions, can you please open a different issue and share details/stack? Thanks! |
Hi @fabiocav, sorry i keep meaning to reply to this.. no i do not believe anything is resolved - simply follow the repro steps, and you will see! 🙂 I am currently working on a new project, with all the latest up to date SDK levels, Azure Functions 2.0 .Net Core etc.. etc.. and still seeing this behaviour. Well the exceptions are slightly different now, but they are related to the same stuff. Host level exceptions (all of them) are annoying noise in Application Insights.. this particular one, relating to the lease lost stuff is especially frustrating because under heavy load / scaling there are just constant exceptions and everyone on my team just 'knows' they are fine, and OK to ignore. Why not just change these to warnings? The number of times i have to explain why these are OK to ignore with a diagram, and explanation of event hubs, and leases is just a massive time sink. Aside: Would you like me to update the ticket to be functions v2.0 specific ? |
Hi @fabiocav, @m1nkeh - I managed to suppress 90 % of these warning messages in my function by creating a dedicated consumer unit within my event hub and then ensuring that my event hub triggered function only connected to that one particular consumer unit. These messages still appear but with much less frequency after taking this approach. I actually work for the same organisation as @m1nkeh so do not believe a new case is justified and I agree with @m1nkeh that if these are not blocking exceptions and just warnings then shouldn't they be suppressed (to reduce the noise in troubleshooting more than anything else). |
+1 for fixing |
This has been fixed with Azure/azure-webjobs-sdk#2176. The next version of the EventHubs package will contain the fix. |
Investigative information
Please provide the following:
Repro steps
Expected behavior
According to this post: https://stackoverflow.com/questions/51602965/azure-eventhub-function-leaselostexception#51603421
These should be tracked as warnings in the latest host deployment?
Actual behavior
Tracked as exceptions
Known workarounds
None
The text was updated successfully, but these errors were encountered: