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
feat: gRPC tracking #3015
feat: gRPC tracking #3015
Conversation
Example Usage: (To reuse the container based on the same image) | ||
|
||
.. code-block:: python | ||
|
||
@cached_contextmanager("{docker_image.id}") | ||
def start_docker_container_from_image(docker_image, timeout=60): | ||
container = ... | ||
yield container | ||
container.stop() |
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.
update documentation how to use this decorator.
if get_debug_mode(): | ||
logger.error( | ||
f"Tracking Error: {err}", stack_info=True, stacklevel=3 | ||
) | ||
else: | ||
logger.info(f"Tracking Error: {err}") |
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.
Showing stack when BENTOML_DEBUG=True
01bc401
to
91493b2
Compare
Codecov Report
@@ Coverage Diff @@
## main #3015 +/- ##
==========================================
+ Coverage 62.29% 62.33% +0.04%
==========================================
Files 131 132 +1
Lines 11321 11522 +201
Branches 1873 1983 +110
==========================================
+ Hits 7052 7182 +130
- Misses 3842 3905 +63
- Partials 427 435 +8
|
0c9481a
to
23f7492
Compare
@@ -56,6 +56,7 @@ def prometheus_client(self): | |||
|
|||
# step 2: | |||
import prometheus_client | |||
import prometheus_client.parser |
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.
This changes is to make sure we don't import prometheus_client.parser again.
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
4db6f76
to
9857585
Compare
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Signed-off-by: Aaron Pham 29749331+aarnphm@users.noreply.github.com
What does this PR address?
Fixes #2991
Depends on #2984
Before submitting:
guide on how to create a pull request.
make format
andmake lint
script have passed (instructions)?those accordingly? Here are documentation guidelines and tips on writting docs.