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.

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

COU should check supported Juju version and raise proper exeption #476

Open
rgildein opened this issue Jul 2, 2024 · 0 comments
Open

COU should check supported Juju version and raise proper exeption #476

rgildein opened this issue Jul 2, 2024 · 0 comments

Comments

@rgildein
Copy link
Contributor

rgildein commented Jul 2, 2024

Right now running COU on 2.9.x failed with not obvious message, which tell user that current Juju is not supported.

For example, running COU against 2.9.35 you will see this output:

$ cou plan  
Full execution log: '/home/rgildein/.local/share/cou/log/cou-20240702172421.log'  
Connecting to 'admin/test' model... ✖
The connection was lost.
Check your connection or increase the timeout.
Default timeout is 10s; to increase to 60s for example:
$ COU_TIMEOUT=60 cou plan

After checking logs, you will see proper error:

$  cat /home/rgildein/.local/share/cou/log/cou-20240702172421.log
...
juju.errors.JujuAPIError: client incompatible with server 2.9.35 

I believe that COU should check the controller version and raise proper error with proper message. e.g.

$ cou plan  
Full execution log: '/home/rgildein/.local/share/cou/log/cou-20240702172421.log'  
Connecting to 'admin/test' model... ✖
Cliet version 2.9.35  is not supported, please upgrade to Juju 3.X
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant