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

NH-35261 NH-38130 liboboe 12.2.0 #132

Merged
merged 6 commits into from
Apr 20, 2023
Merged

Conversation

tammy-baylis-swi
Copy link
Contributor

@tammy-baylis-swi tammy-baylis-swi commented Apr 13, 2023

Bump to liboboe 12.2.0 to use SWIG memory leak fix (updated and re-uploaded Apr 18). Also updates setup.py so that pip uses up-to-date C++ version for extension and points to the correct header file!

The changelog has been updated for this change, plus others I forgot that are already on this branch.

tox tests pass on this PR.

Test traces:

Test released as TestPyPI 0.9.0.0. Install-from-registry tests pass.

Started Flask/Locust benchmarking at 10:00am today with this branch and this testbed branch. 30 min later and memory usage looking good:

CONTAINER ID   NAME                                                                 CPU %     MEM USAGE / LIMIT    MEM %     NET I/O           BLOCK I/O         PIDS
3ecd1d835052   solarwinds-apm-python-testbed_flask-benchmark-app-high-spans-exc_1   8.11%     57.41MiB / 500MiB    11.48%    8.73MB / 50.5MB   4.57MB / 95.9MB   13
c3eb924cb2cf   solarwinds-apm-python-testbed_flask-benchmark-app-high-spans_1       10.49%    64.24MiB / 500MiB    12.85%    8.84MB / 48.2MB   21.6MB / 69.9MB   12
98e0e4b5b45f   solarwinds-apm-python-testbed_flask-benchmark-app-high-load_1        20.42%    63.77MiB / 500MiB    12.75%    181MB / 133MB     40.1MB / 70.6MB   23
97b652b6856d   solarwinds-apm-python-testbed_flask-benchmark-app-high-load-exc_1    31.77%    49.49MiB / 500MiB    9.90%     174MB / 125MB     1.07MB / 69.9MB   18

@tammy-baylis-swi tammy-baylis-swi changed the title NH-35261 liboboe 12.2.0 NH-35261 NH-38130 liboboe 12.2.0 Apr 20, 2023
@tammy-baylis-swi tammy-baylis-swi marked this pull request as ready for review April 20, 2023 17:31
@tammy-baylis-swi tammy-baylis-swi requested a review from a team April 20, 2023 17:31
@tammy-baylis-swi tammy-baylis-swi merged commit f5b820d into main Apr 20, 2023
@tammy-baylis-swi tammy-baylis-swi deleted the NH-35261-liboboe-12_2_0 branch April 20, 2023 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants