Docs: Test deployment shouldn't talk about bonds

Bug #1631321 reported by Jean-Philippe Evrard
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
High
Jesse Pretorius

Bug Description

http://docs.openstack.org/developer/openstack-ansible/install-guide/app-targethosts-networkexample.html#test-environment

Maybe we should simplify this a little further, because prod deploys are explained with bonds already.

-----------------------------------
Release: 14.0.0.0b4.dev51 on 2016-09-08 10:36
SHA: 1a691abf05251b829a71465de53177ac01b3c167
Source: http://git.openstack.org/cgit/openstack/openstack-ansible/tree/doc/source/install-guide/app-targethosts-networkexample.rst
URL: http://docs.openstack.org/developer/openstack-ansible/install-guide/app-targethosts-networkexample.html

Praveen N (praveenn)
Changed in openstack-ansible:
assignee: nobody → Praveen N (praveenn)
Revision history for this message
Jesse Pretorius (jesse-pretorius) wrote :

The file https://github.com/openstack/openstack-ansible/blob/master/etc/network/interfaces.d/openstack_interface.cfg.example is currently implemented as the example for both test and prod examples.

This should not be the case.

Ideally openstack_interface.cfg.example should be renamed to openstack_interface.cfg.prod.example and another file called openstack_interface.cfg.test.example should be created. The test example should be an implementation of network configuration based on a single NIC host.

Here are two single-NIC configurations recently shared in #openstack-ansible:

https://gist.github.com/busterswt/01e706086c6bc6108e66e06fc992d1c0
http://cdn.pasteraw.com/95i1ixw313otqxr6gxppdiubi5a9qjn

Revision history for this message
Jean-Philippe Evrard (jean-philippe-evrard) wrote :

We've discussed this during our bug triage, and decided to expedite this commit. Raised to high.

Changed in openstack-ansible:
status: New → In Progress
importance: Undecided → Wishlist
importance: Wishlist → High
Revision history for this message
Jesse Pretorius (jesse-pretorius) wrote :

Switching assignment as there's been no activity on this one, and ideally it needs to make it into the Newton release. If a patch is pushed up in the next 12 hours then great, otherwise I'll do it.

Changed in openstack-ansible:
assignee: Praveen N (praveenn) → Jesse Pretorius (jesse-pretorius)
Revision history for this message
Praveen N (praveenn) wrote :

Am working on this, will push the patch soon.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible (master)

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

Changed in openstack-ansible:
assignee: Jesse Pretorius (jesse-pretorius) → Praveen N (praveenn)
Revision history for this message
Praveen N (praveenn) wrote :

Patch includes:

1. Renamed file openstack_interface.cfg.example to openstack_interface.cfg.prod.example
2. Created file openstack_interface.cfg.test.example with contents as per link https://gist.github.com/busterswt/01e706086c6bc6108e66e06fc992d1c0

Changed in openstack-ansible:
assignee: Praveen N (praveenn) → Jesse Pretorius (jesse-pretorius)
Changed in openstack-ansible:
assignee: Jesse Pretorius (jesse-pretorius) → Praveen N (praveenn)
Changed in openstack-ansible:
assignee: Praveen N (praveenn) → Jesse Pretorius (jesse-pretorius)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible (master)

Reviewed: https://review.openstack.org/385272
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=5686c70033e2be3f1f095355aec98e5ae1e6975f
Submitter: Jenkins
Branch: master

commit 5686c70033e2be3f1f095355aec98e5ae1e6975f
Author: npraveen35 <email address hidden>
Date: Wed Oct 12 10:25:38 2016 +0530

    [docs] Add network config example for test and prod

    Change-Id: Id390253a9f2e081376b3dab04d6966870bcfff7b
    Closes-Bug: #1631321

Changed in openstack-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/386049

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible (stable/newton)

Reviewed: https://review.openstack.org/386049
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=a0bfb6c8d31a3962eaa0d1569a94e8741017887a
Submitter: Jenkins
Branch: stable/newton

commit a0bfb6c8d31a3962eaa0d1569a94e8741017887a
Author: npraveen35 <email address hidden>
Date: Wed Oct 12 10:25:38 2016 +0530

    [docs] Add network config example for test and prod

    Change-Id: Id390253a9f2e081376b3dab04d6966870bcfff7b
    Closes-Bug: #1631321
    (cherry picked from commit 5686c70033e2be3f1f095355aec98e5ae1e6975f)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible 14.0.0.0rc3

This issue was fixed in the openstack/openstack-ansible 14.0.0.0rc3 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible 15.0.0.0b1

This issue was fixed in the openstack/openstack-ansible 15.0.0.0b1 development milestone.

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.