Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3007)

Issue 107890043: Resolve port conflicts in cf-dea

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 11 months ago by johnsca
Modified:
9 years, 11 months ago
Reviewers:
benjamin.saller, mp+222670
Visibility:
Public.

Description

Resolve port conflicts in cf-dea https://code.launchpad.net/~johnsca/charms/trusty/cf-dea/port-conflicts/+merge/222670 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Resolve port conflicts in cf-dea #

Patch Set 3 : Resolve port conflicts in cf-dea #

Unified diffs Side-by-side diffs Delta from patch set Stats (+157 lines, -93 lines) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M config.yaml View 1 2 chunks +1 line, -2 lines 0 comments Download
M hooks/charmhelpers/contrib/cloudfoundry/contexts.py View 3 chunks +17 lines, -6 lines 0 comments Download
M hooks/charmhelpers/core/services.py View 9 chunks +123 lines, -72 lines 0 comments Download
M hooks/config.py View 1 2 chunks +3 lines, -1 line 0 comments Download
M metadata.yaml View 1 chunk +2 lines, -2 lines 0 comments Download
M templates/dea.yml View 2 chunks +2 lines, -3 lines 0 comments Download
M templates/dea_logging_agent.json View 1 1 chunk +7 lines, -7 lines 0 comments Download

Messages

Total messages: 4
johnsca
Please take a look.
9 years, 11 months ago (2014-06-10 15:53:59 UTC) #1
johnsca
Please take a look.
9 years, 11 months ago (2014-06-11 21:56:18 UTC) #2
benjamin.saller
LGTM, good stuff
9 years, 11 months ago (2014-06-11 23:42:38 UTC) #3
johnsca
9 years, 11 months ago (2014-06-12 18:06:40 UTC) #4
*** Submitted:

Resolve port conflicts in cf-dea

R=benjamin.saller
CC=
https://codereview.appspot.com/107890043
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b