Skip to content
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

Change rawjson progress to one line per JSON record? #4769

Closed
felixfontein opened this issue Mar 17, 2024 · 2 comments · Fixed by #4770
Closed

Change rawjson progress to one line per JSON record? #4769

felixfontein opened this issue Mar 17, 2024 · 2 comments · Fixed by #4770

Comments

@felixfontein
Copy link
Contributor

This would make it easier to parse the raw JSON output, since you can simply split by newlines and feed every line into a JSON parser.

Right now the JSON output comes "nicely" formatted for human consumption, but harder to parse automatically (this was added in #4113 (comment)).

My main intention is to make this easier to parse when appearing in docker compose output (see docker/compose#11478). Since docker compose stdout output can also contain stdout from other sources than buildkit, it's a lot easier to parse stdout when every JSON record is in its own line (and just a single line).

@ndeloof
Copy link

ndeloof commented Mar 17, 2024

this would align with docker commands which produces newline-delimited JSON stream

@felixfontein
Copy link
Contributor Author

I created #4770 to change this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants