No nova-network on controller when multi-host is true#25
Closed
aimonb wants to merge 1 commit intostackforge:masterfrom
NexusIS:multi_host_logic_fix
Closed
No nova-network on controller when multi-host is true#25aimonb wants to merge 1 commit intostackforge:masterfrom NexusIS:multi_host_logic_fix
aimonb wants to merge 1 commit intostackforge:masterfrom
NexusIS:multi_host_logic_fix
Conversation
- Enabling multi-host moves the nova netowrk service off the controller and onto the computes. The current logic is broken in that it always sets the nova::network class to install and enable the nova_network service making the conditional pointless. While it seems harmles to have the extra nova-network service on the controller it is not. When doing HA or using docker to contain services it causes extra services to need to be managed. It also pollutes the nova services catalog.
Contributor
Author
|
Moved to gerrit! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
the controller and onto the computes. The current logic is
broken in that it always sets the nova::network class to
install and enable the nova_network service making the
conditional pointless. While it seems harmles to have the
extra nova-network service on the controller it is not.
When doing HA or using docker to contain services it causes
extra services to need to be managed. It also pollutes the
nova services catalog.