-
Notifications
You must be signed in to change notification settings - Fork 332
feat: add openobserve backend #839
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
base: main
Are you sure you want to change the base?
Conversation
juju4
commented
Mar 30, 2025
- https://openobserve.ai/docs/
- depends on ongoing work on unofficial API and pending merge https://github.com/juju4/python-openobserve/tree/devel-all, https://github.com/JustinGuese/python-openobserve/pulls/juju4
- tests need more review but added some in underlying module https://github.com/juju4/python-openobserve/tree/devel-all/tests
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
most lint stuff seems unrelated to PR. pytest part as said still WIP. |
pytest fixed but a new external error on tests/test_pkg_imports.py::test_conda_reqs |
@juju4 This test is just designed to keep the requirements/setup packages aligned so it's saying that there is something in requirements/setup that's not reflected in the /conda requirements files. |
seems conda package can't be a commit. hope to have a pypi release next week-end |
Current pylint errors not related to PR
done that for now. 3.8 fails with |
Only python 3.8 ci is failing and seems unrelated to PR
|