Opened on 06/07/2016 at 01:32:57 PM

Closed on 12/07/2016 at 09:16:09 AM

#4121 closed change (fixed)

Remove 3rd-party Puppet "firewall" module dependency

Reported by: matze Assignee: matze
Priority: P3 Milestone:
Module: Infrastructure Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29345624

Description (last modified by matze)

Module firewall has been introduced as dependency in infrastructure together with module postgresql, the latter of which being the entity using it. Module postgresql, however, does not depend on module firewall any more:

$ grep -R firewall * | grep -v '^\(Binary file \)\?modules/firewall'
dependencies:modules/firewall = infrastructure-puppetlabs-firewall hg:7005ec47f8a7 git:7b8ddb0
modules/nagios/templates/cgi.cfg.erb:# secure domain (i.e., behind a firewall) to see the current status
Binary file modules/postgresql/.hg/store/data/manifests/globals.pp.i matches
Binary file modules/postgresql/.hg/store/data/spec/system/server__spec.rb.i matches
modules/postgresql/.hg/store/fncache:data/spec/acceptance/firewall_default_parameters_spec.rb.i
modules/postgresql/.hg/store/fncache:data/manifests/server/firewall.pp.i
Binary file modules/postgresql/.hg/store/undo matches
modules/postgresql/.fixtures.yml:    firewall: "https://github.com/puppetlabs/puppetlabs-firewall.git"
modules/postgresql/CHANGELOG.md:This release removes the uninstall ability from the module, removes the firewall
modules/postgresql/CHANGELOG.md:- Removal of firewall management.

Hence we can remove the dependency from the infrastructure repository.

Attachments (0)

Change History (4)

comment:1 Changed on 06/07/2016 at 01:35:58 PM by matze

  • Description modified (diff)

comment:2 Changed on 06/07/2016 at 02:20:45 PM by abpbot

A commit referencing this issue has landed:
Issue 4121 - Remove dependency for module puppetlabs-firewall

comment:3 Changed on 06/07/2016 at 02:22:07 PM by matze

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:4 Changed on 12/07/2016 at 09:16:09 AM by matze

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from matze.
 
Note: See TracTickets for help on using tickets.