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

Parallelize Drone testing #233

Merged
merged 4 commits into from
Nov 1, 2023
Merged

Parallelize Drone testing #233

merged 4 commits into from
Nov 1, 2023

Conversation

dereknola
Copy link
Member

@dereknola dereknola commented Oct 30, 2023 •
edited
Loading

  • Drone now deploys seperate DB testing that runs in parallel.
  • Slightly reduced test load from 5K to 4K, this combined above brings total time down to around ~11 minutes (vs 45-50 before).
  • Converted unit test check to github actions as no building of kine is required. Also we currently lack an unit tests, however several upcoming PRs have added some.
  • Dropped oldest Postgress 11.X, it goes EOL next week.
    Signed-off-by: Derek Nola [email protected]

@dereknola dereknola requested a review from a team as a code owner October 30, 2023 16:51
@dereknola dereknola force-pushed the better_drone branch 3 times, most recently from 39afbae to 09f5998 Compare October 31, 2023 18:36
@dereknola dereknola changed the title [WIP] Better_drone Parallelize Drone testing Oct 31, 2023
@dereknola
Parallelize testing
78980b9 
Signed-off-by: Derek Nola <[email protected]>
@dereknola
Reduce test load
64715ef 
Signed-off-by: Derek Nola <[email protected]>
@dereknola
Convert unit tests to github action
a31fca0 
Signed-off-by: Derek Nola <[email protected]>
@dereknola
Fix histogram type
4855a3d 
Signed-off-by: Derek Nola <[email protected]>
@dereknola dereknola merged commit 13bff3e into k3s-io:master Nov 1, 2023
2 checks passed
@dereknola dereknola mentioned this pull request Nov 1, 2023
Merged
@dereknola dereknola deleted the better_drone branch December 5, 2023 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@briandowns briandowns briandowns approved these changes

@brandond brandond brandond approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dereknola @brandond @briandowns