Upgrades, dockers port bindings after upgrade are different from those which on 5.1 iso

Bug #1350385 reported by Evgeniy L
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Evgeniy L

Bug Description

Steps to reproduce

1. install 5.0
2. run ugprade
3. check ports bindings

Actual result:
all ports are bound on '0.0.0.0'

Expected result:
in 5.1 version we have more complicated binding logic because of access control feature and new rules for firewall
Need to synchronize ports bindings with with those which are in dockerctl

https://github.com/stackforge/fuel-library/blob/aee8adbb5e26826e8e533d56dec265dbe12693c4/deployment/puppet/docker/templates/dockerctl_config.erb#L69-L79

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "5.1"
  api: "1.0"
  build_number: "375"
  build_id: "2014-07-30_15-13-48"
  astute_sha: "b16efcec6b4af1fb8669055c053fbabe188afa67"
  fuellib_sha: "0e60fb01a4600608e0c04741d264ab4edf9324db"
  ostf_sha: "9c0454b2197756051fc9cee3cfd856cf2a4f0875"
  nailgun_sha: "aed9ca4c68fee08a61c96c1d7271b6b42d31eb01"
  fuelmain_sha: "09acf940bc60b8c1dabd08641de21af3e3c8916a"

Tags: upgrade
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

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

Reviewed: https://review.openstack.org/110667
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=873ee736461a6ce922675777f247b30289ef53db
Submitter: Jenkins
Branch: master

commit 873ee736461a6ce922675777f247b30289ef53db
Author: Evgeniy L <email address hidden>
Date: Wed Jul 30 18:35:28 2014 +0400

    Upgrades, sync port binding scheme for containers with dockerctl

    In 5.1 we use new port binding scheme
    for containers, which is required
    for access control feature where
    we added new firewall rules.

    * sync ports binding with dockerctl
    * removed hack from docker engine
      where we converted lists in tuples
      because now we use python based config
      and we can use tuple based port binding
      format where it's required
    * also to get rid of hack was required
      because it was impossible to bind
      several host system ports on one
      container's port

    Change-Id: Ib3520a75362fe439dbe3dbe34e92da3f7b7307fa
    Closes-bug: #1350385

Changed in fuel:
status: In Progress → 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.