Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code take care only of compute and controller on Red Hat osp10 #1

Open
clopnis opened this issue Jun 9, 2017 · 0 comments · May be fixed by #3
Open

Code take care only of compute and controller on Red Hat osp10 #1

clopnis opened this issue Jun 9, 2017 · 0 comments · May be fixed by #3

Comments

@clopnis
Copy link

clopnis commented Jun 9, 2017

Hello,

I case of using a tripleo deployment with ceph node, swift... or other custom, the deployment fail due the code will take other node as compute node.

if $ctrlrs != "_not_there" {
  if $hostname in $ctrlrs {
     $role = 'controller'
  } else {
     $role = 'compute'
}

https://github.com/noironetworks/aci-tripleo-patch/blob/newton/aci-tripleo-patch/puppet/manifests/apic.pp#L5-L10
https://github.com/noironetworks/aci-tripleo-patch/blob/newton/aci-tripleo-patch/puppet/manifests/compute_metadata.pp#L3-L10

@clopnis clopnis changed the title Code take care only of compute and controller Code take care only of compute and controller on Red Hat osp10 Jun 9, 2017
clopnis added a commit to clopnis/aci-tripleo-patch that referenced this issue Jun 9, 2017
The heat template was not working on osp10 cause it was using old resources. Also the puppet code was tried to be run on role as ceph, swift what is not expected and was creating failure at deployment.

Tested and works on newton (Red Hat osp10) and ACI 2.2(2e)

Signed-off-by: Cyril Lopez <[email protected]>

fix noironetworks#1

Signed-off-by: Cyril Lopez <[email protected]>
@clopnis clopnis linked a pull request Jun 9, 2017 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant