Skip to content
This repository has been archived by the owner on Jul 23, 2020. It is now read-only.

POC - Handling API Logic for logging of UI Incidents #998

Closed
4 tasks
arunkumars08 opened this issue Sep 21, 2017 · 8 comments
Closed
4 tasks

POC - Handling API Logic for logging of UI Incidents #998

arunkumars08 opened this issue Sep 21, 2017 · 8 comments

Comments

@arunkumars08
Copy link
Collaborator

Description

This sub task will store the information received from the UI layer in regards to error logging and store in some space.

Why

This could, in a way, help us identify the issues if any from the frontend that are gone unnoticed.

How

As we get the model from the UI layer, to begin with we can store that in a flat file just to track the progress and eventually move it to S3 buckets for better usage.

  • Create an exposable API [Get and Post methods]
  • Write to file the model that is received
  • If things are so far great, write to S3 buckets.
  • Also, provide a mechanism to read the written logs regardless of where it is stored. [ GET call ]
@joshuawilson
Copy link
Member

This is a duplicate of #863
If you have time to work on it let me know and we can plan it.

@arunkumars08
Copy link
Collaborator Author

@joshuawilson Sure Joshua. I will update you on this. We have taken this as a stretch goal. Thanks

@samuzzal-choudhury
Copy link

@arunkumars08 @joshuawilson This work was scoped for logging only the recommender UI exceptions/incidents ATM

@joshuawilson
Copy link
Member

This type of thing should be done at a broader level. All of the UI needs it. If it is done for one, all benefit.

If you don't have the time, then wait. If you do have the time then we should work through the architecture and concerns.

@samuzzal-choudhury
Copy link

@joshuawilson I was not aware that the platform team is already taking a lead on this. I totally agree to your point that the work should benefit all.

@GeorgeActon @krishnapaparaju, I added the user story (as a stretch goal) #754 address logging of recommender UI incidents in the server and was scoped accordingly.

But, apparently we are looking at a generic solution and that will require architectural discussions with a broader audience and @joshuawilson had already opened an issue #863 for that.

I moving the user story and it's related tasks (#754,
#997, #998) back to the "Analytics Backlog" as we will not have the bandwidth to cover those at their broader scope in this sprint.

CC: @arunkumars08 @invincibleJai

@joshuawilson
Copy link
Member

If you do get some bandwidth for this, ping me and we can collaborate on it.

@qodfathr
Copy link
Collaborator

@joshuawilson you called this a dupe of #863. Is that still true? Can this be closed?

@joshuawilson
Copy link
Member

Yes, it's a dup. We have Sentry.io set up and once we get the UI to start logging errors there, you can write up an issue to use it.

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

No branches or pull requests

5 participants