-
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
Can't deploy [email protected] using add_machine #1229
Comments
This might be an issue with the whole series to base migration? |
Using the base |
The migration has not begun in this library. The migration is already basically over in Juju, where the library uses the deprecated API fields. The Juju team will often ask for jRPC dump, which you get by configuring Python logging: logging.basicConfig(level="INFO")
logging.getLogger("juju.client.connection").setLevel("DEBUG") |
|
Looks like series is not transmitted on the rpc |
I guess either support for base= needs to be added or facade version restricted to some older facade where series= argument is still available. |
I think |
I'm confirming the above. The It has disappeared between 3.0.2 and 3.0.3: 3.0.2...3.0.3#diff-28797347270467fe3dcb301f7927c3b5eae6bce5732e7c59aa69bc10cc658845 What folks were getting was the default that given Juju model deployed on. I'm not perfectly clear, but I think this can be worked around by either of: juju model-config [email protected]
juju model-config default-series=noble There's always the alternative to shell out to juju cli to create machines for testing. P.S. I'm working on general series to base transition in #1182 but that's still going to take a while. |
@dimaqq Is correct, although that's less than ideal. |
Thanks for your input, I'll switch add-machine calls with juju cli calls instead for the moment. |
Description
Model.add_machine takes only a
series
parameters, when usingjammy
, everything works as expected, but when usingnoble
, the machine is still deployed as jammypython-libjuju/juju/model.py
Line 1469 in f87b50c
Urgency
Blocker for our release
Python-libjuju version
3.6.0.0
Juju version
3.6.0
Reproduce / Test
In a machine model supporting adding machines:
MAAS example
The text was updated successfully, but these errors were encountered: