Juju 3 is not supported

Bug #2021603 reported by Barry Price
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mojo: Continuous Delivery for Juju
Triaged
Critical
Unassigned

Bug Description

Attempting a deploy on my laptop, running the latest/stable Juju snap (3.2 at time of writing) gives:

2023-05-30 16:40:19 [ERROR] Unknown error
Traceback (most recent call last):
  File "/snap/mojo/141/lib/python3.6/site-packages/mojo/cli.py", line 692, in run_with_args
    args.func(args)
  File "/snap/mojo/141/lib/python3.6/site-packages/mojo/utils.py", line 389, in wrapped
    return method(*args, **kwargs)
  File "/snap/mojo/141/lib/python3.6/site-packages/mojo/cli.py", line 360, in run_from_manifest
    manifest.run(project, workspace, args.stage, args.interactive)
  File "/snap/mojo/141/lib/python3.6/site-packages/mojo/manifest.py", line 126, in run
    phase_name.run(project, workspace, stage)
  File "/snap/mojo/141/lib/python3.6/site-packages/mojo/phase.py", line 811, in run
    self._check_juju_version()
  File "/snap/mojo/141/lib/python3.6/site-packages/mojo/phase.py", line 1344, in _check_juju_version
    raise BundlePhaseException("Bundles require at least juju version 2.3+")
mojo.phase.BundlePhaseException: Bundles require at least juju version 2.3+

This is because the detection code here looks for both 2+ for the major version, and 3+ for the minor version - 3.2 actually supports everything we're trying to gate here, but the code doesn't know about it:

https://bazaar.launchpad.net/~mojo-maintainers/mojo/trunk/view/head:/mojo/phase.py#L1338

Related branches

Benjamin Allot (ballot)
Changed in mojo:
importance: Undecided → Critical
Revision history for this message
Barry Price (barryprice) wrote :

Just want to add that while fixing this particular specific check looks relatively simple, there are many, many version checks throughout the code and a whole suite of juju2-specific tests.

Adding juju3 support should really be done properly, if it's going to be added.

Loïc Gomez (kotodama)
Changed in mojo:
status: New → Confirmed
status: Confirmed → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.