fix 3.1.x for several items

blueboxgroup / ursula

Ansible playbooks for operating OpenStack - Powering Blue Box Cloud.

https://www.blueboxcloud.com

204 stars 5 forks source link

fix 3.1.x for several items #2822

Closed panxia6679 closed 7 years ago

panxia6679 commented 7 years ago

specify lbaas pannel version
wrong cert in defaults-2.0.yml
disable pip editable
add heat template for ci-full
update all.yml for ci-full

bbc-jenkins commented 7 years ago

Can one of the admins verify this patch?

\"ok to test\" to accept this pull request for testing
\"test this please\" for a one time test run
\"add to whitelist\" to add the author to the whitelist

bbc-jenkins commented 7 years ago

Can one of the admins verify this patch?

\"ok to test\" to accept this pull request for testing
\"test this please\" for a one time test run
\"add to whitelist\" to add the author to the whitelist

bbc-jenkins commented 7 years ago

Can one of the admins verify this patch?

\"ok to test\" to accept this pull request for testing
\"test this please\" for a one time test run
\"add to whitelist\" to add the author to the whitelist

nirajdp76 commented 7 years ago

I do not understand why we are making these changes. 31x works fine and we should not be making changes to branch release just for sake of improving it. We have revived ci envs in master.

Probably your approach to use ci-fill envs is not correct as in 31x we used a script to deploy.

panxia6679 commented 7 years ago

Thanks Niraj for your detailed comments. We have to run scripts under test/ to prepare VM based environment for 31x. I raised the PR to directly use one ursula command to create VM based environment. Given the direction we should not make more changes of improvement for old releases , I am going to close the PR.