Project

Profile

Help

Actions
Send by e-mail Copy link

Issue #6753

closed

pulp_installer requires the firewall python package to be installed

Added by mdepaulo@redhat.com over 4 years ago. Updated about 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
ppicka
Category:
Installer - Moved to GitHub issues
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 84
Quarter:

Description

As documented in https://pulp.plan.io/issues/6725, the managed system must have the python package "firewall" installed.

1st of all, this is only true if pulp_configure_firewall is set to "auto" (the default) or "firewalld".

2nd, if firewalld is installed and running, but the python module for it is not installed, then we should install the distro package that provides it (for the distro's default python interpreter) when the variable is set to "auto" or "firewalld".

For reference on this being the intended behaviour of an ansible role, consider how the ansible apt module will automatically install the python3-apt package (presumably via commands) if it is missing.

If firewalld is not installed and running, then "auto" should do nothing.

Actions
Copy link
 #1

Updated by dkliban@redhat.com over 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 74
Actions
Copy link
 #2

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 74 to Sprint 75
Actions
Copy link
 #3

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 75 to Sprint 76
Actions
Copy link
 #4

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 76 to Sprint 77
Actions
Copy link
 #5

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 77 to Sprint 78
Actions
Copy link
 #6

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 78 to Sprint 79
Actions
Copy link
 #7

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 79 to Sprint 80
Actions
Copy link
 #8

Updated by mdepaulo@redhat.com over 4 years ago

  • Assignee set to ppicka
Actions
Copy link
 #9

Updated by mdepaulo@redhat.com over 4 years ago

Note:

  1. We install the firewall package in the dockerfile used by molecule create. You need to delete that logic, and add a task to install it in the pulp_webserver role. https://github.com/pulp/pulp_installer/blob/master/molecule/scenario_resources/Dockerfile.j2

  2. I just remembered that we pre-build that container image and push nightly (github actions "cronjob). Please work with fao89 on a solution to test this change at PR test time: https://github.com/pulp/pulp_installer/blob/master/.github/build_container.yaml

Actions
Copy link
 #10

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 80 to Sprint 81
Actions
Copy link
 #11

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 81 to Sprint 82
Actions
Copy link
 #12

Updated by pulpbot over 4 years ago

  • Status changed from NEW to POST
Actions
Copy link
 #13

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 82 to Sprint 83
Actions
Copy link
 #14

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 83 to Sprint 84

Added by ppicka about 4 years ago

Revision a30a3419 | View on GitHub

Firewalld python package

Added installation of firewalld python package to installer when specified to install firewall.

closes: #6753 https://pulp.plan.io/issues/6753

Added by ppicka about 4 years ago

Revision a30a3419 | View on GitHub

Firewalld python package

Added installation of firewalld python package to installer when specified to install firewall.

closes: #6753 https://pulp.plan.io/issues/6753

Actions
Copy link
 #15

Updated by ppicka about 4 years ago

  • Status changed from POST to MODIFIED
Actions
Copy link
 #16

Updated by dkliban@redhat.com about 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF