Skip to content

core:platform Add Bareos firewalld services#1237

Merged
joergsteffens merged 2 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/master/add-firewalld-rules
Aug 25, 2022
Merged

core:platform Add Bareos firewalld services#1237
joergsteffens merged 2 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/master/add-firewalld-rules

Conversation

@bruno-at-bareos
Copy link
Contributor

@bruno-at-bareos bruno-at-bareos commented Aug 24, 2022

This PR is about preparing official Bareos firewalld services xml before submitting them to upstream.

upstream PR#1009 is now open

OP#4665

Thank you for contributing to the Bareos Project!

Please check

  • Short description and the purpose of this PR is present above this paragraph
  • Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

Checklist for the reviewer of the PR (will be processed by the Bareos team)

General
  • PR name is meaningful
  • Purpose of the PR is understood
  • Separate commit for this PR in the CHANGELOG.md, PR number referenced is same
  • Commit descriptions are understandable and well formatted
  • [ ] If backport: add original PR number and target branch at top of this file: Backport of PR#000 to bareos-2x
Source code quality
  • Source code changes are understandable
  • Variable and function names are meaningful
  • Code comments are correct (logically and spelling)
  • Required documentation changes are present and part of the PR
  • bareos-check-sources --since-merge does not report any problems
  • git status should not report modifications in the source tree after building and testing

@bruno-at-bareos bruno-at-bareos self-assigned this Aug 24, 2022
@bruno-at-bareos bruno-at-bareos marked this pull request as draft August 24, 2022 13:16
@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/add-firewalld-rules branch from 73d6be3 to 4c03c82 Compare August 25, 2022 12:01
@bruno-at-bareos bruno-at-bareos changed the title packaging: Add firewalld Bareos services core:platform Add Bareos firewalld services Aug 25, 2022
@bruno-at-bareos bruno-at-bareos marked this pull request as ready for review August 25, 2022 12:04
bruno-at-bareos and others added 2 commits August 25, 2022 17:30
- xml service files will be proposed to firewalld upstream project.
  short description has be done following Bareos Style Guide for
  common names.

Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
Co-authored-by: Jörg Steffens <joergsteffens@users.noreply.github.com>
Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/add-firewalld-rules branch from b46298c to e2c22e5 Compare August 25, 2022 15:30
@joergsteffens joergsteffens merged commit b3d0501 into bareos:master Aug 25, 2022
@bruno-at-bareos bruno-at-bareos deleted the dev/bruno/master/add-firewalld-rules branch August 31, 2022 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants