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.

Sign up for GitHub

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

Jump to bottom

GHA: Bundle post-test steps into a reusable action #9135

Closed
Tracked by #9127
oleschoenburg opened this issue Apr 13, 2022 · 0 comments · Fixed by #10045
Closed
Tracked by #9127

GHA: Bundle post-test steps into a reusable action #9135

oleschoenburg opened this issue Apr 13, 2022 · 0 comments · Fixed by #10045
Labels
area/test Marks an issue as improving or extending the test coverage of the project kind/toil Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc. version:8.1.0-alpha5 Marks an issue as being completely or in parts released in 8.1.0-alpha5 version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0

Comments

@oleschoenburg
Copy link
Member

oleschoenburg commented Apr 13, 2022 •
edited

With #9129, #9035, #9130 and #9132 implemented, we will have a lot of steps that run after the tests are finished to collect reports and statistics. Since we want to keep our workflows maintainable, this would be a great opportunity to build some re-usable composite actions.

See also: https://dev.to/jameswallis/using-github-composite-actions-to-make-your-workflows-smaller-and-more-reusable-476l
@oleschoenburg oleschoenburg mentioned this issue Apr 13, 2022
Closed
19 tasks
@oleschoenburg oleschoenburg changed the title GHA: Bundle post-test steps into a reusable workflow GHA: Bundle post-test steps into a reusable action Apr 13, 2022
@npepinpe npepinpe added kind/toil Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc. area/test Marks an issue as improving or extending the test coverage of the project labels Apr 14, 2022
@npepinpe npepinpe assigned npepinpe and unassigned npepinpe Apr 25, 2022
@menski menski mentioned this issue Aug 9, 2022
Merged
@menski menski mentioned this issue Aug 16, 2022
Merged
zeebe-bors-camunda bot added a commit that referenced this issue Aug 16, 2022
@zeebe-bors-camunda @menski
merge: #10077
17630e8 
10077: Always upload test artifacts even on failure r=menski a=menski

## Description

Always upload test archive, even if tests fail.

## Related issues

follow-up to #9135 and #10045 



Co-authored-by: Sebastian Menski <sebastian.menski@camunda.com>
@Zelldon Zelldon added the version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0 label Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/test Marks an issue as improving or extending the test coverage of the project kind/toil Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc. version:8.1.0-alpha5 Marks an issue as being completely or in parts released in 8.1.0-alpha5 version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Extract reusable action for test artifact upload camunda/zeebe
5 participants
@npepinpe @menski @oleschoenburg @deepthidevaki @Zelldon