[deb] Wrong architecture format in generated Release files

Bug #1638628 reported by Alexandru Avadanii
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
packetary
Fix Released
Undecided
Alexandru Avadanii

Bug Description

Run packetary:

$ packetary --threads-num 10 clone \
  -t deb \
  -r ubuntu_mirror.yaml \
  -R ubuntu_packages.yaml \
  -d .

ubuntu_mirror.yaml:
- name: ubuntu
  path: /home/alav/fuel/build/f_isoroot/f_repobuild/nailgun/mirrors/ubuntu
  priority: null
  section:
  - main
  - multiverse
  - restricted
  - universe
  suite: trusty
  type: deb
  uri: http://mirror.zetup.net/ubuntu/

ubuntu_packages.yaml:
mandatory: exact
packages:
- name: acpi-support
[...]

Everything goes well, but the output artifacts have one small format problem:

$ grep Architectures: nailgun/mirrors/ubuntu/dists/trusty/Release
Architectures: x86_64

The correct format for "Architectures" is the same as `dpkg-architecture -qDEB_HOST_ARCH1`, i.e. in this case "amd64" instead of "x86_64".

To me, the rootcause looks like a missing translation step between the two formats in [1] for "repository.architecture".
I'll attach a patch for this; I can also submit it via gerrit if it looks ok.

[1] https://github.com/openstack/packetary/blob/master/packetary/drivers/deb_driver.py#L412

Revision history for this message
Alexandru Avadanii (alexandru-avadanii) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to packetary (master)

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

Changed in packetary:
assignee: nobody → Alexandru Avadanii (alexandru-avadanii)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to packetary (master)

Reviewed: https://review.openstack.org/392936
Committed: https://git.openstack.org/cgit/openstack/packetary/commit/?id=874d9bd9007cc917527b73ad14a50b53fa68a90e
Submitter: Jenkins
Branch: master

commit 874d9bd9007cc917527b73ad14a50b53fa68a90e
Author: Alexandru Avadanii <email address hidden>
Date: Thu Oct 27 22:24:28 2016 +0200

    deb_driver: Translate repository.architecture

    When generating repository metadata, the Release file gets the
    architecture list in an untranslated format, ending up with
    'x86_64' instead of 'amd64'.

    Closes-bug: 1638628

    Change-Id: I8471db6d54157175ff275c969ed5195d4616fa0f
    Signed-off-by: Alexandru Avadanii <email address hidden>

Changed in packetary:
status: In Progress → 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.