Juju 2.7-rc2, --client-only flag not implemented in add-k8s

Bug #1851849 reported by 🤖 prod-jenkaas-charmed-osm
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
High
Anastasia
2.7
Fix Released
High
Anastasia

Bug Description

Hello,

I am testing the juju 2.7-rc2 snap, and I found an issue:

First of all, if I add the k8s cloud with the following command, it doesn't work:

davigar15@Canonical:~/osm-dress-rehearshal$ microk8s.config | juju add-k8s k8s
This operation can be applied to both a copy on this client and to the one on a controller.
Do you want to add k8s cloud k8s to:
    1. client only (--client)
    2. controller "aws-osm-ha" only (--controller aws-osm-ha)
    3. both (--client --controller aws-osm-ha)
Enter your choice, or type Q|q to quit: ERROR EOF

With the --local flag it works, BUT, in the `juju add-k8s --help` command, it shows that the --local is deprecated:

```
--local (= false)
    DEPRECATED (use --client-only): Local operation only; controller not affected
```

It says to use --client-only instead, and this is what happens when doing that:
```
davigar15@Canonical:~/osm-dress-rehearshal$ microk8s.config | juju add-k8s k8s --client-only
ERROR option provided but not defined: --client-only
```

Using --client instead of --client-only is working:
```
davigar15@Canonical:~/osm-dress-rehearshal$ microk8s.config | juju add-k8s k8s --client

k8s substrate "microk8s/localhost" added as cloud "k8s".
You can now bootstrap to this cloud by running 'juju bootstrap k8s'.
```

I don't know if it is an issue in the help of the add-k8s, or an alias is missing.

Thanks,
David

Revision history for this message
David (davigar15) wrote :

Ups, it was me @davigar15

Revision history for this message
Ian Booth (wallyworld) wrote :

This is a typo in the help text for the deprecated --local option.

--client is what's required

(not --client-only)

Changed in juju:
milestone: none → 2.7.1
assignee: nobody → Anastasia (anastasia-macmood)
status: New → Triaged
importance: Undecided → High
Revision history for this message
Anastasia (anastasia-macmood) wrote :

This has been fixed as a drive-by in: https://github.com/juju/juju/pull/10892

Changed in juju:
status: Triaged → In Progress
Changed in juju:
milestone: 2.7.1 → 2.8-beta1
status: In Progress → Fix Committed
Revision history for this message
Anastasia (anastasia-macmood) wrote :
Harry Pidcock (hpidcock)
Changed in juju:
status: Fix Committed → Fix Released
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.