Project

Profile

Help

Issue #6753

pulp_installer requires the firewall python package to be installed

Added by mdepaulo@redhat.com over 1 year ago. Updated 10 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
Installer
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.

Associated revisions

Revision a30a3419 View on GitHub
Added by ppicka 11 months ago

Firewalld python package

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

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

Revision a30a3419 View on GitHub
Added by ppicka 11 months ago

Firewalld python package

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

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

History

#1 Updated by dkliban@redhat.com over 1 year ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 74

#2 Updated by rchan over 1 year ago

  • Sprint changed from Sprint 74 to Sprint 75

#3 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 75 to Sprint 76

#4 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 76 to Sprint 77

#5 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 77 to Sprint 78

#6 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 78 to Sprint 79

#7 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 79 to Sprint 80

#8 Updated by mdepaulo@redhat.com about 1 year ago

  • Assignee set to ppicka

#9 Updated by mdepaulo@redhat.com about 1 year 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

#10 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 80 to Sprint 81

#11 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 81 to Sprint 82

#12 Updated by pulpbot about 1 year ago

  • Status changed from NEW to POST

#13 Updated by rchan 12 months ago

  • Sprint changed from Sprint 82 to Sprint 83

#14 Updated by rchan 11 months ago

  • Sprint changed from Sprint 83 to Sprint 84

#15 Updated by ppicka 11 months ago

  • Status changed from POST to MODIFIED

#16 Updated by dkliban@redhat.com 10 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF