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

Issue 52440043: Remove support for PyJuju and rapi from charm.

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 4 months ago by rharding
Modified:
10 years, 4 months ago
Reviewers:
mp+201510
Visibility:
Public.

Description

Remove support for PyJuju and rapi from charm. - Removes the support for running rapi/pyjuju. - Removes the agent used to communicate with zookeeper. - Removes anything pertaining to zookeeper. - Attempts to clean up docs and such to make sense with pure juju-core. Note: - The functional test run was taking 62min to complete for me. The timeout is bumped to 80min to help allow it to complete. This was against ec2. https://code.launchpad.net/~rharding/charms/precise/juju-gui/remove-pyjuju/+merge/201510 (do not edit description out of merge proposal)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+107 lines, -847 lines) Patch
M Dependencies.md View 1 chunk +1 line, -2 lines 0 comments Download
M Makefile View 1 chunk +1 line, -1 line 0 comments Download
M README.md View 4 chunks +4 lines, -26 lines 0 comments Download
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M config.yaml View 3 chunks +4 lines, -30 lines 0 comments Download
M config/haproxy.cfg.template View 1 chunk +3 lines, -7 lines 0 comments Download
D config/juju-api-agent.conf.template View 1 chunk +0 lines, -18 lines 0 comments Download
D config/juju-api-improv.conf.template View 1 chunk +0 lines, -12 lines 0 comments Download
M hooks/backend.py View 3 chunks +3 lines, -42 lines 0 comments Download
M hooks/utils.py View 15 chunks +16 lines, -141 lines 0 comments Download
M tests/20-functional.test View 12 chunks +19 lines, -72 lines 0 comments Download
M tests/helpers.py View 2 chunks +0 lines, -31 lines 0 comments Download
M tests/test_backends.py View 17 chunks +41 lines, -227 lines 0 comments Download
M tests/test_helpers.py View 3 chunks +0 lines, -71 lines 0 comments Download
M tests/test_utils.py View 15 chunks +13 lines, -167 lines 0 comments Download

Messages

Total messages: 1
rharding
10 years, 4 months ago (2014-01-15 00:02:29 UTC) #1
Please take a look.
Sign in to reply to this message.

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