-
Notifications
You must be signed in to change notification settings - Fork 83
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
feat(tmc): send logs of the user-supplied command to Terramate Cloud. #1182
Conversation
✅ Deploy Preview for terramate-io-docs canceled.
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1182 +/- ##
==========================================
- Coverage 62.75% 62.62% -0.13%
==========================================
Files 102 103 +1
Lines 16395 16579 +184
==========================================
+ Hits 10288 10383 +95
- Misses 5683 5766 +83
- Partials 424 430 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
|
e7d9879
to
c9312fa
Compare
# Reasons for This Change Terramate Cloud supports visualizing the deployment logs in real-time. ## Description of the changes A log syncer was created that proxies stdout/stderr writes using an in-memory pipe. The syncer creates 3 goroutines (1 for stdout, 1 for stderr and 1 for synchronizing logs) that *MUST* be shutdown/terminated after execution of each stack. The syncer supports a customizable `batchSize` and `maxLineSize` but the requirement of the latter will be fixed in a separate PR. Signed-off-by: Tiago Natel <t.nateldemoura@gmail.com>
c8ebe9d
to
9e642fe
Compare
Reasons for This Change
Terramate Cloud supports visualizing the deployment logs in real-time.
Description of the changes
A log syncer was created that proxies stdout/stderr writes using an in-memory pipe. The syncer creates 3 goroutines (1 for stdout, 1 for stderr and 1 for synchronizing logs) that MUST be shutdown/terminated after execution of each stack.
The syncer supports a customizable
batchSize
andmaxLineSize
but the requirement of the latter will be fixed in a separate PR.