-
Notifications
You must be signed in to change notification settings - Fork 270
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
Disable samba module in ncurses suite #10270
Conversation
Add yaml file for isolated samba test
@@ -1400,7 +1400,6 @@ sub load_extra_tests_y2uitest_ncurses { | |||
unless (is_s390x) { | |||
loadtest "console/yast2_proxy"; | |||
loadtest "console/yast2_vnc"; | |||
loadtest "console/yast2_samba"; |
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 could be excluded based on a variable. The reason is that is will be manageable letting others knows whats going on and it will be easier to recover. What do you think?
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.
we could just mention in the follow-up ticket not forget to re-enable in O3 with proper schedule.
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.
other than a small comment looks good man
@@ -0,0 +1,8 @@ | |||
name: samba_ncurses | |||
description: > | |||
Samba test has to be excluded from ncurses suite until resolution of bsc#1146736. This test suite is to be executed in development groups. |
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.
Nitpicking, but could you please split it in multiple lines.
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, I was thinking to enable this module in development job group, so we can track current status more easily. WDYT?
Update: ah, I guess that's why you have added separate schedule for it. So we are all set.
Samba test has to be excluded from ncurses suite until resolution of bsc#1146736 and executed in development groups.