Fix deploying non-stable channels and explicit revs (#144)

* Fix deploying non-stable channels and explicit revs

Fixes #143

* Bump timeout on status test

* Minor docstring fix

Bump VERSION and changelog for release

Patch/update schema juju 2.3 alpha1 (#142)

* Adds latest 2.3 alpha1 schema

Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* Update doc to be consistent with existing schema filenames

Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* Update python code from `make client`

This verifies that integration tests pass locally on Juju 2.3-alpha1

Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>

Improve API doc navigation and coverage (#141)

* Remove unused requirements.txt

* Clean up generated doc files

* Improve API doc coverage and navigation

Fixes #139

Add type info to Model.add_machine docs (#138)

Resolves #137

Bump VERSION and changelog for release

Changes from docs build

Fix formatting in docs

Add machine status properties (#133)

* Add machine status properties

Add properties to access the machine provisioning and Juju machine agent
status information.

Unfortunately, there seems to be a bug with the AllWatcher related to
these fields (https://bugs.launchpad.net/juju/+bug/1695335) that causes
the deltas to contain stale data, so this also includes a hack to work
around that by fetching newer data from the FullStatus API.

* Fixed test for AWS

Add model context manager (#128)

Implement Application.upgrade_charm method (#132)

* Implement Application.upgrade_charm method

* Use model.block_until instead of a direct loop on asyncio.sleep

* Raise JujuError if upgrade_charm is a noop

Same behavior as Juju CLI here.

* Add integration tests for upgrade_charm

* Fix upgrade_charm hangs forever if switch arg does not start with 'cs:'

Bump version and changelog for release

Accept new / unknown API fields gracefully (#131)

* Accept new / unknown API fields gracefully

From discussions in Montreal, it seems that new fields in the Juju
schema definitions are expected to be handled gracefully even without a
bump to the facade version. That is, facade versions will only be bumped
if a field is removed or changed.

Fixes #130

* Update schema and generated code for juju-2.2-rc1

The agent-version field that was added to ModelInfo is of a type that
requires some special parsing and serialization logic.  This type was
used before but not in an API that we were apparently exercising.

s/pip/pip3/ in install instructions (#129)

Strip local:-prefix from local charm urls (#121)

Bump version and changelog for release

Support (and prefer) per-controller macaroon files (#125)

Juju 2.2-beta4 moved to per-controller macaroon files, which breaks auth.

Fixes #124

Add missing version var (used in release target)

Update version and changelog for 0.4.1 release (#119)

Also add tagging logic to release target in Makefile

Remove VERSION_MAP and rely on facade list from controller (#118)

Based on feedback from core, the facades list is complete and accurate,
but does depend on the connection type.  So, we don't need to maintain
our own mapping of it.

Tested with Juju 2.1.2 and Juju 2.2-beta3

Refactor connection task management to avoid cancels (#117)

Connections in libjuju require a few tasks to help manage the data
flowing through it.  The model AllWatcher task, the connection packet
receiver, and the connection keep-alive pinger.  Due to how these
connections were being managed, they had to be tracked and cancelled
externally, leading to exceptions being masked and the need to shield
AllWatcher notifier callbacks, making them entirely uncancellable.  This
refactor cleans that up by using events to track when the connection is
being shutdown and cleanly stopping the tasks internally.

Refactored login code to better handle redirects (#116)

Fixes #114: build_facades not handling discharge-required results
Fixes #115: JAAS update broke controller connections

Also ensures that the receiver and pinger tasks get cleaned up properly
when a connection is closed.

Also makes the model AllWatcher share the model connection to reduce the
number of open connections required.  The independent connection is no
longer needed since the websocket responses are properly paired with the
requests.

Update changelog for 0.4.0

Bump version for release

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