-
Notifications
You must be signed in to change notification settings - Fork 51
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.
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
Testing KUKSA Server Unittest #629
Conversation
Now trying to use 4 everywhere
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe need to open the PR again.
- ".github/workflows/kuksa_val_unittest.yml" | ||
- "kuksa-val-server/**" | ||
- "kuksa_certificates/**" | ||
- "proto/**" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the same issue as in #611 not the right location to look for proto files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed by #633
This aligns unit-tests with other triggers and now it works. It also removes explicit install of gRPC and Boost. That is included from cmake so not needed. The old warning that it might be needed for "older" versions of cmake shall likely not be relevant any longer, at least not if we assume that Ubuntu 20 or newer is used.
Still the build takes time. A possible remedy for CI could be to join both builds to the same so we do not need to install boost and gRPC twice.
Now also using 4 threads (
-j 4
in all builds)