Skip to content

Conversation

@joejulian
Copy link
Contributor

@joejulian joejulian commented May 26, 2023

see https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#grouping-log-lines

What this PR does / why we need it:
The logging between tests is incredibly verbose. Github has a way of collapsing logs into groups. This takes advantage of that feature.

Special notes for your reviewer:
See https://github.com/redpanda-data/helm-charts/actions/runs/5087615217/jobs/9143147560?pr=521#step:15:2372 for an example.

@joejulian joejulian force-pushed the github_logging_groups branch from 5b88fdd to dada55f Compare May 26, 2023 05:58
@joejulian joejulian force-pushed the github_logging_groups branch from dada55f to 24acd59 Compare May 26, 2023 06:01
@pull-request-size pull-request-size bot added size/L and removed size/M labels May 26, 2023
Copy link

@RafalKorepta RafalKorepta left a comment

Choose a reason for hiding this comment

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

Love it :)

@davidkarlsen davidkarlsen requested a review from cpanato May 27, 2023 09:02
Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

lgtm
thanks

@cpanato cpanato merged commit 373aa72 into helm:main May 27, 2023
@joejulian joejulian deleted the github_logging_groups branch May 31, 2023 23:36
marcofranssen added a commit to spiffe/helm-charts that referenced this pull request Jul 11, 2023
helm/chart-testing#556
Signed-off-by: Marco Franssen <marco.franssen@gmail.com>
marcofranssen added a commit to spiffe/helm-charts that referenced this pull request Jul 13, 2023
helm/chart-testing#556
Signed-off-by: Marco Franssen <marco.franssen@gmail.com>
marcofranssen added a commit to spiffe/helm-charts that referenced this pull request Jul 19, 2023
helm/chart-testing#556
Signed-off-by: Marco Franssen <marco.franssen@gmail.com>
marcofranssen added a commit to spiffe/helm-charts that referenced this pull request Jul 19, 2023
helm/chart-testing#556
Signed-off-by: Marco Franssen <marco.franssen@gmail.com>
marcofranssen added a commit to spiffe/helm-charts that referenced this pull request Jul 19, 2023
helm/chart-testing#556
Signed-off-by: Marco Franssen <marco.franssen@gmail.com>
marcofranssen added a commit to spiffe/helm-charts that referenced this pull request Jul 19, 2023
helm/chart-testing#556
Signed-off-by: Marco Franssen <marco.franssen@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants