-
Notifications
You must be signed in to change notification settings - Fork 360
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
WX-927 GCP Batch: LogsPolicy.PATH now streams the logs to GCS #7529
base: develop
Are you sure you want to change the base?
Conversation
Instead of pushing the logs file after the job completes, the logs are now streamed to GCS.
Hi Alex, in our sync meetings we discussed an aysnc "sidecar" type solution for log file syncing, much like PAPI v2 has now. We were concerned that mounting a GCS filesystem that includes potentially large task files might consume a lot of bandwidth as these files are written. Streaming of files other than task logs are not necessary for our use cases. |
The mounted GCS bucket is only for logs. The outputs still go through delocalization. There is not an option to do a sidecar within GCP Batch. Google would have to enable it within the product. We have asked and the response was this to do it this way. |
OK thank you for the clarification, we'll try it this way then. 😄 |
Unfortunately this approach seems to have some issues:
In the course of evaluating this I discovered that the |
Sketch of stdout/stderr-like sidecar-ish "streaming" solution here, definitely not fully baked yet. |
Thanks for sharing, do let me know if you need anything from me. |
Thanks Alex. I just realized today that the approach in my PR does not capture output from any of the setup / localization / delocalization runnables. I'm not sure yet how much this matters, but it may turn out that the logging story isn't completely resolved yet. 🙂 |
Description
Instead of pushing the logs file after the job completes, the logs are now streamed to GCS.
This is how it works:
Notes:
tail
the GCS file but runningcat
continuously display the new logs.Release Notes Confirmation
CHANGELOG.md
CHANGELOG.md
in this PRCHANGELOG.md
because it doesn't impact community usersTerra Release Notes