Review and merge the old capomastro-deploy branch code with the new charm
Bug #1385221 reported by
Caio Begotti
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Capomastro |
Fix Released
|
Low
|
Caio Begotti |
Bug Description
If it still makes sense... need to check it first anyway because that's all legacy stuff.
Related branches
Changed in capomastro: | |
assignee: | nobody → Caio Begotti (caio1982) |
Changed in capomastro: | |
importance: | Undecided → Low |
status: | New → Confirmed |
Changed in capomastro: | |
status: | Confirmed → Fix Released |
To post a comment you must log in.
The old branch is lp:~ce-infrastructure/bygmester/capomastro-deploy/ and I think most if not all of it is already explained in the README file of the new Capomastro charm. What it basically does as I can see is to automate the actual deployment commands present in the new README anyway.
I think working on #1389669 makes more sense as the rest of this capomastro-deploy branch would be made obsolete by it. I mean, we can still work on this bug and rewrite the deploy helpers, but IMHO it would have little use in the long term. The only reason to work on this is to acquire knowledge on how David and Kevin did do things to hook it up with haproxy etc.