-
Notifications
You must be signed in to change notification settings - Fork 206
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.
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
Error in CI with faas-cli logs #149
Comments
@LucasRoesler when you have a moment, could you a make suggestion or a recommendation for resolution or investigation? |
I created a branch to investigage and the first run passed, but the second run had this interesting bit $ /usr/local/bin/faas-cli logs figlet --since 15m --follow=false
WARNING! Communication is not secure, please consider using HTTPS. Letsencrypt.org offers free SSL/TLS certificates.
2021-01-17T20:04:54Z 2021/01/17 20:04:54 Version: 0.13.0 SHA: fa93655d90d1518b04e7cfca7d7548d7d133a34e
2021-01-17T20:04:54Z 2021/01/17 20:04:54 Read/write timeout: 1s, 1s. Port: 8080
2021-01-17T20:04:54Z 2021/01/17 20:04:54 Writing lock-file to: /tmp/.lock
2021-01-17T20:04:54Z 2021/01/17 20:04:54 Metrics server. Port: 8081
2021-01-17T20:04:59Z 2021/01/17 20:04:59 Forking fprocess.
2021-01-17T20:04:59Z 2021/01/17 20:04:59 Wrote 162 Bytes - Duration: 0.001486 seconds
2021-01-17T20:05:09Z 2021/01/17 20:05:09 SIGTERM received.. shutting down server in 1s
2021-01-17T20:05:09Z 2021/01/17 20:05:09 Removing lock-file : /tmp/.lock
2021-01-17T20:05:10Z 2021/01/17 20:05:10 No new connections allowed. Exiting in: 1s
$ /usr/local/bin/faas-cli logs figlet --since 15m --follow=false | grep Forking
Makefile:52: recipe for target 'test-e2e' failed
make: *** [test-e2e] Error 1 https://github.com/LucasRoesler/faasd/runs/1717867383?check_suite_focus=true The take away is that the CLI did get logs the first time but when piped to the |
Alternatively, we could use the provider here, since it is testing the same things |
There's an error that's failing the CI when
faas-cli logs
is run.I added a journalctl call to fetch the logs separately, and they are present:
So the issue appears to be related to timing or a race condition.
faas-cli logs figlet
is not working as expected in CI, but does work when used by end-users in normal use.The text was updated successfully, but these errors were encountered: