autograde / quickfeed

10%

master: 24%

LAST BUILD ON BRANCH fixed-open-file-desc-issue-finally

branch: fixed-open-file-desc-issue-finally

CHANGE BRANCH

x

Reset

• fixed-open-file-desc-issue-finally
• access-control-test
• active-page-fix
• add-golangci-lint
• adds-ismanual-hastests
• assignment-info-url
• criteria-from-json
• database-readme
• database-upgrade
• delete-repo-subscription
• dev-testing
• dev-tooling
• disallow-org-repos-by-students
• docker-improvements
• envoy-config-template
• fix-docker-image-slash-seperator-issue
• fix-go117-test-failure
• fix-missing-test-scores
• fix-testless-submissions
• fix-too-many-open-files
• fix_access_bug
• getsubmissions-simplification
• group_results
• groups
• grpc-based-kit
• hide-after-deadline
• inactive-students
• jsonindent
• lab-sorting-fix
• lab_results_optimization
• local-cert-config
• make-proto-swift
• master
• merged-misc-folders
• npm-update-june-2021
• pending-students-fix
• progressbar
• read-session-secret-from-env
• refs/tags/kit/v0.1.0
• remove-selenium
• replace-deprecated-jsonpb
• replace-gogo
• replace-w-with-v-formatting
• results-layout
• review-score-fix
• review-updates
• safe-production-mode
• shortcuts
• slipdays-issue-fix
• streaming-submissions
• submission-order-fix
• submission_comments
• teacher-access
• update-auth-and-api
• update-score
• username_fix
• webhooks-localhost
• zero-score-issue
• zero-weight-fix

Build # 2586

Build Type

push

travis-ci

Committed by meling

Commit Message

another attempt at fixing the too many open file descriptors issue

This commit fixes #319 (for real this time).
Now we create only one docker client when we start quickfeed,
instead of creating a separate docker client instance for each
job we want to run. This should hopefully save even more resources
and maybe some execution time for tests.

If this does not work correctly for some reason, we can revert to
the previous version, and instead make sure we close the client
between each job run. But that seems wasteful, and hence I wanted
to reuse the client across jobs. The way I understand the client,
it can still create, stop, get logs from and remove several
containers; that is, we shouldn't need a separate client for doing
these tasks.

Run Details

19 of 19 new or added lines in 1 file covered. (100.0%)

1716 of 18023 relevant lines covered (9.52%)

1.31 hits per line