Hardcoded /etc/init.d/iptables call in cobbler manifest

Bug #1524828 reported by Vladimir Khlyunev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Alex Schultz

Bug Description

BVT failed
https://product-ci.infra.mirantis.net/job/8.0.ubuntu.bvt_2/262/

Error message from logs:
2015-12-10 11:58:13 +0000 Puppet (debug): Executing 'iptables -t mangle -A POSTROUTING -p udp --dport 68 -j CHECKSUM --checksum-fill; /etc/init.d/iptables save'
2015-12-10 11:58:13 +0000 Puppet (err): sh: /etc/init.d/iptables: No such file or directory
2015-12-10 11:58:13 +0000 /Stage[main]/Cobbler::Checksum_bootpc/Exec[checksum_fill_bootpc]/returns (err): change from notrun to 0 failed: sh: /etc/init.d/iptables: No such file or directory

Snapshot https://product-ci.infra.mirantis.net/job/8.0.ubuntu.bvt_2/262/artifact/logs/fail_error_ceph_rados_gw-fuel-snapshot-2015-12-10_12-18-58.tar.xz

Root cause: hardcoded /etc/init.d/iptables call in
https://github.com/openstack/fuel-library/blob/master/deployment/puppet/cobbler/manifests/checksum_bootpc.pp#L23

description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/255871

Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Bogdan Dobrelya (bogdando)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (master)

Change abandoned by Bogdan Dobrelya (<email address hidden>) on branch: master
Review: https://review.openstack.org/255871

Changed in fuel:
assignee: Bogdan Dobrelya (bogdando) → Fuel Library Team (fuel-library)
status: In Progress → Triaged
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Alex Schultz (alex-schultz)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/256005

Changed in fuel:
status: Triaged → In Progress
tags: added: team-bugfix
Dmitry Pyzhov (dpyzhov)
tags: added: feature-centos7 team-linux
removed: team-bugfix
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/256005
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=fca6c1dc769efcf6fd21c293d7e15495b9df81d8
Submitter: Jenkins
Branch: master

commit fca6c1dc769efcf6fd21c293d7e15495b9df81d8
Author: Alex Schultz <email address hidden>
Date: Thu Dec 10 10:29:43 2015 -0700

    Update cobbler checksum firewall exec for centos7

    This change removes the reference to the iptables init script which no
    longer exists under CentOS7 and replaces it with a iptables-save that is
    similar to the way it is done for Debian based systems.

    Change-Id: Ib6a4b7a87c452e6397d0b708f3408c32c11bd01f
    Closes-Bug: #1524828

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Alexander Zatserklyany (zatserklyany) wrote :

Fix released

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "417"

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.