cypress: log stdout/stderr when docker exec fails #9154

kegsay · 2022-08-09T12:15:44Z

Otherwise you cannot debug anything with errors like:

> Command failed: docker exec 134c9a0afd7dadd0b82ce69b4d72d3d6d8ca1b211540d4390a88357b68fa03b9 pg_isready -U postgres

Now we include the stdout/err prior to logging this.

element-web notes: none

This change is marked as an internal change (Task), so will not be included in the changelog.

Otherwise you cannot debug anything with errors like: ``` > Command failed: docker exec 134c9a0afd7dadd0b82ce69b4d72d3d6d8ca1b211540d4390a88357b68fa03b9 pg_isready -U postgres ``` Now we include the stdout/err prior to logging this.

weeman1337

LGTM 👍

Otherwise you cannot debug anything with errors like: ``` > Command failed: docker exec 134c9a0afd7dadd0b82ce69b4d72d3d6d8ca1b211540d4390a88357b68fa03b9 pg_isready -U postgres ``` Now we include the stdout/err prior to logging this.

cypress: log stdout/stderr when docker exec fails

0417157

Otherwise you cannot debug anything with errors like: ``` > Command failed: docker exec 134c9a0afd7dadd0b82ce69b4d72d3d6d8ca1b211540d4390a88357b68fa03b9 pg_isready -U postgres ``` Now we include the stdout/err prior to logging this.

kegsay requested a review from a team as a code owner August 9, 2022 12:15

kegsay requested review from kerryarchibald, weeman1337 and duxovni August 9, 2022 12:15

github-actions bot deployed to Netlify August 9, 2022 12:20 View deployment

weeman1337 added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Aug 9, 2022

weeman1337 approved these changes Aug 9, 2022

View reviewed changes

kegsay merged commit 147ec49 into develop Aug 9, 2022

kegsay deleted the kegan/stdout-docker-exec branch August 9, 2022 14:22

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

cypress: log stdout/stderr when docker exec fails #9154

cypress: log stdout/stderr when docker exec fails #9154

kegsay commented Aug 9, 2022 •

edited by github-actions bot

Loading

weeman1337 left a comment

cypress: log stdout/stderr when docker exec fails #9154

cypress: log stdout/stderr when docker exec fails #9154

Conversation

kegsay commented Aug 9, 2022 • edited by github-actions bot Loading

weeman1337 left a comment

Choose a reason for hiding this comment

kegsay commented Aug 9, 2022 •

edited by github-actions bot

Loading