Connector.connect require controllers.yaml on 3.2 release

juju / python-libjuju

Python library for the Juju API

Apache License 2.0

57 stars 93 forks source link

Connector.connect require controllers.yaml on 3.2 release #1039

Closed jneo8 closed 3 months ago

jneo8 commented 3 months ago

The 3.2 release relay on controllers.yaml file for building connection

from juju.controller import Controller

async main():
  ctrl = Controller(max_frame_size=6**24)
  ctrl.connect(username=username, password=password, cacert=cacert, endpoint=endpoint)

Failed to connect to Juju controller at 10.5.0.189:17070: [Errno 2] No such file or directory: '~/.local/share/juju/controllers.yaml'

It's been fixed on a3a1c120f0cb3d8429f56bbec6a1b386c27385e8, need backporting for 3.2.

cderici commented 3 months ago

@jneo8 For pylibjuju we don't make releases for each track like Juju does. We have two tracks, 2.9, and 3.x. And the latest 3.x release is compatible with all Juju 3.y. As you mentioned, this fix is released, and you should be using the fix if you're using the latest pylibjuju. Did you pin pylibjuju to a particular version like 3.2?

jneo8 commented 3 months ago

If latest 3.x version python-libjuju is compatible with all juju 3.y, I think we can use the latest version of that, thanks.