Update VERSION and changelog for release

Fix test failures (#163)

* Update tox.ini to enable testing with other Py envs

* Remove invalid failing test

Fixes #162

* Fix error type when connecting

* Fix KeyError on model name during integration tests

* Fix model name collision

* Gracefully downgrade when controller supports newer facades than we do

Fixes #145

removing cli command to add ssh keys (#161)

* removing cli command to add ssh keys

* add ssh-key after model-creation

* awaiting method

* Update controller.py

* adding user ssh-key when model is created

* fixed wrong config placement

Make Application.upgrade_charm upgrade resources (#158)

* Add Resource to client overrides due to missing 'origin' field
* Make Application.upgrade_charm upgrade resources
* Add test_upgrade_charm_resource
* upgrade_charm: Only call the charm store once

Expand integration tests to use stable/edge versions of juju (#155)

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

Docs are build automatically by RTD, so remove from `make release`

Update README to point to RTD for docs instead of PythonHosted.org

Update docs requirement to fix RTD build

Update docs requirement to fix RTD build

Bump VERSION and changelog for release

Implement scp functionality (#149)

* Redefine SCP interface

* Implement Machine.scp_to and Machine.scp_from

* Implement Unit.scp_to and Unit.scp_from

* Fix Unit.machine to better handle missing machines

* Use os.path.expanduser for scp key path

* Use local-cloud address if public address is not found

* Rename Machine.public_address to Machine.dns_name to match CLI

* fix scp tests not waiting long enough

Add Unit.public_address property

Adds support for getting/setting config on a model (#152)

* Adds support for getting/setting config on a model

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

Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* Support ConfigValue type for interacting with model config

Bump version and update changelog for release

Improve handling of closed connections (#148)

* Prevent circular reference between Monitor and Connection

* Improve monitor status logic

* Stop watcher task cleanly on disconnect

Instead of the watcher task blowing up with an exception when the
connection is closed, it should exit cleanly and allow the Monitor to
report the connection error.

Fixes #147

* Raise ConnectionClosed exception in Model.block_until

Rather than blocking indefinitely when waiting on a model change that
will never happen if the connection gets closed out from under it, this
makes `Model.block_until` raise a `websockets.ConnectionClosed` error so
that it can be caught and dealt with by the client.

* Automatically reconnect lost websocket connections

The receiver or all_watcher should immediately reconnect a lost
connection, or it will be reconnected automatically upon issuance of the
next RPC call.  However, there is still a small chance that the
disconnect could happen between sending a API call and receiving the
response, in which case a websockets.ConnectionClosed error will still
be raised to the caller.  These should be quite rare, though.

* Gracefully handle add_signal_handler failing in a thread in loop.run

* Restart AllWatcher if controller stops it

Fixes conjure-up/conjure-up#965

* Explicitly let the controller know we're stopping the watcher

* Skip reconnect if close was requested

Configurable and larger max message size (#146)

The default max message / frame size for the websocket library is 1MB.
However, the initial AllWatcher response for large models can exceed
that.  Increased the default to 4MB and made it configurable.

Fixes #136

Bump rev for release

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.