-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add Deployment Logging Tutorial #935
Conversation
Codecov Report
@@ Coverage Diff @@
## master #935 +/- ##
=======================================
Coverage 93.32% 93.32%
=======================================
Files 81 81
Lines 5030 5030
=======================================
Hits 4694 4694
Misses 336 336
|
@@ -577,7 +577,7 @@ def check_task_is_cached(self, state: State, inputs: Dict[str, Result]) -> State | |||
state._result = state._result.to_result() | |||
return state | |||
else: | |||
self.logger.debug( | |||
self.logger.warning( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Open to push back on this change; it will always happen eventually, but knowing when it happens seems important so I elevated this log level to "WARNING".
|
||
## An Example | ||
|
||
Let's walk through a basic example. To begin, we will create a dummy webserver on our local machine that we will POST logs to: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀
Thanks for contributing to Prefect!
Please describe your work and make sure your PR:
CHANGELOG.md
(if appropriate)docs/outline.toml
for API reference docs (if appropriate)Note that your PR will not be reviewed unless all three boxes are checked.
What does this PR change?
This PR adds a "Deployment: Logging" tutorial for explaining how to interact with Prefect logs (Part 2 in a series on Deployment related information).
Why is this PR important?
Is functionality really present if it isn't documented well? 🤔