Skip to content

refactor(server): expand artifact build log with stderr#303

Merged
alexey-igrychev merged 5 commits intomainfrom
chore/trdl/expand-stderr
Feb 27, 2025
Merged

refactor(server): expand artifact build log with stderr#303
alexey-igrychev merged 5 commits intomainfrom
chore/trdl/expand-stderr

Conversation

@Fral738
Copy link
Copy Markdown
Collaborator

@Fral738 Fral738 commented Feb 21, 2025

Error log example

unable to build release artifacts: can't build artifacts: error executing command: ERROR: permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get "http://%2Fvar%2Frun%2Fdocker.sock/_ping": dial unix /var/run/docker.sock: connect: permission denied exit status 1"

Error log example

unable to build release artifacts: can't build artifacts: error executing command: ERROR: failed to solve: process "/bin/sh -c ls -la /home/asdadada/aasgasgasg && chmod +x build.sh && ./build.sh v0.0.9 && cp -a release-build/v0.0.9/* /result" did not complete successfully: exit code: 1 

@Fral738 Fral738 requested a review from iapershin February 21, 2025 14:18
Copy link
Copy Markdown
Contributor

@iapershin iapershin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to use pipes here instead of copying all stderr

@Fral738 Fral738 marked this pull request as draft February 26, 2025 08:17
@Fral738 Fral738 force-pushed the chore/trdl/expand-stderr branch 2 times, most recently from 14a994b to 6c4b041 Compare February 26, 2025 08:55
Signed-off-by: Evgeniy Frolov <evgeniy.frolov@flant.com>
@Fral738 Fral738 force-pushed the chore/trdl/expand-stderr branch from 6c4b041 to a13e44e Compare February 26, 2025 08:56
@Fral738 Fral738 marked this pull request as ready for review February 26, 2025 08:56
@Fral738 Fral738 requested a review from iapershin February 26, 2025 08:56
Fral738 and others added 3 commits February 27, 2025 13:18
…rr handling

Co-authored-by: Yaroslav Pershin <62902094+iapershin@users.noreply.github.com>
Signed-off-by: Evgeniy Frolov <frolov1998kc@mail.ru>
… error message format

Co-authored-by: Yaroslav Pershin <62902094+iapershin@users.noreply.github.com>
Signed-off-by: Evgeniy Frolov <frolov1998kc@mail.ru>
Signed-off-by: Evgeniy Frolov <evgeniy.frolov@flant.com>
Signed-off-by: Evgeniy Frolov <evgeniy.frolov@flant.com>
@alexey-igrychev alexey-igrychev changed the title chore(server): expand artifact build log with stderr refactor(server): expand artifact build log with stderr Feb 27, 2025
@alexey-igrychev alexey-igrychev merged commit 91a5bf4 into main Feb 27, 2025
11 of 12 checks passed
@alexey-igrychev alexey-igrychev deleted the chore/trdl/expand-stderr branch February 27, 2025 14:47
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 this pull request may close these issues.

3 participants