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.

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

Add Softtoken Tests #89 #145

Closed
wants to merge 2 commits into from
Closed

Add Softtoken Tests #89 #145

wants to merge 2 commits into from

Conversation

robertmin1
Copy link
Contributor

@robertmin1 robertmin1 commented Oct 18, 2022

Fix #89
The test was done using a debian as an image.
Depending on the feedback I will add more softtoken tests.
I am not sure if I can combine this container with the first one, since they both use fedora, but knowing which one failed might be a foreseeable issue.

@robertmin1
Copy link
Contributor Author

robertmin1 commented Oct 18, 2022

Test failed - Let me look into it

@robertmin1
Copy link
Contributor Author

Managed to fix the error : Failed to connect to the bus: Failed to connect to socket /run/dbus/system_bus_socket
By installing and configuring dbus

@JeremyRand
Copy link
Member

Cirrus YML files support multiple matrix instructions. So I think you can simplify this by simply adding a 2nd matrix instruction below the first one, and move the CI_MAIN_MODULE and CI_BAK_MODULE variables to the 2nd matrix. If the dbus change is still useful after doing that, adding it to the first matrix (so that it runs regardless of CKBI/Softoken) as a 2nd commit seems reasonable.

@robertmin1
Copy link
Contributor Author

robertmin1 commented Oct 23, 2022

Okay, implemented.
The OOM score is read-only, hence it causes the error "[1023/101542.384179:ERROR:zygote_host_impl_linux.cc(262)] Failed to adjust OOM score of renderer with pid 6888: Permission denied (13)"

.cirrus.yml Outdated Show resolved Hide resolved
Add Softtoken Tests #89

Add Softtoken Tests #89

Add Softtoken Tests #89

Add Softtoken Tests #89

Add Softtoken Tests #89
@robertmin1 robertmin1 closed this Oct 26, 2022
@robertmin1 robertmin1 deleted the softtoken branch October 26, 2022 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Softoken tests
2 participants