Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Attaching resources is not implemented #294

Closed
aieri opened this issue Mar 13, 2019 · 2 comments · Fixed by #601
Closed

Attaching resources is not implemented #294

aieri opened this issue Mar 13, 2019 · 2 comments · Fixed by #601

Comments

@aieri
Copy link
Contributor

aieri commented Mar 13, 2019

the attach() function of class juju.application.Application is not implemented yet. This prevents charms that require resources from being deployed via libjuju.
@ajkavanagh
Copy link

+1

Yup, I need this too. I'm looking in how to do it.

@cderici
Copy link
Contributor

cderici commented Nov 23, 2021

Let's move this to the front lines.

Also we should probably have a binding for list-resources (the implementation is in get_resources)

python-libjuju/juju/application.py

Line 436 in a5ff411

async def get_resources(self):

cderici added a commit to cderici/python-libjuju that referenced this issue Dec 10, 2021
@cderici
Add attach-resource
940f32c 
Fixes juju#294
@cderici cderici mentioned this issue Dec 10, 2021
Merged
jujubot added a commit that referenced this issue Dec 13, 2021
@jujubot
Merge pull request #601 from cderici/attach-resources
9b9ef49 
#601

#### Description

This adds `attach-resource` functionality to pylibjuju.

Fixes #294

#### QA Steps

```sh
tox -e integration -- tests/integration/test_model.py::test_attach_resource
```

#### Notes & Discussion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[JUJU-244] Add attach-resource cderici/python-libjuju
3 participants
@cderici @ajkavanagh @aieri