@cryptotaxi247 / kubo / commits / 7a9b1716f

daemon: don't print anything when raising file descriptor limit

This isn't an error and 99.999% of users won't care. License: MIT Signed-off-by: Steven Allen <steven@stebalien.com>

Steven Allen committed Jan 7, 2019 at 19:25 UTC 7a9b1716f9a6f84e7611f08a97e9b546bd1ebf92
2 files changed +2 -6
cmd/ipfs/daemon.go
+1 -5
@@ -200,12 +200,8 @@ func daemonFunc(req *cmds.Request, re cmds.ResponseEmitter, env cmds.Environment
200
201 managefd, _ := req.Options[adjustFDLimitKwd].(bool)
202 if managefd {
203 - if changedFds, newFdsLimit, err := utilmain.ManageFdLimit(); err != nil {
203 + if _, _, err := utilmain.ManageFdLimit(); err != nil {
204 log.Errorf("setting file descriptor limit: %s", err)
205 - } else {
206 - if changedFds {
207 - fmt.Printf("Successfully raised file descriptor limit to %d.\n", newFdsLimit)
208 - }
205 }
206 }
207
test/sharness/t0060-daemon.sh
+1 -1
@@ -133,7 +133,7 @@ TEST_ULIMIT_PRESET=1
133 test_launch_ipfs_daemon
134
135 test_expect_success "daemon raised its fd limit" '
136 - grep "raised file descriptor limit to 2048." actual_daemon > /dev/null
136 + grep -v "setting file descriptor limit" actual_daemon > /dev/null
137 '
138
139 test_expect_success "daemon actually can handle 2048 file descriptors" '