Feature/api version support (#109)

Set of changes that adds basic Facade Version support.

Instead of building a monolithic _client.py, we build a _client<version>.py for each integer Facade version, and put Facades there.

One issue is that we store the auxillary classes in a single, monolithic _definitions.py. We should probably refactor to split those up by Facade version. For now, they all seem to be backwards compatible, and all tests pass.

(I've tested against 2.0.2, 2.1.2, 2.2-alpha1 and 2.2-beta2.)

Also ninja'd in a call to the Pinger class, which should prevent timeouts in JaaS.

Fix exception messages and lint error

Bump rev of ghost charm in test to fix intermittent hook failure due to apt lock

Expanding controller.py with basic user functions, get_models and destroy (#89)

* Added user functions to controller.py, as well as the get_models call

* Fixes in controller.py

* Added controller.py integration tests

* Small fix in integration test

* Removed typo

* Disabled=False in controller.get_user does not work

* Added grant calls for model and controller, add-ssh, remove-ssh, get-ssh and get-machines

* Added last calls

* Added model.revoke('username')

* Edited docstring

* Added integrationtest for controller.py

* Fixed indentation error

* Added async

* Fixed type

* Fixed typo

* Expanded model integration tests

* Fixed typo

* ssh-key

* Typo

* Fix in remove_ssh_key

* Disconnecting models

* Added missing awaits

* Fixes

* Fixes

* Fixes

* Fixes

* Fixes

* Fixes

* Removed await in model grant

* Corrected remarks

* Typo

* Fixed tab error

* Typo

* Removed typo's

* Removed list from await expression

* Removed not needed await

* Added missing import

* Test

* Added decorators

* Merged #102

* Typo

* Updated _client.py

* Merged another

Added Monitor class to Connection. (#105)

Added Monitor class to Connection.

Calling Monitor.status allows a client to determine whether the
connection is "connected", "disconnected" or in an "error" state, and
take appropriate action as a result.

This is meant to address the issue of watchers failing silently when the
websocket connection goes away due to a network issue.

Support placement lists (#103)

Include resources from store when deploying (#102)

Allow underscore to dash translation when accessing model attributes (#101)

Added controller to ssh fix. (#100)

If a user has established a link to a controller other than the one in
"focus" locally, our ssh key workaround doesn't work. This change fixes
the issue by adding the controler name to the model name when appropriate.

Merge pull request #92 from juju/bug/schemagen

Regen schema to pick up missing APIs

Regen schema to pick up missing APIs

Due to some portions of the API living in a different location in the
Juju source, they were missed.  The schema gen code has been factored
out into its own repo (https://github.com/juju/schemagen) and the
missing APIs fixed.

Merge pull request #82 from juju/feature/examples-use-loop

Updated examples to use juju.loop

Updated examples to use juju.loop

Also updated the README.

Made a few other tweaks to make them more consistent in style.

Merge pull request #78 from juju/receiver-close

Don't raise ConnectionClosed error in receiver task

Don't raise ConnectionClosed error in receiver task

Merge pull request #76 from juju/bug/match-websocket-responses

Fix out-of-order websocket responses

Improve error handling

Added test for out of order receive and fix bug creating task

Refactor to use IdQueue pattern

Fix issue where we do not check to make sure that we are receiving the correct response.

Merge pull request #74 from petevg/bug/fix-invalid-annotations

Raise errors in a centralized fashion.

Merge branch 'master' into bug/fix-invalid-annotations

Merge pull request #63 from juju/feature/model_get_status

Fixes for conjure-up

Fixed up tests.

Unit test for series logic

Raise errors in a centralized fashion.

Fixes error with annotations that's exposed when we actually run tests
with the "raise errors" fix.

Added tests to verify that errors get raised.

Fixes based on review

Fix single app deploy not using config_yaml

Refactored to reduce duplicated code and ensure that both deploy paths
(bundle and single app) use new config_yaml param instead of config.

Improve constraints parsing for empty values

Retry calls to charmstore and increase timeout to 5s

Added integration test for explicit loop and threaded functionality

Make connect_model and deploy a bit more friendly

Fix self.loop reference in BundleHandler

Fix invalid model name if already contains username

Pass through event loop

There were several places where the default event loop was used instead
of the given event loop.

Fix model name not including user

The model names in models.yaml include the username prefix.

Also fixed a lint error.

Implement Model.get_status

Merge pull request #66 from petevg/bug/fix-landscape-deploy

Use config-yaml by default when deploying a bundle.

Merge pull request #68 from petevg/bug/fix-integration-tests-xenial

Explicitly set basepython=python3 for integration tests.

Explicitly set basepython=python3 for integration tests.

Fixes them on machines that don't have python3 set as the default.

Merge pull request #64 from juju/issues/27

Add integration tests.

Use config-yaml by default when deploying a bundle.

This invokes better/more modern behavior for blank strings in the
options for a given charm in the bundle.yaml. Specifically, it allows us
to deploy landscap-dense-maas without running into a KeyError.

This has been tested by using matrix to successfully deploy
landscape-dense-maas, using the code in this PR.

Add integration tests.

- Covers most of what is demonstrated in the examples/ directory.
- The integration tests are not run by default (with the 'tox' command),
  but they are run by TravisCI.
- Each integration test gets its own clean model in which to run, which
  is destroyed at the end of the test. See tests.base.CleanModel.
- Includes charmstore and local bundle tests (local bundle has machine
  and placement directives).

Fixes #27.

Update add_machine example in docs.

Add some missing constants and show how machines are destroyed.

Update changelog for 0.3.0

v0.3.0

Merge pull request #59 from juju/issues/58

Replace pass with NotImplementedError in method stubs

Replace pass with NotImplementedError in method stubs

This is arguably more user-friendly than silently passing when an
unimplemented method is called.

Fixes #58.

Merge pull request #56 from juju/issues/51

Implement Model.add_machine()

Enable parallel testing

Should help speed up tests, especially as more functional tests
are added in the future. Each can be run in parallel on a separate
model.

Also removed unused import from client/connection.py

Make Machine.destroy() block

Block the current coro until the machine is actually removed
from the remote model. This follows the same pattern as other
operations, e.g. Model.deploy().

Also clean up tests, making each functional test run on its
own clean model.

Add functional tests for Model.add_machine()

Clean up other tests, ensuring that connections are closed
and a new event loop is used for each test.

Update py.test args in tox.ini to properly on nocapture so
logging output can be seen when needed.

Correct MB/GB constants

Implement Model.add_machine()

Also fixes a bug that was causing the 'to' parameter to
Model.deploy() to not be handled correctly.

Add docs and examples for adding machines and containers
and deploying charms to them.

Fixes #51.

Merge pull request #55 from petevg/bug/fix-placement-docstring

Updated docstring for placement directive in python-libjuju.

Updated docstring for placement directive in python-libjuju.

Update changelog for 0.2.0

v0.2.0

Merge pull request #47 from petevg/feature/fix-juju-ssh

Dropped in fixes for juju ssh.

Simplified call to run_in_exector

ThreadPoolExecutor is the default, so we don't need to import it.

Better fix for moving stuff into an Executor.

Hides all the underpinnings in the utils class.

Create a new model for each test run of examples/add_model.py.

Moved execution of utils.read_ssh_key to a thread.

It's a blocking operation, so into a thread it goes!

Fixed error in add_model example.

We weren't actually awaiting the 'juju ssh' test, and had a mistake in
the command, to boot.

Dropped in fixes for juju ssh.

Workaround for https://bugs.launchpad.net/juju/+bug/1643076

This requires dropping to the cli client, but it does give us machines
that actually have the juju admin's ssh key.

Merge pull request #46 from juju/feature/loop-helper

Add loop helpers and simplify deploy example

Add loop helpers and simplify deploy example

Merge pull request #44 from juju/local-charm-bundle-support

Add support for local charms

Refactoring

- remove superfluous ensure_future() wrappers
- Make Model.deploy() call add_local_charm_dir() implicitly
  if it's passed a local charm dir path.

Add support for local charms

- Adds support for deploying local charms.
- Adds support for using local charm paths in bundles.

Merge pull request #43 from juju/feature/get-controller-cloud

Add ability to get cloud for controller

Add ability to get cloud for controller

Also use this as default when just a credential is given

Fix bug in Model.destroy_unit()

Merge pull request #40 from juju/deploy-errors

Detect errors in bundle deploy

Detect errors in bundle deploy

Add changelog to docs

Docs

Merge pull request #39 from juju/bug/show-controller

Fixed detecting controller for non-superuser

Fixed detecting controller for non-superuser

The `show-controller` command can only be used by superusers, so usually
fails on shared controllers.

https://bugs.launchpad.net/juju/+bug/1652106

v0.1.2

Include docs in distribution

v0.1.1

Add MANIFEST.in

Need VERSION file included in distribution. Fixes #38.

v0.1.0

Docs

Release prep

Add long description for pypi

Cleanup for first release

Docs

Add action example and docs

Merge pull request #37 from petevg/bug/36/fix-landscape-deploy

Fixes for landscape bundle.

Fixes for landscape bundle.

Parses constraints in some places where we weren't parsing them.

Normalizes keys in machine params.

Adds support for Placements on an lxd container *or* a machine.

Substitutes lxd containers for lxc containers.

Remove incorrect docstring

Unit.run() docs

Merge pull request #31 from juju/tweaks

Minor improvements

Minor improvements

* Sleep a small amount in block_until to avoid CPU-heavy busy-wait
* Better repr of ModelEntities for debugging
* Better errors when statuses accessed on dead units and applications

Merge pull request #34 from petevg/bug/fix-z-in-timestamp

Improved datetime parsing for unit.py

Improved datetime parsing for unit.py

Now uses dateutils lib to handle timezones, nanoseconds, and other
headaches.

Trivial bug fix

Relation docs