ceph_apt_pin_packages default value is incorrect

Bug #1646855 reported by Miguel Alejandro Cantu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Low
Major Hayden

Bug Description

ceph_apt_pin_packages references "InkTank" as the originator of the packages.
According to the the apt_preferences man page:
"Origin:
    this line names the originator of the packages in the directory tree of the Release file. Most commonly, this is Debian. Specifying this origin in the APT preferences file would require the line:
Pin: release o=Debian"

I think it would be best to set this value to "origin download.ceph.com" instead of "release o=Inktank". I don't see Inktank referenced in the Release files:

http://download.ceph.com/debian-hammer/dists/trusty/
http://download.ceph.com/debian-infernalis/dists/trusty/
http://download.ceph.com/debian-jewel/dists/trusty/
etc..

I could mis-understanding the way apt_preferences work, so I apologize if this is an incorrect understanding.

Praveen N (praveenn)
Changed in openstack-ansible:
assignee: nobody → Praveen N (praveenn)
Revision history for this message
Logan V (loganv) wrote :
Revision history for this message
Jean-Philippe Evrard (jean-philippe-evrard) wrote :

FYI, it's happening in templates/ceph_pin.pref.j2

Revision history for this message
Jean-Philippe Evrard (jean-philippe-evrard) wrote :

Also ceph_apt_pinned_packages maybe worth a change.

Changed in openstack-ansible:
status: New → Confirmed
importance: Undecided → Low
tags: added: low-hanging-fruit
Revision history for this message
Jean-Philippe Evrard (jean-philippe-evrard) wrote :
Revision history for this message
Dale Dude (daledude) wrote :

This caused me an issue while upgrading from mitaka to newton. All my nodes were updated to ceph client 10 but I require ceph client 9 due to backward incompatibilities.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-ceph_client (master)

Fix proposed to branch: master
Review: https://review.openstack.org/416602

Changed in openstack-ansible:
assignee: Praveen N (praveenn) → Major Hayden (rackerhacker)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-ceph_client (master)

Reviewed: https://review.openstack.org/416602
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-ceph_client/commit/?id=5aea478fc34c0d271fd3d3871101de1fde6838cf
Submitter: Jenkins
Branch: master

commit 5aea478fc34c0d271fd3d3871101de1fde6838cf
Author: Major Hayden <email address hidden>
Date: Wed Jan 4 10:03:50 2017 -0600

    Set correct release for ceph pin

    This patch sets the correct release name for Ceph packages. It was
    originally `Inktank` but it has now changed to `RedHat`.

    The `ceph_pin_pref.j2` file was removed since it isn't used by any
    of the tasks in the role.

    Closes-Bug: 1646855
    Change-Id: If45055cdcaebbc47e6091bac84adf8bec00f8bec

Changed in openstack-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-ceph_client (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/416984

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-ceph_client (stable/newton)

Reviewed: https://review.openstack.org/416984
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-ceph_client/commit/?id=928a2f73d3f4a1b0f07899ffdc41dc59323d0f08
Submitter: Jenkins
Branch: stable/newton

commit 928a2f73d3f4a1b0f07899ffdc41dc59323d0f08
Author: Major Hayden <email address hidden>
Date: Wed Jan 4 10:03:50 2017 -0600

    Set correct release for ceph pin

    This patch sets the correct release name for Ceph packages. It was
    originally `Inktank` but it has now changed to `RedHat`.

    The `ceph_pin_pref.j2` file was removed since it isn't used by any
    of the tasks in the role.

    Closes-Bug: 1646855
    Change-Id: If45055cdcaebbc47e6091bac84adf8bec00f8bec
    (cherry picked from commit 5aea478fc34c0d271fd3d3871101de1fde6838cf)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-ceph_client 15.0.0.0b3

This issue was fixed in the openstack/openstack-ansible-ceph_client 15.0.0.0b3 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-ceph_client 14.0.7

This issue was fixed in the openstack/openstack-ansible-ceph_client 14.0.7 release.

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.