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

Issue 104820043: Finished charm using services framework

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

Description

Finished charm using services framework https://code.launchpad.net/~johnsca/charms/trusty/cf-hm9000/services/+merge/221770 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 4

Patch Set 2 : Finished charm using services framework #

Total comments: 2

Patch Set 3 : Finished charm using services framework #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1094 lines, -881 lines) Patch
A [revision details] View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
D files/README.md View 1 chunk +0 lines, -3 lines 0 comments Download
D files/default-config.json View 1 chunk +0 lines, -27 lines 0 comments Download
A files/hm9000 View 0 chunks +-1 lines, --1 lines 0 comments Download
D files/hm9000.json.erb View 1 chunk +0 lines, -30 lines 0 comments Download
D files/hm9000_analyzer_ctl View 1 chunk +0 lines, -41 lines 0 comments Download
D files/hm9000_api_server_ctl View 1 chunk +0 lines, -40 lines 0 comments Download
D files/hm9000_evacuator_ctl View 1 chunk +0 lines, -40 lines 0 comments Download
D files/hm9000_fetcher_ctl View 1 chunk +0 lines, -41 lines 0 comments Download
D files/hm9000_listener_ctl View 1 chunk +0 lines, -44 lines 0 comments Download
D files/hm9000_metrics_server_ctl View 1 chunk +0 lines, -40 lines 0 comments Download
D files/hm9000_sender_ctl View 1 chunk +0 lines, -41 lines 0 comments Download
D files/hm9000_shredder_ctl View 1 chunk +0 lines, -41 lines 0 comments Download
D files/syslog_forwarder.conf.erb View 1 chunk +0 lines, -65 lines 0 comments Download
A hooks/cc-relation-changed View 1 chunk +5 lines, -0 lines 0 comments Download
M hooks/charmhelpers/contrib/cloudfoundry/common.py View 2 chunks +0 lines, -57 lines 0 comments Download
D hooks/charmhelpers/contrib/cloudfoundry/config_helper.py View 1 chunk +0 lines, -11 lines 0 comments Download
M hooks/charmhelpers/contrib/cloudfoundry/contexts.py View 1 1 chunk +59 lines, -55 lines 0 comments Download
D hooks/charmhelpers/contrib/cloudfoundry/install.py View 1 chunk +0 lines, -35 lines 0 comments Download
D hooks/charmhelpers/contrib/cloudfoundry/services.py View 1 chunk +0 lines, -118 lines 0 comments Download
D hooks/charmhelpers/contrib/cloudfoundry/upstart_helper.py View 1 chunk +0 lines, -14 lines 0 comments Download
M hooks/charmhelpers/contrib/openstack/context.py View 1 chunk +1 line, -1 line 0 comments Download
M hooks/charmhelpers/contrib/openstack/neutron.py View 1 chunk +17 lines, -1 line 0 comments Download
M hooks/charmhelpers/contrib/openstack/utils.py View 2 chunks +6 lines, -1 line 0 comments Download
M hooks/charmhelpers/contrib/storage/linux/lvm.py View 1 chunk +1 line, -1 line 0 comments Download
M hooks/charmhelpers/contrib/storage/linux/utils.py View 3 chunks +19 lines, -5 lines 0 comments Download
M hooks/charmhelpers/core/hookenv.py View 2 chunks +98 lines, -1 line 0 comments Download
M hooks/charmhelpers/core/host.py View 4 chunks +52 lines, -0 lines 0 comments Download
A hooks/charmhelpers/core/services.py View 1 2 1 chunk +357 lines, -0 lines 0 comments Download
A hooks/charmhelpers/core/templating.py View 1 chunk +51 lines, -0 lines 0 comments Download
M hooks/charmhelpers/fetch/__init__.py View 11 chunks +96 lines, -64 lines 0 comments Download
A hooks/config.py View 1 1 chunk +80 lines, -0 lines 0 comments Download
M hooks/config-changed View 1 chunk +5 lines, -2 lines 0 comments Download
A hooks/etcd-relation-changed View 1 chunk +5 lines, -0 lines 0 comments Download
M hooks/install View 1 chunk +43 lines, -8 lines 0 comments Download
A hooks/metrics-relation-changed View 1 chunk +5 lines, -0 lines 0 comments Download
A hooks/nats-relation-changed View 1 chunk +5 lines, -0 lines 0 comments Download
D hooks/relation-name-relation-broken View 1 chunk +0 lines, -2 lines 0 comments Download
D hooks/relation-name-relation-changed View 1 chunk +0 lines, -9 lines 0 comments Download
D hooks/relation-name-relation-departed View 1 chunk +0 lines, -5 lines 0 comments Download
D hooks/relation-name-relation-joined View 1 chunk +0 lines, -5 lines 0 comments Download
M hooks/start View 1 chunk +5 lines, -4 lines 0 comments Download
M hooks/stop View 1 chunk +5 lines, -7 lines 0 comments Download
M hooks/upgrade-charm View 1 chunk +5 lines, -6 lines 0 comments Download
M metadata.yaml View 1 2 chunks +6 lines, -9 lines 0 comments Download
D notes.md View 1 chunk +0 lines, -8 lines 0 comments Download
A templates/cf-hm9k-analyzer.conf View 1 1 chunk +17 lines, -0 lines 0 comments Download
A templates/cf-hm9k-api-server.conf View 1 1 chunk +17 lines, -0 lines 0 comments Download
A templates/cf-hm9k-evacuator.conf View 1 1 chunk +17 lines, -0 lines 0 comments Download
A templates/cf-hm9k-fetcher.conf View 1 1 chunk +17 lines, -0 lines 0 comments Download
A templates/cf-hm9k-listener.conf View 1 1 chunk +17 lines, -0 lines 0 comments Download
A templates/cf-hm9k-metrics-server.conf View 1 1 chunk +17 lines, -0 lines 0 comments Download
A templates/cf-hm9k-sender.conf View 1 1 chunk +17 lines, -0 lines 0 comments Download
A templates/cf-hm9k-shredder.conf View 1 1 chunk +17 lines, -0 lines 0 comments Download
A templates/hm9000.json View 1 2 1 chunk +31 lines, -0 lines 1 comment Download

Messages

Total messages: 7
johnsca
Please take a look.
11 years ago (2014-06-02 17:49:21 UTC) #1
benjamin.saller
Thanks for this, a little hard to verify as you indicated. It looks like we ...
11 years ago (2014-06-02 18:29:44 UTC) #2
johnsca
On 2014/06/02 18:29:44, benjamin.saller wrote: > https://codereview.appspot.com/104820043/diff/1/hooks/config.py#newcode24 > hooks/config.py:24: 'required_data': [contexts.NatsRelation(), > What do you ...
11 years ago (2014-06-02 19:43:47 UTC) #3
johnsca
Please take a look.
11 years ago (2014-06-02 22:14:06 UTC) #4
benjamin.saller
Thanks for this, since it depends on the helpers change lets resolve that before I ...
11 years ago (2014-06-04 21:02:23 UTC) #5
johnsca
Please take a look.
11 years ago (2014-06-05 18:17:46 UTC) #6
benjamin.saller
11 years ago (2014-06-06 16:45:31 UTC) #7
This LGTM,+1

https://codereview.appspot.com/104820043/diff/40001/templates/hm9000.json
File templates/hm9000.json (right):

https://codereview.appspot.com/104820043/diff/40001/templates/hm9000.json#new...
templates/hm9000.json:14: {% for unit in etcd -%}
nice, thanks
Sign in to reply to this message.

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