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

[BUG] Not able to see any Logs from SDK Storage Blob #5130

Closed
3 tasks done
vmaheshw opened this issue Aug 27, 2019 · 10 comments
Closed
3 tasks done

[BUG] Not able to see any Logs from SDK Storage Blob #5130

vmaheshw opened this issue Aug 27, 2019 · 10 comments
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved. Storage Storage Service (Queues, Blobs, Files)

Comments

@vmaheshw
Copy link

vmaheshw commented Aug 27, 2019

Describe the bug
I want to see the logs for http request, retry number and http response with total time (any delay). I used the following api, but I'm not able see any logs.
BlobServiceClient storageClient = new BlobServiceClientBuilder().endpoint(endpoint).credential(credential).httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS).buildClient();

Exception or Stack Trace

To Reproduce
Current git Master and add this line in BasicExamples.java
Code Snippet
Add the code snippet that causes the issue.

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Setup (please complete the following information):

  • OS: [e.g. iOS] iOS
  • IDE : [e.g. IntelliJ] Intellij
  • Version of the Library used: Latest git

Additional context
Add any other context about the problem here.

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

  • Bug Description Added
  • Repro Steps Added
  • Setup information Added
@rickle-msft
Copy link
Contributor

An added piece of information that @vmaheshw sent me separately:
"In terms of debuggability, Warning for minDurationToLogSlowRequest is really useful with V10.5. It helps us in finding the Network issue or the APP issue."

This would be an update to the logging policy in azure core. @alzimmermsft could you give some insight on whether it would be possible to add this?

@rickle-msft rickle-msft added Azure.Core azure-core customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved. labels Aug 27, 2019
@rickle-msft
Copy link
Contributor

@JonathanGiles could you also comment on whether we'd be able to bring forward some of the v11 functionality like logging slow requests as a warning?

@JonathanGiles
Copy link
Member

Definitely. Open to PRs to enable this.

@rickle-msft rickle-msft added Storage Storage Service (Queues, Blobs, Files) and removed Azure.Core azure-core labels Aug 29, 2019
@vmaheshw
Copy link
Author

vmaheshw commented Sep 3, 2019

System.setProperty(BaseConfigurations.AZURE_LOG_LEVEL, "1");

@rickle-msft This is how I'm able to see the logs. Is this the only way?

@rickle-msft
Copy link
Contributor

That is one of the ways currently. @alzimmermsft suggested that there is one other option and can hopefully share it here. I also gather that there are plans to revamp that interface. Logging comes from the core SDK instead of Storage specifically, so I'll have to rely on Alan for more details.

@alzimmermsft
Copy link
Member

@vmaheshw There are two ways to set the logging level, either setting a property or using an environment variable. Both will check for BaseConfigurations.AZURE_LOG_LEVEL and the property takes precedence over the environment variable. Is the concern around this on the value that needs to be set or the default logging level?

@kurtzeborn
Copy link
Member

Logs can be enabled via the two methods mentioned above. @vmaheshw please re-open this issue if you have further needs or requests regarding this.

@ghost
Copy link

ghost commented Sep 11, 2019

Thanks for working with Microsoft on GitHub! Tell us how you feel about your experience using the reactions on this comment.

@vmaheshw
Copy link
Author

vmaheshw commented Oct 9, 2019

@rickle-msft : Any update on the feature to got WARN for slow requests and customize the timer as discussed above. This was present in V10.5

@rickle-msft
Copy link
Contributor

Logging is handled in Azure Core, so I can't say for certain, but from what I'm seeing, it looks like it'll be included in preview 5 (next month) as the goal for preview 4 has been to get as many of the api breaks finished as possible so they are out of the way earlier rather than later

@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

No branches or pull requests

5 participants