VMBuilder

Merge lp://staging/~powersj/vmbuilder/jenkins_kvm into lp://staging/~ubuntu-on-ec2/vmbuilder/jenkins_kvm

  1. jenkins_kvm
  2. Merge into jenkins_kvm
Proposed by Joshua Powers
Status: Merged
Merged at revision: 835
Proposed branch: lp://staging/~powersj/vmbuilder/jenkins_kvm
Merge into: lp://staging/~ubuntu-on-ec2/vmbuilder/jenkins_kvm
Diff against target: 62 lines (+33/-19)
1 file modified
jenkins/CloudImages_Base_Release_Delta.sh (+33/-19)
To merge this branch: bzr merge lp://staging/~powersj/vmbuilder/jenkins_kvm
Reviewer Review Type Date Requested Status
Robert C Jennings (community) Approve
David Krauser (community) Approve
Review via email: mp+385215@code.staging.launchpad.net

Commit message

Fix the release, codename, and suite mappings.

The mappings are completely out of date and not correct at all for the last couple LTS. Furthermore, then script used to translate a suite (e.g. focal) to version is not working and even states that it would not work after the Z release.

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

Would it be better to use distro-info for this rather than hard coding a mapping that will be perpetually out of date?

Revision history for this message
Joshua Powers (powersj) wrote :

Looks like distro-info's "--fullname" gets us what we eventually use:

$ distro-info --series=bionic --fullname
Ubuntu 18.04 LTS "Bionic Beaver"

Except this runs on Jerff with version 0.8 and I do not see the --series flag till version 0.13.

Revision history for this message
David Krauser (davidkrauser) wrote :

Seems like a reasonable change, but are we still using this? What do we use it for?

Revision history for this message
Joshua Powers (powersj) wrote :

> Seems like a reasonable change, but are we still using this? What do we use it
> for?

https://cloud-images.ubuntu.com/releases/bionic/release/unpacked/release_notes.txt
https://cloud-images.ubuntu.com/releases/focal/release/unpacked/release_notes.txt

Revision history for this message
David Krauser (davidkrauser) :
review: Approve
Revision history for this message
Robert C Jennings (rcj) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches