Tendrl setup #6

fbalak · 2016-11-29T15:44:41Z

All changes from all other branches of mine, previous branches are deprecated. The pull request is opened for sake of a review.

…ittle code, changed default variable value of tendrl_gluster_bridge_repo_branch

…opying

This is a workaround for configuration management issue: Tendrl/ceph-integration#57

mbukatov · 2016-12-13T09:41:40Z

I'm going through the last step: installation of tendrl-api.

tendrl-devel role should be included automatically based on metadata dependencies of any role which would require devel tools to be installed during installation from source code

Default `exclude_groups` value was wrong, the default should follow suggestion for "production setup only" exalty as documented in README[1] file, which states: > [tendrl-api] $ bundle install --path vendor/bundle --binstubs vendor/bin \ > --without development test documentation [1] https://github.com/Tendrl/tendrl-api/blob/74db59b7cfb7ac1a1df20cfcda948c33a67061d8/README.adoc

This is initial version of suggested workaround for: Tendrl/api#41

related to Tendrl/api#42

fix of workaround for: Tendrl/api#41

based on Tendrl/api#25

Previous code was not idempotent (multiple runs would corrupt the etcd.yml file) and this commit fixes that problem.

based on Tendrl/api#25 Epel repository is enabled for tendrl-api role only, in clear and explicit way.

based on based on Tendrl/node-agent#75

mbukatov · 2016-12-19T16:07:30Z

Ops, my idea (based on ansible docs, see [1]) about setting enablerepo=None doesn't work, when I do that, I get:

FAILED! => {"changed": false, "failed": true, "msg": "Error setting/accessing repos: Error getting repository data for None, repository not found"}

[1] ansible docs for yum module states:

enablerepo
`Repoid' of repositories to enable for the install/update operation. These repos will not persist beyond
the transaction. When specifying multiple repos, separate them with a ",".
[Default: None]

From now on, roles doesn't care about epel (or any other repositories) at all. One needs to enable epel by adding epel role into ap playbook and then list tendrl roles.

mbukatov · 2016-12-20T12:26:09Z

@mkudlej could you review the current state again? I don't expect major changes at this point.

mkudlej

It seems OK to me. I haven't tested if code is working.

mbukatov · 2016-12-20T13:19:48Z

Source installation needs a little tweak:

# curl http://localhost/api/1.0/Flows
<h1>Internal Server Error</h1>

Update: Since this may actually be a bug in tendrl-api (see Tendrl/api#31), it's not blocking merge of the pull request.

Since it's not clear how to setup etcd in clustered mode, it's not necessary to install it like this, even though it's stated in the installation.rst file. Related to Tendrl/node-agent#88

mbukatov · 2016-12-20T16:32:24Z

Ok, let's merge it to a master. There are many open questions about particular configuration details, or known issues on the tendrl side, so that the Tendrl installed via tendrl_gluster.yml playbook would not work (either tendrl-api, tendrl-node-agent or both are crashing), but from the ansible setup point of view, the setup is fine.

fbalak and others added 30 commits November 29, 2016 16:00

add tendrl-devel role

20ddc0e

virtualenv installs from yum not pip, upgraded documentation

0af7226

removed package list from README.md

eadfa88

added tendrl-bridge-common role

dfd2b9a

added etcd role

2012996

added tendrl-gluster-bridge role

258f205

removed commented code

82d2610

added tendrl-tendrl role

730a6a3

added defaults folder with variables to tendrl-gluster-bridge

9125525

added tendrl-ceph-bridge role

624372b

added dependency files

6d7156c

added tendrl-node-agent role

8dba8fb

polish etcd role

e0895d4

update readme of tendrl-devel role

1b3b22e

polish tendrl-api role (renamed from tendrl-tendrl)

85f685b

polish roles according to new specification

0a8d299

deleted old virtualenv files, disabled etcd seed

c43f08e

corrected gluster_bridge installation command

522cfc3

deleted tendrl-tendrl dir (now it is known as tendrl-api), polished l…

6d51481

…ittle code, changed default variable value of tendrl_gluster_bridge_repo_branch

corrected configuration

2c47ed2

added qe_tendrl playbook

3a90aab

unified and corrected variable names, added changes from node_agent repo

2923c09

Enable etcd service

1ed7e80

Add several other dependencies

ed76c2a

Fix dependency commit plus update of source install - tendrl-api role

714d31e

change repo names, correct node_agend installation, logging configs c…

e059321

…opying

correct tendrl.conf, rename variables, rename qe_tendrl.yml

732a43c

Correct folder creation

ff4fd10

add gluster role

4c481b8

add tendrl-frontend role

53f6fb7

mbukatov added 2 commits December 12, 2016 20:38

tendrl-ceph-integration: cleanup

b11dc96

tendrl-ceph-integration: config workaround

1ce397e

This is a workaround for configuration management issue: Tendrl/ceph-integration#57

mbukatov added 9 commits December 14, 2016 13:13

tendrl-api: cleanup

b776a81

rename: tendrl.yml to tendrl_gluster.yml

5a9941d

tendrl_gluster.yml: drop explicit tendrl-devel

d6778a6

tendrl-devel role should be included automatically based on metadata dependencies of any role which would require devel tools to be installed during installation from source code

tendrl-api: httpd related configuration

beb6c9d

tendrl-api: enable httpd_can_network_connect

50a3fcf

tendrl-api: source install: service workaround

814b90b

This is initial version of suggested workaround for: Tendrl/api#41

tendrl-api: enable boolean for source install only

0367ce4

related to Tendrl/api#42

tendrl-api: fix of workaround

54d8947

fix of workaround for: Tendrl/api#41

dahorak approved these changes Dec 19, 2016

View reviewed changes

mbukatov added 4 commits December 19, 2016 14:52

tendrl_gluster.yml: enable epel for server role

0b88d5d

based on Tendrl/api#25

tendrl-api: fix etcd.yml hack

3dd86a5

Previous code was not idempotent (multiple runs would corrupt the etcd.yml file) and this commit fixes that problem.

tendrl_gluster.yml: enable epel for tendrl-api

141c046

based on Tendrl/api#25 Epel repository is enabled for tendrl-api role only, in clear and explicit way.

tendrl_gluster.yml: enable epel for tendrl-node-agent

a6b9580

based on based on Tendrl/node-agent#75

mbukatov added 2 commits December 20, 2016 11:00

yum --enablerepo hack for epel

4fd3559

siplify epel handling in tendrl roles

4356449

From now on, roles doesn't care about epel (or any other repositories) at all. One needs to enable epel by adding epel role into ap playbook and then list tendrl roles.

fix minor issues in README files

d82b069

mkudlej approved these changes Dec 20, 2016

View reviewed changes

mbukatov mentioned this pull request Dec 20, 2016

stacktrace in development environment Tendrl/api#31

Closed

tendrl-node-agent: drop etcd dependency

783492f

Since it's not clear how to setup etcd in clustered mode, it's not necessary to install it like this, even though it's stated in the installation.rst file. Related to Tendrl/node-agent#88

mbukatov approved these changes Dec 20, 2016

View reviewed changes

mbukatov merged commit ceb7b24 into master Dec 20, 2016

mbukatov deleted the tendrl-setup branch December 20, 2016 16:34

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Tendrl setup #6

Tendrl setup #6

fbalak commented Nov 29, 2016

mbukatov commented Dec 13, 2016

mbukatov commented Dec 19, 2016

mbukatov commented Dec 20, 2016

mkudlej left a comment

mbukatov commented Dec 20, 2016 •

edited

mbukatov commented Dec 20, 2016

Tendrl setup #6

Tendrl setup #6

Conversation

fbalak commented Nov 29, 2016

mbukatov commented Dec 13, 2016

mbukatov commented Dec 19, 2016

mbukatov commented Dec 20, 2016

mkudlej left a comment

Choose a reason for hiding this comment

mbukatov commented Dec 20, 2016 • edited

mbukatov commented Dec 20, 2016

mbukatov commented Dec 20, 2016 •

edited