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.

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

Testing K2 Integration #820

Draft
wants to merge 705 commits into
base: develop-k2-integration
Choose a base branch
from

Conversation

umaannamalai
Copy link
Contributor

Before contributing, please read our contributing guidelines and code of conduct.

Overview

Describe the changes present in the pull request

Related Github Issue

Include a link to the related GitHub issue, if applicable

Testing

The agent includes a suite of tests which should be used to
verify your changes don't break existing functionality. These tests will run with
Github Actions when a pull request is made. More details on running the tests locally can be found
here,
For most contributions it is strongly recommended to add additional tests which
exercise your changes.

@github-actions
Copy link

github-actions bot commented Jun 1, 2023

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Elapsed time
⚠️ MARKDOWN markdownlint 2 0 1 0.52s
✅ MARKDOWN markdown-link-check 2 0 1.6s
❌ PYTHON bandit 163 11 23.87s
✅ PYTHON black 664 243 0 40.88s
❌ PYTHON flake8 664 360 15.25s
✅ PYTHON isort 664 293 0 1.6s
❌ PYTHON pylint 664 1023 120.95s
✅ RST rstcheck 2 0 8.75s
✅ RST rstfmt 2 1 0 3.05s
❌ RST rst-lint 2 1 1.05s
✅ YAML prettier 7 1 0 1.26s
✅ YAML v8r 7 0 10.21s
✅ YAML yamllint 7 0 1.23s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@codecov-commenter
Copy link

Codecov Report

Merging #820 (3d44b8c) into develop-k2-integration (c2d4629) will decrease coverage by 3.17%.
The diff coverage is 85.71%.

❗ Current head 3d44b8c differs from pull request most recent head fe17e36. Consider uploading reports for the commit fe17e36 to get more accurate results

@@                    Coverage Diff                     @@
##           develop-k2-integration     #820      +/-   ##
==========================================================
- Coverage                   76.40%   73.24%   -3.17%     
==========================================================
  Files                         198      198              
  Lines                       19327    19369      +42     
  Branches                     3307     3308       +1     
==========================================================
- Hits                        14766    14186     -580     
- Misses                       3716     4268     +552     
- Partials                      845      915      +70     
Impacted Files Coverage Δ
newrelic/config.py 55.54% <64.70%> (+0.11%) ⬆️
newrelic/core/config.py 94.56% <100.00%> (+0.05%) ⬆️

... and 57 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mergify mergify bot added the tests-failing label Jun 1, 2023
@stale
Copy link

stale bot commented Aug 12, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 12, 2023
hmstepanek and others added 25 commits March 27, 2024 15:12
Report only raw content dictionary for anthropic claude.
* Use six from packages

* Fixup: no pre-commit hook

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
…relic-python-agent into memory-usage-sampler-config
Add config setting to toggle memory usage runtime metrics.
* Update aiohttp in tests

* Remove unnecessary and buggy tests

* [Mega-Linter] Apply linters fixes

* Bump tests

---------

Co-authored-by: TimPansino <TimPansino@users.noreply.github.com>
* Format celery tests

* Add celery test for nested tasks

* Update pytest

* Add DT to celery

* Add distributed tracing testing for celery

* Pin pytest for 3.7

* Formatting

* Fix issue on older celery versions

* Add skip for Py2

* Fix skipif mark

* Linting

* Update newrelic/hooks/application_celery.py

Co-authored-by: Hannah Stepanek <hstepanek@newrelic.com>

* Minor updates

* Remove unused pytest

* [Mega-Linter] Apply linters fixes

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Hannah Stepanek <hstepanek@newrelic.com>
Co-authored-by: hmstepanek <hmstepanek@users.noreply.github.com>
* move validate_custom_event into new dir

* Add validator shift to mlmodels

* Remove validators from fixtures.py

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Update artifact actions to newer versions

* Fix missing coverage dependency on firestore

* Add missing codecov token
* Update and pin all github actions to hashes

* Linting
* Rename files for better conventions

* Move worker config to conftest

* Update existing tests

* Add all celery task running methods to tests

* Patch celery name extraction crash

* Add correct metrics for all tests

* Fix map and starmap validators

* Fully corrected celery tests

* Remove celery tracer instrumentation

* Fix issue with wrappers not being detected by celery

* Add additional logic to celery task naming for mapping tasks

* Patch distributed tracing for new instrumentation point

* Expand celery testing matrix to all supported minor versions

* Remove unused code, and reorganize

* Linting

* Add another comment

* Remove all python 2 testing requirements
umaannamalai and others added 29 commits October 1, 2024 22:54
* Add Python 3.13 tests to tox

* Fix memcache tests

* Fix agent unittests under developer mode

* Add wheels for 3.13

* Replace PyEval_CallObject with PyObject_Call in C extensions

* Remove Python 2 specific code for C extensions

* Update includes for C extensions for Py 3.13 compatibility

* Add support for setuptools_scm v7 and v8

* Address feedback

* Adding additional tox envs that are now supported
Instrument SQLiteVec vectorstore in LangChain
LLM Custom Attributes Context Manager API
* Pin gRPC version supported in Python 3.7

* Update tox.ini

Co-authored-by: Uma Annamalai <uannamalai@newrelic.com>

* Clean up tox runners

Co-authored-by: Uma Annamalai <umaannamalai@users.noreply.github.com>
Co-authored-by: Timothy Pansino <TimPansino@users.noreply.github.com>
Co-authored-by: Hannah Stepanek <hmstepanek@users.noreply.github.com>

---------

Co-authored-by: Uma Annamalai <uannamalai@newrelic.com>
Co-authored-by: Uma Annamalai <umaannamalai@users.noreply.github.com>
Co-authored-by: Timothy Pansino <TimPansino@users.noreply.github.com>
Co-authored-by: Hannah Stepanek <hmstepanek@users.noreply.github.com>
* Initial commit for azure sidecar

* Merge changes from main

* Change Azure init container env var name

* Pin gRPC version supported in Python 3.7

* Update tox.ini

Co-authored-by: Uma Annamalai <uannamalai@newrelic.com>

* Clean up tox runners

Co-authored-by: Uma Annamalai <umaannamalai@users.noreply.github.com>
Co-authored-by: Timothy Pansino <TimPansino@users.noreply.github.com>
Co-authored-by: Hannah Stepanek <hmstepanek@users.noreply.github.com>

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Uma Annamalai <uannamalai@newrelic.com>
Co-authored-by: Uma Annamalai <umaannamalai@users.noreply.github.com>
Co-authored-by: Timothy Pansino <TimPansino@users.noreply.github.com>
Co-authored-by: Hannah Stepanek <hmstepanek@users.noreply.github.com>
Co-authored-by: Timothy Pansino <11214426+TimPansino@users.noreply.github.com>
@stale stale bot removed the stale label Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.