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.

Sign up for GitHub

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

Jump to bottom

Improve observability for CWYD #418

Closed
5 tasks done
adamdougal opened this issue Mar 12, 2024 · 0 comments
Closed
5 tasks done

Improve observability for CWYD #418

adamdougal opened this issue Mar 12, 2024 · 0 comments
Assignees
@adamdougal
Labels
enhancement New feature or request

Comments

@adamdougal
Copy link
Collaborator

adamdougal commented Mar 12, 2024 •
edited
Loading

Motivation

There are many moving parts in this project (Admin app, back end functions, chat app). If something fails, it's not clear where to look. We should document (and consolidate if possible) where to find debug logs and metrics.

How would you feel if this feature request was implemented?

debug

Requirements

  • Document how to debug errors
  • Stretch: Provide a "single pane of glass" view for metrics and logs

Tasks

To be filled in by the engineer picking up the issue
@adamdougal adamdougal added the enhancement New feature or request label Mar 12, 2024
@tanya-borisova tanya-borisova self-assigned this Mar 13, 2024
@superhindupur superhindupur changed the title Document how to debug Improve observability for CWYD Mar 27, 2024
@adamdougal adamdougal self-assigned this Apr 8, 2024
adamdougal added a commit that referenced this issue Apr 8, 2024
@adamdougal
Add more metrics and logs to workbook
0ea8ebf 
- This required templating resource names
- Extracted resources to hidden params to reduce templating effort
- Let me know if you want a link to a working example

Required by #418
@adamdougal adamdougal mentioned this issue Apr 8, 2024
Merged
adamdougal added a commit that referenced this issue Apr 9, 2024
@adamdougal
Add more metrics and logs to workbook
bc9306e 
- This required templating resource names
- Extracted resources to hidden params to reduce templating effort
- Let me know if you want a link to a working example

Required by #418
adamdougal added a commit that referenced this issue Apr 9, 2024
@adamdougal
Add documentation for workbook
701d519 
Required by #418
@adamdougal adamdougal mentioned this issue Apr 9, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Apr 9, 2024
@adamdougal
Add more metrics and logs to workbook (#630)
7af0de9 
* Add more metrics and logs to workbook

- This required templating resource names
- Extracted resources to hidden params to reduce templating effort
- Let me know if you want a link to a working example

Required by #418

* Extract workbook logic into a separate module

* Use resources directly to retrieve names

* Fix main.json after merge
github-merge-queue bot pushed a commit that referenced this issue Apr 9, 2024
@adamdougal @ross-p-smith
Add documentation for workbook (#639)
a81251b 
* Add documentation for workbook

Required by #418

* Remove space

Co-authored-by: Ross Smith <ross-p-smith@users.noreply.github.com>

---------

Co-authored-by: Ross Smith <ross-p-smith@users.noreply.github.com>
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this issue Apr 30, 2024
@adamdougal
Add more metrics and logs to workbook (Azure-Samples#630)
bca1ced 
* Add more metrics and logs to workbook

- This required templating resource names
- Extracted resources to hidden params to reduce templating effort
- Let me know if you want a link to a working example

Required by Azure-Samples#418

* Extract workbook logic into a separate module

* Use resources directly to retrieve names

* Fix main.json after merge
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this issue Apr 30, 2024
@adamdougal @ross-p-smith
Add documentation for workbook (Azure-Samples#639)
b5cbf97 
* Add documentation for workbook

Required by Azure-Samples#418

* Remove space

Co-authored-by: Ross Smith <ross-p-smith@users.noreply.github.com>

---------

Co-authored-by: Ross Smith <ross-p-smith@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adamdougal adamdougal

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@adamdougal @tanya-borisova