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

Configure logging for the application #637

Closed
3 tasks done
Tracked by #418
cecheta opened this issue Apr 9, 2024 · 0 comments
Closed
3 tasks done
Tracked by #418

Configure logging for the application #637

cecheta opened this issue Apr 9, 2024 · 0 comments
Assignees
@adamdougal
Labels
enhancement New feature or request

Comments

@cecheta
Copy link
Collaborator

cecheta commented Apr 9, 2024 •
edited by adamdougal
Loading

Motivation

Configure logging for the application properly. Configure an application logger, as logs with print() do not seem to get sent to App Insights. Also configure log level, as by default Flask sets the log level to Warning.

https://flask.palletsprojects.com/en/2.3.x/logging/

Requirements

A list of requirements to consider this feature delivered

  • User logger instead of print statements within app
  • Add useful log statements to application
  • Configure log level to INFO

Tasks
@cecheta cecheta added the enhancement New feature or request label Apr 9, 2024
This was referenced Apr 9, 2024
Closed
Closed
Merged
@adamdougal adamdougal self-assigned this Apr 9, 2024
adamdougal added a commit that referenced this issue Apr 9, 2024
@adamdougal
Switch print statements to use logger
560714e 
Required by #637
@adamdougal adamdougal mentioned this issue Apr 9, 2024
Merged
adamdougal added a commit that referenced this issue Apr 9, 2024
@adamdougal
Add additional logs and stop using root logger
a7bf945 
Required by #637
@adamdougal adamdougal mentioned this issue Apr 9, 2024
Merged
adamdougal added a commit that referenced this issue Apr 9, 2024
@adamdougal
Allow the setting of log level by env vars
c6e0cb7 
Required by #637
@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 additional logs and stop using root logger (#641)
c3a4eb5 
* Add additional logs and stop using root logger

Required by #637

* Add test to increase coverage
github-merge-queue bot pushed a commit that referenced this issue Apr 9, 2024
@adamdougal
Switch print statements to use logger (#640)
3e71fe1 
* Switch print statements to use logger

Required by #637

* Fix log format
adamdougal added a commit that referenced this issue Apr 10, 2024
@adamdougal
Allow the setting of log level by env vars
1fccbd5 
Required by #637
adamdougal added a commit that referenced this issue Apr 10, 2024
@adamdougal
Allow the setting of log level by env vars
2b4e933 
Required by #637
adamdougal added a commit that referenced this issue Apr 10, 2024
@adamdougal
Allow the setting of log level by env vars
86a8ff6 
Required by #637
github-merge-queue bot pushed a commit that referenced this issue Apr 11, 2024
@adamdougal
Allow the setting of log level by env vars (#642)
bd598ad 
* Allow the setting of log level by env vars

Required by #637

* Allow the setting of log level by env vars

Required by #637

* Add loglevel to env.sample

* Allow log level to be set via bicep

* Set log level in admin app

* Correctly configure logs in web app

* Ensure log level is set for functions

* Remove duplicated tests

* Fix main.json after merge

* Reduce verbose azure logs
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this issue Apr 30, 2024
@adamdougal
Add additional logs and stop using root logger (Azure-Samples#641)
ba2a2ec 
* Add additional logs and stop using root logger

Required by Azure-Samples#637

* Add test to increase coverage
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this issue Apr 30, 2024
@adamdougal
Switch print statements to use logger (Azure-Samples#640)
7f1a917 
* Switch print statements to use logger

Required by Azure-Samples#637

* Fix log format
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this issue Apr 30, 2024
@adamdougal
Allow the setting of log level by env vars (Azure-Samples#642)
f1c7db6 
* Allow the setting of log level by env vars

Required by Azure-Samples#637

* Allow the setting of log level by env vars

Required by Azure-Samples#637

* Add loglevel to env.sample

* Allow log level to be set via bicep

* Set log level in admin app

* Correctly configure logs in web app

* Ensure log level is set for functions

* Remove duplicated tests

* Fix main.json after merge

* Reduce verbose azure logs
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 @cecheta