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

Docs

s/service_name/application_name/g

add_model() can accept just a model name

Make add_model() more user-friendly

Docs for connecting to a controller

Add more connect_ methods on Controller

Macaroon auth doc

Docs

Fix sphinx parsing errors

Start getting docs updated

Merge pull request #25 from juju/local-bundle-deploy

Add support for deploying local bundles

Merge pull request #28 from petevg/bug/fixes-for-null-cases

Bug/fixes for null cases

Merge pull request #26 from simonklb/get-metrics

Minor fix to get_metrics

addUnits no longer attempts to send [None] for `placement`.

This happens when we attempt to parse_placement on a null or blank
directive, and it leads to a null pointer error in the API. Instead, we
pass a simple None for `placement` if we don't have a placement
directive.

Fixed two "null case" bugs:

add_unit no longer fails when we pass "None" for our placement
directive.

checking leadership on a subordinate no longer fails, as we are more
careful about pull stuff out of the full status map.

Minor fix to get_metrics

json -> dict

Merge pull request #24 from juju/placement-fix

NoneType and style fixes for placement parsing lib

NoneType and style fixes for placement parsing lib

Add support for deploying local bundles

Get ModelInfo when Model connected