Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Removes old, empty directories from datatype root directories (#99)
* Removes old, empty directories Pusher did not remove old directories. Every extant directory creates an inotify watch in pusher. Over time and/or for experiments (e.g., wehe) that create a large amount of directories, this can create a very large number of useless watches, which eat up memory, and eventually may run up against fs.inotify.max_user_watches, causing other containers on the system to fail. This commit also fixes an apparent variable name bug in which the value for maxFileAge was put into a variable named minFileAge. * Removes old, empty directories when walking dir Previously, directories were not touched. On long-running containers directories would begin to pile up causing a large amount of useless pusher inotify directories watches. * Uses Go v1.18 to build Docker image, and Alpine 3.15 * Updates Go modules * Reverts prometheus/client_golang to v1.11.0 I was running into this with newer versions: prometheus/prometheus#10574 * Updates Travis to use Go v1.18 * Reverts Go to v1.17 (from v1.18) The travis build was failing with something like this: mattn/goveralls#207 ... and the Docker build was failing with this error: Step 8/10 : COPY --from=build /go/bin/pusher / COPY failed: stat go/bin/pusher: file does not exist * Adds a clarifying comment about removing dirs * Simplifies dir removal logic, removes branching
- Loading branch information
Showing
6 changed files
with
573 additions
and
103 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
language: go | ||
|
||
go: | ||
- "1.13" | ||
- "1.17" | ||
|
||
services: | ||
- docker | ||
|
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
Oops, something went wrong.