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.

Sign up for GitHub

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

Jump to bottom

use on demand imports for profilers #1054

Merged
merged 39 commits into from
Nov 4, 2024
Merged

use on demand imports for profilers #1054

merged 39 commits into from
Nov 4, 2024

Conversation

AlyaGomaa
Copy link
Collaborator

No description provided.

@AlyaGomaa
move all the profiling logic from main to managers/profiler_manager.py
70b3536
@AlyaGomaa
profiler_manager.py: delete the memory_profiler_multiproc_test()
7a9d7f9
@AlyaGomaa
profiler_manager.py: import viztracer and memray on demand
3c05fab
@AlyaGomaa
move the logic of stopping the profilers to process_manager's shutdow…
eed5fd7 
…n_gracefully()
@AlyaGomaa
remove vistrace, memray, tqdm, setuptools and wheel from requirements…
07d7cdc 
….txt
@AlyaGomaa
update changelog
bee4a58
@AlyaGomaa
update slips version to 1.1.3
beae32a
@AlyaGomaa
update slips.gif to use 1.1.3
156078e
@AlyaGomaa
CI: add test_database.py and test_symbols_handler to unit tests
300c663
@AlyaGomaa
test_database.py: use a different port for each unit test
e80c38a
@AlyaGomaa
update profiler unit tests
558a081
@AlyaGomaa
CI: cache zeek builds
ccff6dd
@AlyaGomaa
unit-tests: cache apt and pip dependencies
9c57833
@AlyaGomaa
unit-tests: hardcode cache keys to avoid cache miss
4c9e7f1
@AlyaGomaa
unit-tests:use actions/setup-python@v5 for caching python dependencies
91b34de
@AlyaGomaa
add apt_Dependencies file in install/ to avoid repetition of dependen…
a0fa9b2 
…cies in dockerfiles and github runners
@AlyaGomaa
CI: try caching /var/cache/ and /var/lib
42b1b14
@AlyaGomaa
CI: install python dependencies/restore from cache before running uni…
24be296 
…t tests
@AlyaGomaa
CI: cache pip, zeek and apt dependencies in integration tests workflo…
c73f7f6 
…w to avoid building slips for each test
@AlyaGomaa
CI: unit-tests: run unit tests without installing zeek
f35b258
@AlyaGomaa
CI: make installing slips a reusable workflow
1988fcd
@AlyaGomaa
CI: fix a typo in install-slips-dependencies.yml
cf2a88d
@AlyaGomaa
CI: make hardcoded zeek repos optional variables
2a96716
@AlyaGomaa
CI: fix installing python dependencies
4f30789
@AlyaGomaa
CI: integration-tests: fix using reusable workflow
054b7c2
@AlyaGomaa
CI: dependencies: add missing "type" property to workflow_call
f5fbfac
@AlyaGomaa
CI: make sure integration-tests can access Zeek from the cache if it …
4a06d6c 
…was successfully installed by install-dependencies-using-reusable-workflo
@AlyaGomaa
Merge pull request #1056 from stratosphereips/alya/reusable-workflows
24d22d6 
CI: cache zeek builds
@AlyaGomaa
move all the profiling logic from main to managers/profiler_manager.py
1c88582
@AlyaGomaa
profiler_manager.py: delete the memory_profiler_multiproc_test()
79c0816
@AlyaGomaa AlyaGomaa merged commit a7c0c44 into develop Nov 4, 2024
55 checks passed
@AlyaGomaa AlyaGomaa deleted the alya/use-on-demand-imports branch November 4, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
Slips
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@AlyaGomaa