Multiple storage-master roles not supported [HA requirement]

Bug #1366216 reported by Michael Nishimoto
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Fix Committed
High
Vinod Nair
R1.1
Fix Committed
High
Vinod Nair
R1.30
Won't Fix
High
Jeya ganesh babu J

Bug Description

We need to support multiple storage-master nodes because this is required for HA configs.
Today, the fab scripts only look at the first node in the storage-master list.

Once this is done, we will need to make this work in the SM environment.

Tags: storage
Changed in juniperopenstack:
milestone: r1.11 → none
information type: Proprietary → Public
summary: - Multiple storage-master roles not supported
+ Multiple storage-master roles not supported [HA requirement]
Changed in juniperopenstack:
milestone: none → r2.0-fcs
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/4246
Committed: http://github.org/Juniper/contrail-fabric-utils/commit/8e7deddbcaa7dd35a8b08e26d293af448eea8f1b
Submitter: Zuul
Branch: master

commit 8e7deddbcaa7dd35a8b08e26d293af448eea8f1b
Author: spuru <email address hidden>
Date: Thu Oct 30 16:38:03 2014 -0700

Added Storage HA support

Closes-Bug: #1366216
Feature: Storage HA support added over existing contrail HA nodes.
Change:
1. cinder backend rbd settngs done for openstack storage nodes.
2. cinder template config set to cinder HA vip.
3. global public_network set to newly added storage node address.
4. new cursh map is loaded using crushtool.
5. cinder rabbit host settings done for openstack storage nodes.
6. skipped compute-restart on openstack storage nodes.
Test: verified storage HA by adding 2 openstack storage nodes on
storage-master role. HA Cinder volume properly attached to
VMs.
Change done based on Review comments:
1. removed tabs in storage-ceph-storage.py
2. grep -w added to match exact wip address
3. "storage os" comment changed to "storage openstack"
4. "cfg-hostlist" changed to "config-hosts" as per convention
5. added comments for new parameters
6. setup_storage call done for add_storage_node

Change-Id: Iae7a954adbdf05dc6cd05952ee1d1a9490a13929

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/4244
Committed: http://github.org/Juniper/contrail-provisioning/commit/7252bada51ad4b1cc37123536658cd1edd0ee258
Submitter: Zuul
Branch: master

commit 7252bada51ad4b1cc37123536658cd1edd0ee258
Author: spuru <email address hidden>
Date: Thu Oct 30 16:22:38 2014 -0700

Added Storage HA support

Closes-Bug: #1366216
Feature: Storage HA support added over existing contrail HA nodes.
Change:
1. cinder backend rbd settngs done for openstack storage nodes.
2. cinder template config set to cinder HA vip.
3. global public_network set to newly added storage node address.
4. new cursh map is loaded using crushtool.
5. cinder rabbit host settings done for openstack storage nodes.
6. skipped compute-restart on openstack storage nodes.
7. added rabbit_port for cinder ha.

Test: verified storage HA by adding 2 openstack storage nodes on
storage-master role. HA Cinder volume properly attached to
VMs.

Change done based on Review comments:
1. removed tabs in storage-ceph-storage.py
2. grep -w added to match exact wip address
3. "storage os" comment changed to "storage openstack"
4. "cfg-hostlist" changed to "config-hosts" as per convention
5. added comments for new parameters
6. changed virsh secret list parsing logic

Change-Id: Ibb3d488202bd84efaa14575542a4aae4a896e57a

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/4111
Committed: http://github.org/Juniper/contrail-fabric-utils/commit/73cefe7989d1c5e5583a821e194f0ad34dc573f4
Submitter: Zuul
Branch: R1.10

commit 73cefe7989d1c5e5583a821e194f0ad34dc573f4
Author: spuru <email address hidden>
Date: Sun Oct 26 22:50:16 2014 -0700

Added Storage HA support

Closes-Bug: #1366216
Feature: Storage HA support added over existing contrail HA nodes.
Change:
1. cinder backend rbd settngs done for openstack storage nodes.
2. cinder template config set to cinder HA vip.
3. global public_network set to newly added storage node address.
4. new cursh map is loaded using crushtool.
5. cinder rabbit host settings done for openstack storage nodes.
6. skipped compute-restart on openstack storage nodes.
Test: verified storage HA by adding 2 openstack storage nodes on
storage-master role. HA Cinder volume properly attached to
VMs.
Change done based on Review comments:
1. removed tabs in storage-ceph-storage.py
2. grep -w added to match exact wip address
3. "storage os" comment changed to "storage openstack"
4. "cfg-hostlist" changed to "config-hosts" as per convention
5. added comments for new parameters
6. setup_storage call done for add_storage_node

Change-Id: I32b2f0e3cccf8c5ea6c05be88ef71199befe6e9d

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/4110
Committed: http://github.org/Juniper/contrail-provisioning/commit/6f30d37371eae195b9d273b46b7de9c04cb1def9
Submitter: Zuul
Branch: R1.10

commit 6f30d37371eae195b9d273b46b7de9c04cb1def9
Author: spuru <email address hidden>
Date: Sun Oct 26 22:48:58 2014 -0700

Added Storage HA support

Closes-Bug: #1366216
Feature: Storage HA support added over existing contrail HA nodes.
Change:
1. cinder backend rbd settngs done for openstack storage nodes.
2. cinder template config set to cinder HA vip.
3. global public_network set to newly added storage node address.
4. new cursh map is loaded using crushtool.
5. cinder rabbit host settings done for openstack storage nodes.
6. skipped compute-restart on openstack storage nodes.
7. added rabbit_port for cinder ha.

Test: verified storage HA by adding 2 openstack storage nodes on
storage-master role. HA Cinder volume properly attached to
VMs.

Change done based on Review comments:
1. removed tabs in storage-ceph-storage.py
2. grep -w added to match exact wip address
3. "storage os" comment changed to "storage openstack"
4. "cfg-hostlist" changed to "config-hosts" as per convention
5. added comments for new parameters
6. changed virsh secret list parsing logic
Change-Id: Idb0de4a72f753997188a71acadc4142dc5ce8032

Changed in juniperopenstack:
status: New → Fix Committed
assignee: Jeya ganesh babu (jjeya) → Vinod Nair (vinodnair)
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.