Karl Isenberg comments

Results 288 comments of


Karl Isenberg

“Broken pipe” errors when piping inside asciinema

I see this pretty repeatably when reading from urandom: ``` SUFFIX=$(< /dev/urandom tr -dc '[:digit:]' | head -c4) tr: write error: Broken pipe tr: write error ```

Automatically increase diff verbosity up to some threshold budget

It looks like the `defaultReporter` uses a `formatOptions` struct to control verbosity of Diff output. `formatOptions` even has `VerbosityLevel` and `LimitVerbosity` options built-in. But it doesn't look like there's any...

[WIP] Update Syncing condition to include all errors

/retest

[WIP] Update Syncing condition to include all errors

/retest

[WIP] Update Syncing condition to include all errors

Extracted the duration period cleanup: https://github.com/GoogleContainerTools/kpt-config-sync/pull/273

[WIP] Update Syncing condition to include all errors

Extracted flakey bug fix: https://github.com/GoogleContainerTools/kpt-config-sync/pull/274

[WIP] Update Syncing condition to include all errors

Extracted periodic status updates: https://github.com/GoogleContainerTools/kpt-config-sync/pull/279

[WIP] Update Syncing condition to include all errors

Extracted part of the logic to combine sync errors: https://github.com/GoogleContainerTools/kpt-config-sync/pull/281 The logic to preserve the non-applier errors from the updater will be done in a subsequent PR.

[WIP] Update Syncing condition to include all errors

Extracted thread-safety improvements to https://github.com/GoogleContainerTools/kpt-config-sync/pull/301

[WIP] Update Syncing condition to include all errors

Extracted the hydration e2e changes: https://github.com/GoogleContainerTools/kpt-config-sync/pull/310