-
Notifications
You must be signed in to change notification settings - Fork 102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Local resources in sidecar charms do not work #550
Comments
Probably related to: |
cderici
added a commit
to cderici/python-libjuju
that referenced
this issue
Oct 4, 2021
jujubot
added a commit
that referenced
this issue
Oct 5, 2021
…s-550 #552 ### Description Problem is described in #550. This PR solves it by getting resources information from the deployed bundle (instead of setting it to empty as before). ### QA Steps ``` $ pyton3 -m tox -e py3 -- tests/unit/test_bundle.py all tests should pass $ python3 examples/deploy_local_bundle_with_resources.py Connecting to model unknown facade CAASModelConfigManager unexpected facade CAASModelConfigManager found, unable to decipher version to use unknown facade RaftLease unexpected facade RaftLease found, unable to decipher version to use unknown facade Secrets unexpected facade Secrets found, unable to decipher version to use unknown facade SecretsManager unexpected facade SecretsManager found, unable to decipher version to use unknown facade SecretsRotationWatcher unexpected facade SecretsRotationWatcher found, unable to decipher version to use Deploying bundle Waiting for active Successfully deployed! Removing bundle Disconnecting from model Success ``` ### Notes & Discussion Fixes #550
wolsen
added a commit
to wolsen/python-libjuju
that referenced
this issue
Dec 21, 2021
PR juju#600 updated the get_public_address method for units to look up the public address in the ApplicationFacade's UnitsInfo object. However, the change used the get(str) method of the Type class which performs a lookup in the json dictionary using the hyphenated key rather than the underscore key, thus always returns None since this key is not in the JSON dictionary. Changing it to return the lookup of 'public-address' instead of 'public_address' results in the unit's public address being returned. Fixes juju#550 Signed-off-by: Billy Olsen <billy.olsen@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello,
Deploying a bundle with local resources do not work.
Steps to reproduce:
The bundle:
juju status:
juju debug-log:
The text was updated successfully, but these errors were encountered: