haproxy-status should depend on socat package

Bug #1479033 reported by Matthew Mosesohn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Alex Schultz
7.0.x
Won't Fix
Medium
Alex Schultz
8.0.x
Fix Released
Medium
Alex Schultz

Bug Description

socat is included in 2 random places (openstack-controller.pp and database.pp top level tasks). We should just include a dependency in fuel-misc to socat package.

Changed in fuel:
milestone: none → 7.0
importance: Undecided → Medium
assignee: nobody → Fuel Library Team (fuel-library)
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/206607

Changed in fuel:
status: New → In Progress
Changed in fuel:
status: In Progress → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-main (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/206661

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Sergii Golovatiuk (sgolovatiuk) wrote :

We cannot include this package as we install it from epel repo. It means that epel repo should be configured before we install the package.

Changed in fuel:
status: In Progress → Invalid
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

This isn't invalid. We use socat for haproxy-status.sh already. We just should move the dependency to it into the right package. It's already being used in Fuel for a very long time. We aren't pulling any RPMs from EPEL

Changed in fuel:
status: Invalid → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to packages/centos6/socat (7.0)

Reviewed: https://review.fuel-infra.org/10167
Submitter: Artem Silenkov <email address hidden>
Branch: 7.0

Commit: c594173b59ff0672865fdbeacb7e53c47a7a0b67
Author: Sergey Otpuschennikov <email address hidden>
Date: Thu Aug 6 13:38:13 2015

Empty commit for build socat in 7.0

Partial-Bug: #1479033
Change-Id: Ia592b42a9b73889e155126c085c8440ad217d5ee

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/206661
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=b866dec7729277d5ebbfdcb3533c33f230bffda8
Submitter: Jenkins
Branch: master

commit b866dec7729277d5ebbfdcb3533c33f230bffda8
Author: Alex Schultz <email address hidden>
Date: Tue Jul 28 14:10:32 2015 -0500

    Adding socat as an rpm requirement

    The fuel-misc package includes a script that needs socat but we are not
    providing it as part of our centos repository. This change adds socat to
    the requirements-rpm.txt

    Change-Id: I951b137627880b20ece3c64e6a0c8b6ab473daab
    Related-Bug: 1479033

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

Reviewed: https://review.openstack.org/206607
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=5c3e685c9cc5395da615d44f1f2b453e2612a8bb
Submitter: Jenkins
Branch: master

commit 5c3e685c9cc5395da615d44f1f2b453e2612a8bb
Author: Alex Schultz <email address hidden>
Date: Tue Jul 28 11:14:17 2015 -0500

    Moving socat to a fuel-misc dependency

    fuel-misc provides haproxy-status which needs socat. Additionally we
    are currently requiring the socat package in several places in
    puppet to try and satisfy this dependancy. This will consolidate
    them into a single requirement where it is needed.

    Depends-On: I951b137627880b20ece3c64e6a0c8b6ab473daab
    Change-Id: If3069c8c7bc68337596195e27e2213a4c9ec3540
    Closes-Bug: 1479033

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
Revision history for this message
Mikhail Samoylov (msamoylov) wrote :

Verified by code review and noop tests

tags: added: on-verification
Changed in fuel:
status: Fix Committed → Fix Released
status: Fix Released → Fix Committed
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.