Error out when user provides own simplestreams url with no matches

Bug #1900439 reported by Simon Richardson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Expired
High
Unassigned

Bug Description

If a user defines their own simplestreams agent-metadata-url and we don't find a matching binary, then we fallback to one in the $PATH. This is very confusing and not what I expected to happen.

Instead we should fail hard and explain that what they've provided doesn't match what we expect.

John A Meinel (jameinel)
Changed in juju:
importance: Undecided → High
status: New → Triaged
Revision history for this message
Ian Booth (wallyworld) wrote :

This is by design - simplestreams uses a list of search paths. Just because a user provides their own URL, doesn't mean they always want to use *only* the entries from that URL. They may want to override only a specific version etc.

Is there are reason why the established behaviour is wrong? It's been documented that way since the start so we'll need to be careful if we want to change it.

Revision history for this message
Simon Richardson (simonrichardson) wrote :

I find the behavior confusing, if I do, others surely will!

Let's put it another way if you pass in both an agent-metadata-url and an agent-version I'm pretty sure you shouldn't be looking elsewhere! You're telling Juju to look here for the agent-version and it's not there and you go look somewhere else, then we bootstrap, but we do it with something they don't want.

We didn't find the agent in the metadata, we should error out fast rather than bootstrap with something they didn't want.

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

Think of it as like adding /usr/local/bin to your path. You don't want it to error and stop searching but rather move on to say /usr/bin if what you ask for isn't there.

A common use case is when someone wants to just test out a new focal ami - they don't want to have to copy the entire upstream simplestreams set of files just to plug in an override.

Changed in juju:
milestone: 2.9.1 → 3.0.0
Changed in juju:
milestone: 3.0.0 → 3.0.1
Changed in juju:
milestone: 3.0.1 → 3.0.2
Changed in juju:
milestone: 3.0.2 → 3.0.3
Revision history for this message
Juan M. Tirado (tiradojm) wrote :

Old issue. Reopen if needed.

Changed in juju:
milestone: 3.0.3 → none
status: Triaged → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for Canonical Juju because there has been no activity for 60 days.]

Changed in juju:
status: Incomplete → Expired
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.