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

26 tests tracking memory usage the slow pipeline #89

Merged
merged 22 commits into from
May 13, 2024
Merged

26 tests tracking memory usage the slow pipeline #89

merged 22 commits into from
May 13, 2024

Conversation

j-i-l
Copy link
Collaborator

@j-i-l j-i-l commented May 12, 2024

Closes #26

  • startied with slow check using gct and valgrind
  • syntax fix in setenv
  • bug fix; forgot interpreter
  • running with gct only
  • typo in Rshell
  • build with flags; adding single file test
  • fixing formatting
  • docker is root anyways
  • adding TODO for feature: automate tagging an releasing #81

@j-i-l j-i-l linked an issue May 12, 2024 that may be closed by this pull request
Tests tracking memory usage (the slow pipeline) #26
Closed
@j-i-l j-i-l self-assigned this May 12, 2024
@j-i-l j-i-l force-pushed the 26-tests-tracking-memory-usage-the-slow-pipeline branch from 469933c to fdd7845 Compare May 12, 2024 17:32
@j-i-l j-i-l force-pushed the 26-tests-tracking-memory-usage-the-slow-pipeline branch from fdd7845 to 0f7bd8c Compare May 12, 2024 17:36
@j-i-l
Copy link
Collaborator Author

j-i-l commented May 12, 2024

Ok, this is looking good.

What remains to be done is to properly handle the outcome of the memory tests

@j-i-l j-i-l changed the base branch from main to streamline-release-process May 12, 2024 18:13
@j-i-l
Copy link
Collaborator Author

j-i-l commented May 12, 2024

Going to merge this into the branch of #81 in order to introduce the new worflow structure at onec.

@j-i-l j-i-l mentioned this pull request May 12, 2024
Merged
26 tasks
@j-i-l j-i-l force-pushed the streamline-release-process branch 10 times, most recently from 507dca8 to ab54319 Compare May 12, 2024 20:57
@j-i-l
Copy link
Collaborator Author

j-i-l commented May 12, 2024

Note: test-simulateabn runs for over 3h already. All others terminated rather quickly

@j-i-l j-i-l force-pushed the streamline-release-process branch from 434e8da to 330ce2c Compare May 12, 2024 21:05
@j-i-l
Copy link
Collaborator Author

j-i-l commented May 12, 2024

Note: test-simulateabn runs for over 3h already. All others terminated rather quickly

Almost 4h now...
We might want to think of splitting up this test

@j-i-l
Copy link
Collaborator Author

j-i-l commented May 13, 2024

Finally, it took 4:30 for this single test to run

Note: test-simulateabn runs for over 3h already. All others terminated rather quickly

Almost 4h now... We might want to think of splitting up this test

@j-i-l
Copy link
Collaborator Author

j-i-l commented May 13, 2024

This can go to the branch of #81

@j-i-l j-i-l merged commit 8a8b2df into streamline-release-process May 13, 2024
4 of 28 checks passed
@matteodelucchi matteodelucchi deleted the 26-tests-tracking-memory-usage-the-slow-pipeline branch May 22, 2024 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@j-i-l j-i-l

Labels
memory::check
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Tests tracking memory usage (the slow pipeline)
1 participant
@j-i-l