Merge lp:~andreserl/maas-images/lp1654063 into lp:maas-images

Proposed by Andres Rodriguez
Status: Rejected
Rejected by: Scott Moser
Proposed branch: lp:~andreserl/maas-images/lp1654063
Merge into: lp:maas-images
Diff against target: 26 lines (+4/-4)
1 file modified
conf/centos.yaml (+4/-4)
To merge this branch: bzr merge lp:~andreserl/maas-images/lp1654063
Reviewer Review Type Date Requested Status
Scott Moser Pending
Review via email: mp+323163@code.launchpad.net

Commit message

Remove minor version for CentOS Images.

CentOS images imported by MAAS are always the latest version. MAAS doesn't lock to a specific minor version. As such, removing the versioning from the image description.

To post a comment you must log in.
Revision history for this message
Scott Moser (smoser) wrote :

Hi, I'm rejecting this.
If the change is still necessary you should put it up for review against maas-images git branch.

Unmerged revisions

360. By Andres Rodriguez

Fix CentOS versioning to only include mayor version and not minor

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'conf/centos.yaml'
--- conf/centos.yaml 2016-09-17 00:28:38 +0000
+++ conf/centos.yaml 2017-04-25 18:34:14 +0000
@@ -9,9 +9,9 @@
99
10versions:10versions:
11 centos70:11 centos70:
12 version: 7.012 version: 7
13 path_version: 713 path_version: 7
14 release_title: "CentOS 7.0"14 release_title: "CentOS 7"
15 curtin_files: "{curtin_path}/centos7"15 curtin_files: "{curtin_path}/centos7"
16 packages:16 packages:
17 # CentOS 7 removed python-oauth which is required for cloud-init to17 # CentOS 7 removed python-oauth which is required for cloud-init to
@@ -21,8 +21,8 @@
21 - https://kojipkgs.fedoraproject.org/packages/python-oauth/1.0.1/10.el7/noarch/python-oauth-1.0.1-10.el7.noarch.rpm21 - https://kojipkgs.fedoraproject.org/packages/python-oauth/1.0.1/10.el7/noarch/python-oauth-1.0.1-10.el7.noarch.rpm
22 - linux-firmware22 - linux-firmware
23 centos66:23 centos66:
24 version: 6.624 version: 6
25 release_title: "CentOS 6.6"25 release_title: "CentOS 6"
26 curtin_files: "{curtin_path}/centos6"26 curtin_files: "{curtin_path}/centos6"
2727
28# vi: syntax=yaml nowrap28# vi: syntax=yaml nowrap

Subscribers

People subscribed via source and target branches