/etc/xinetd.d/swiftcheck does not allow all the networks in multirack envs

Bug #1500461 reported by Aleksandr Didenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Aleksandr Didenko
7.0.x
Won't Fix
High
Aleksandr Didenko

Bug Description

/etc/xinetd.d/swiftcheck parameter only_from contains only local (for particular node) management and storage networks. While in multirack envs we should add all management/storage networks there. This will cause problems if we deploy controllers in separate racks.

How to reproduce:
1) deploy multirack env with swift

Expected result:
/etc/xinetd.d/swiftcheck contains all management/storage networks in "only_from"

Actual result:
/etc/xinetd.d/swiftcheck contains only 1 management and 1 storage network in "only_from"

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/228487

Changed in fuel:
status: New → In Progress
tags: added: low-hanging-fruit
tags: added: feature-nodegroup
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/228487
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=2a92075c2a864abf747f03ad4e4cedf3655e62e0
Submitter: Jenkins
Branch: master

commit 2a92075c2a864abf747f03ad4e4cedf3655e62e0
Author: Aleksandr Didenko <email address hidden>
Date: Mon Sep 28 10:00:10 2015 -0500

    Fix swiftcheck for multirack

    Add storage/management networks from all nodegroups to swiftcheck
    'only_from' parameter.

    Change-Id: Ic8642be2c2e80e5f036c5b24eb0198a80f249628
    Closes-bug: #1500461

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Vitaly Sedelnik (vsedelnik) wrote :

Aleksandr, please review the fix and backport to 7.0 if applicable. If not, please update the status accordingly (Won't Fix or Invalid).

Dmitry Pyzhov (dpyzhov)
no longer affects: fuel/8.0.x
Revision history for this message
Aleksandr Didenko (adidenko) wrote :

I don't think that we need this fix in 7.0. The only way to deploy controllers in separate L3 segments/racks in 7.0 is using external LB. And in this case we do not setup VIPs and do not use haproxy on controllers at all. So this bug does not affect setup with external LB. Marked as won't fix for 7.0.

Dmitry Pyzhov (dpyzhov)
tags: added: area-library
tags: added: on-verification
Revision history for this message
Andrey Lavrentyev (alavrentyev) wrote :

Verification is blocked by: https://bugs.launchpad.net/fuel/+bug/1518341
Postponed until it gets fixed

Revision history for this message
Andrey Lavrentyev (alavrentyev) wrote :

Verified on ISO #529

[root@nailgun ~]# cat /etc/fuel/version.yaml
VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "529"
  build_id: "529"
  fuel-nailgun_sha: "baec8643ca624e52b37873f2dbd511c135d236d9"
  python-fuelclient_sha: "4f234669cfe88a9406f4e438b1e1f74f1ef484a5"
  fuel-agent_sha: "658be72c4b42d3e1436b86ac4567ab914bfb451b"
  fuel-nailgun-agent_sha: "b2bb466fd5bd92da614cdbd819d6999c510ebfb1"
  astute_sha: "b81577a5b7857c4be8748492bae1dec2fa89b446"
  fuel-library_sha: "e2d79330d5d708796330fac67722c21f85569b87"
  fuel-ostf_sha: "3bc76a63a9e7d195ff34eadc29552f4235fa6c52"
  fuel-mirror_sha: "fb45b80d7bee5899d931f926e5c9512e2b442749"
  fuelmenu_sha: "e071216cb214e34b4d861478033425ee6a54a3be"
  shotgun_sha: "63645dea384a37dde5c01d4f8905566978e5d906"
  network-checker_sha: "a43cf96cd9532f10794dce736350bf5bed350e9d"
  fuel-upgrade_sha: "616a7490ec7199f69759e97e42f9b97dfc87e85b"
  fuelmain_sha: "a365f05b903368225da3fea9aa42afc1d50dc9b4"

tags: removed: on-verification
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.