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
Sp2 merge #140
Sp2 merge #140
Conversation
Rakefile
Outdated
@@ -1,5 +1,7 @@ | |||
require "yast/rake" | |||
|
|||
Yast::Tasks.submit_to :sle15sp2 |
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 should be removed or revoked the same as the Dockerfile
@@ -81,6 +81,9 @@ def configure_firewall | |||
# Convenience method to enable / disable the firewalld service depending | |||
# on the proposal settings | |||
def configure_firewall_service | |||
# do not run in autoyast as it is done in second stage (bsc#1177778) | |||
return if Yast::Mode.auto |
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.
As commented in #139 it probably should not be done in master as it should be already handled.
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.
return Y2Firewall::Clients::Auto.new.write |
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.
After checked it is ok to be added.
This reverts commit affba0a.
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
✔️ Internal Jenkins job #16 successfully finished |
✔️ Public Jenkins job #39 successfully finished |
merge of #139
regarding moving firewall config to first stage, I would let it to own PBI.