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

Ci/Improve scheduled tasks #1028

Merged
merged 7 commits into from
Dec 12, 2023
Merged

Conversation

Luni-4
Copy link
Collaborator

@Luni-4 Luni-4 commented Dec 1, 2023

Pull Request Template

Checklist

  • Confirmed that run-checks all script has been executed.
  • Made sure the book is up to date with changes in this PR.

Related Issues/PRs

Provide links to relevant issues and dependent PRs.

Changes

This PR:

  • Runs scheduled tasks at different days and hours in order to avoid loading too much CI
  • Comment more the jobs
  • Move valgrind job in a different file that does not run anymore when a new release is pushed, but only weekly
  • Add more memory detectors

Testing

Describe how these changes have been tested.

@Luni-4 Luni-4 marked this pull request as draft December 1, 2023 15:02
@Luni-4 Luni-4 marked this pull request as ready for review December 1, 2023 16:04
Copy link
Contributor

@dcvz dcvz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Comment on lines +144 to +147
sudo apt-get update -y -qq
sudo add-apt-repository ppa:kisak/kisak-mesa -y
sudo apt-get update
sudo apt install -y libegl1-mesa libgl1-mesa-dri libxcb-xfixes0-dev mesa-vulkan-drivers
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is repeating itself a bit everywhere, is there a way to abstract that with an action template?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nathanielsimard nathanielsimard merged commit 1a5f252 into tracel-ai:main Dec 12, 2023
9 checks passed
@Luni-4 Luni-4 deleted the scheduled-opt branch December 12, 2023 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants