-
Notifications
You must be signed in to change notification settings - Fork 22
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
test: support for qdevice tls and kaptb options #153
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #153 +/- ##
=======================================
Coverage 80.45% 80.45%
=======================================
Files 3 3
Lines 133 133
=======================================
Hits 107 107
Misses 26 26
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
[citest] |
Support for these options is purely a pcs thing, there is no code update needed to be done in the role itself. This test demonstrates that the options are indeed supported by the role.
39221ba
to
9f95237
Compare
[citest] |
I guess the output of
|
I'm already working on it. It looks like pcs has been updated on Fedora and a condition in the test doesn't take it into account. |
[citest] |
Support for these options is purely a pcs thing, there is no code update needed to be done in the role itself. This test demonstrates that the options are indeed supported by the role.
Issue Tracker Tickets (Jira or BZ if any):
RHEL-3264
RHEL-7745