-
Notifications
You must be signed in to change notification settings - Fork 71
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix/cache maxsize + tag override for artifacts/tasks + proxy issue (#327
) * Only take into account first forwarded host/proto * Fix issue with tag override for artifacts/tasks * Fix issue with cache exceeding set size There were two main issues with the current implementation: - the size of files was considered to be the number of bytes which is not the best for large number of small files which can happen when the cache is dealing with artifacts (lots of small ones). The fix is to reason based on the block size. - there was a possibility where a file could be marked as deletable and would then be re-created only to be then deleted again. This fix addresses this situation. * Added an expiration to the pg pools to recycle connections * no exception, so have to use echo (based on code convention here) * actually convention seems to be "warn(None, ...)" Co-authored-by: Jackie Tung <jackie@outerbounds.co>
- Loading branch information
1 parent
94855d3
commit 9c1d587
Showing
5 changed files
with
68 additions
and
36 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters