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 new YaST2 UI firewalld #6113
Conversation
9fea2c6
to
3b940d5
Compare
be6e052
to
3a2a434
Compare
tests/yast2_gui/yast2_firewall.pm
Outdated
|
||
sub configure_firewalld { | ||
record_info('Start-Up', "Managing the firewalld service"); | ||
stop_service; |
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.
What's about service status asserting and not just believing that GUI did all the work?
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.
good suggestion, I added a new bug for that in the first comment along with VR updated.
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.
to verify right after the service I have to come back to the user console because I was ask for user password in the UI, very strange..., and now a "workaround box" appears. Do you think is correct the transition? Forget about it, it is not needed. Updated.
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.
Except the fact that we got 150 needles to maintain for 3 distributions, LGTM.
3a2a434
to
b68a7cd
Compare
b68a7cd
to
ff173b5
Compare
Add new YaST2 UI firewalld: fate#324662:
Verify change of service status.
Verify change of default zone when the interface is assigned to default and when the interface is assigned to a different zone.
Add one service and one port.
Verify with firewall-cmd previous changes are applied.
Record corresponding soft-failures found: bsc#1114677 bsc#1114673 bsc#1114807
Related ticket: https://progress.opensuse.org/issues/42482
Needles:
opensuse needlessle needlesVerification run: