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

Drop SuSEfirewall2 completely #806

Merged
merged 2 commits into from
Sep 29, 2020
Merged

Drop SuSEfirewall2 completely #806

merged 2 commits into from
Sep 29, 2020

Conversation

teclator
Copy link
Contributor

@teclator teclator commented Aug 28, 2018

Drop old SUSEFirewalld2 code which should not be used anymore.

See #806 (comment)

@coveralls
Copy link

coveralls commented Aug 28, 2018

Coverage Status

Coverage increased (+8.2%) to 40.557% when pulling d89566b on drop_susefirewall_code into a54ec44 on master.

@mvidner
Copy link
Member

mvidner commented Aug 28, 2018

Related: yast/yast-network#668

Please mention the original FATE ticket in the changelog: https://fate.suse.com/323460
And maybe also these in the commit message:

@teclator teclator changed the title Drop SuSEfirewall2 completely [WIP] Drop SuSEfirewall2 completely Sep 19, 2018
Since SLE-15 firewalld is the default firewall and we have dropped
the support for configuring SuSEFirewall2 through YaST.

- https://lists.opensuse.org/yast-devel/2018-02/msg00002.html
- https://lizards.opensuse.org/2017/12/07/highlights-of-yast-development-sprint-47/ "Firewalld will be the default firewall in SLE15 (Installer adapted)"
@teclator teclator changed the title [WIP] Drop SuSEfirewall2 completely Drop SuSEfirewall2 completely Sep 29, 2020
@teclator teclator merged commit 9c012c6 into master Sep 29, 2020
@teclator teclator deleted the drop_susefirewall_code branch September 29, 2020 14:53
@yast-bot
Copy link
Contributor

✔️ Public Jenkins job #255 successfully finished
✔️ Created OBS submit request #838582

@yast-bot
Copy link
Contributor

✔️ Internal Jenkins job #120 successfully finished
✔️ Created IBS submit request #227514

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.

None yet

5 participants