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

No UploadToBlobAsync Operations Monitoring correlation #12

Closed
martinabbott opened this issue Dec 21, 2016 · 2 comments
Closed

No UploadToBlobAsync Operations Monitoring correlation #12

martinabbott opened this issue Dec 21, 2016 · 2 comments
Assignees
Labels
area-service Issue points to a problem in the service. question Further information is requested.

Comments

@martinabbott
Copy link

Hi,

When using the UploadToBlobAsync operation the SDK performs three actions that are logged in operational monitoring, e.g.,

{"time":"2016-12-21T07:31:24.1350268Z","operationName":"sasUriRequest","category":"FileUploadOperations","level":"Information","deviceId":"device1"}
{"fileUploadStatusCode":0,"time":"2016-12-21T07:31:24.8878250Z","operationName":"fileUploadToStorage","category":"FileUploadOperations","level":"Information","deviceId":"device1"}
{"time":"2016-12-21T07:31:24.8878250Z","operationName":"completionRequest","category":"FileUploadOperations","level":"Information","deviceId":"device1"}

The problem with this, is that if there are a lot of devices all uploading files, it can become nearly impossible to correlate events that relate to a single invocation of the method. I want to do this to build a dashboard so I can see when things are failing, but would currently have to group by deviceId and some vague bracket of time to determine the number of events I have, or have something like a count of each event type against a deviceId over time.

What would be good would be either the filename to be included in the monitoring, or a correlationId, or even better both.

Thanks,

Martin

@tameraw tameraw added enhancement New feature or request. area-service Issue points to a problem in the service. question Further information is requested. and removed enhancement New feature or request. labels Jan 6, 2017
@nberdy
Copy link
Contributor

nberdy commented Jan 12, 2017

Thanks for the feedback! I've added a backlog item to the service to provide these fields.

Best,
Nicole

@tameraw
Copy link
Contributor

tameraw commented Jan 12, 2017

@martinabbott - Closing this issue as it pertains to the service and not with the SDK. Thanks.

@tameraw tameraw closed this as completed Jan 12, 2017
timtay-microsoft pushed a commit to timtay-microsoft/azure-iot-sdk-csharp that referenced this issue Dec 9, 2019
feat(digitaltwin-device): add UT for DigitalTwinInterfaceClient
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-service Issue points to a problem in the service. question Further information is requested.
Projects
None yet
Development

No branches or pull requests

4 participants