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: Build and test the Go client #9128

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

GHA: Build and test the Go client #9128

lenaschoenburg opened this issue Apr 13, 2022 · 0 comments · Fixed by #9202
Assignees
@lenaschoenburg
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-alpha1 Marks an issue as being completely or in parts released in 8.1.0-alpha1 version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0

Comments

@lenaschoenburg
Copy link
Member

lenaschoenburg commented Apr 13, 2022 •
edited
Loading

We should add another job to the test workflow that builds and tests the Go client.
Based on the script we use for Jenkins this seems to be fairly easy:

https://github.com/camunda/zeebe/blob/0bcea598445a984aee0009e9fe5f8a0c531f86f3/.ci/scripts/distribution/test-go.sh#L6
@lenaschoenburg lenaschoenburg mentioned this issue Apr 13, 2022
Closed
19 tasks
@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
@lenaschoenburg lenaschoenburg self-assigned this Apr 21, 2022
@lenaschoenburg lenaschoenburg mentioned this issue Apr 21, 2022
Merged
ghost pushed a commit that referenced this issue Apr 22, 2022
@lenaschoenburg
merge: #9202
be07133 
9202: ci: test the go client r=oleschoenburg a=oleschoenburg

Adds another job to our test workflow that runs tests for the Go client. In Jenkins we used gotestsum to generate JUnit reports but since we don't do anything with test reports yet, I've left that part out. Container logs are printed out on test failure.

closes #9128 

Co-authored-by: Ole Schönburg <ole.schoenburg@gmail.com>
@ghost ghost closed this as completed in #9202 Apr 22, 2022
@deepthidevaki deepthidevaki added the version:8.1.0-alpha1 Marks an issue as being completely or in parts released in 8.1.0-alpha1 label May 3, 2022
@ChrisKujawa ChrisKujawa added the version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0 label Oct 4, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lenaschoenburg lenaschoenburg

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-alpha1 Marks an issue as being completely or in parts released in 8.1.0-alpha1 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.

ci: test the go client camunda/camunda
4 participants
@npepinpe @lenaschoenburg @deepthidevaki @ChrisKujawa