Incorrect naming and structure of roles in ansible_deployer

Bug #1794578 reported by Alexandre Levine
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
In Progress
Undecided
Alexandre Levine
OpenContrail
In Progress
Undecided
Alexandre Levine

Bug Description

Current ansible_deployer has roles like "install_contrail", "configure_contrail", etc..., which doesn't look like correct roles in term of names. They will be renamed and roles will be restructured a bit to fit new correct roles

Changed in opencontrail:
assignee: nobody → Alexandre Levine (alexandrelevine)
status: New → In Progress
Revision history for this message
Alexandre Levine (alexandrelevine) wrote :

The following review addresses the change:
https://review.opencontrail.org/#/c/45910/

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/45910
Submitter: Alexandre Levine (<email address hidden>)

Jeba Paulaiyan (jebap)
tags: added: ansible provisioning
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/45910
Committed: http://github.com/Juniper/contrail-ansible-deployer/commit/d38bb361749b9c628a5739c6bc5c3e5ff55aef70
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit d38bb361749b9c628a5739c6bc5c3e5ff55aef70
Author: alexandrelevine <email address hidden>
Date: Wed Sep 26 10:44:44 2018 -0700

Refactored roles in ansible_deployer.
Incorrectly named roles like install_contrail, configure_contrail, will be replaced with correct namings like "contrail", "kvm", "k8s", etc, and "contrail_deployer, "kvm_deployer", "k8s_deployer", etc. The difference is xxx roles are for deployment to nodes and xxx_deployer roles are to be run on the machine where ansible_deployer is run. Basically, these latter roles could've been named "contrail_ansible_deployer", "kvm_ansible_deployer", etc, because they fulfill roles of ansible_deployer for particular service.

Change-Id: I0be6338daf78fb5f825548a98be606d4bfe01b60
Closes-Bug: #1794578

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/46595
Submitter: Alexandre Levine (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/46595
Committed: http://github.com/Juniper/contrail-ansible-deployer/commit/ae69865ce735d47612410555f3932c642ceae273
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit ae69865ce735d47612410555f3932c642ceae273
Author: alexandrelevine <email address hidden>
Date: Fri Sep 28 11:54:06 2018 -0700

Added explanation about new naming of the roles into README.md
Moved root_ca_generator functionality to contrail_deployer role

Change-Id: Iee51153f718cd0fda8575648261f8387aa04e4f2
Closes-Bug: #1794578

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/46769
Submitter: Andrey Pavlov (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/46769
Committed: http://github.com/Juniper/contrail-ansible-deployer/commit/b9c9f2caa5be1ed92856fe6de70f74567c4bee5c
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit b9c9f2caa5be1ed92856fe6de70f74567c4bee5c
Author: Andrey Pavlov <email address hidden>
Date: Sat Oct 6 11:55:16 2018 +0300

remove include of absent file

Change-Id: If5bbd70766ec059e6951c4f2bbc9ea3cfd0dabee
Partial-Bug: #1794578

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.