prettytable 0.6.0 test dependency has bad md5sum in PyPi

Bug #1000757 reported by Dan Prince
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Medium
Dan Prince

Bug Description

Keystone requires prettytable for testing. The most recent version of prettytable (0.6.0) seems to have a bad md5sum in PyPi:

Downloading/unpacking prettytable (from -r /tmp/tmp.WkJ62u5Tso/keystone_source/dist/keystone-2012.2/tools/test-requires (line 25))
  Using download cache from /home/smokestack/.pip/unittest_runner_cache/http%3A%2F%2Fpypi.python.org%2Fpackages%2Fsource%2FP%2FPrettyTable%2Fprettytable-0.6.tar.gz
  MD5 hash of the package http://pypi.python.org/packages/source/P/PrettyTable/prettytable-0.6.tar.gz#md5=c231839c06651bfe744a8cde3e9a7a5b (from http://pypi.python.org/simple/PrettyTable/) (10dca492a089b9a3c42194b2fa8cceda) doesn't match the expected hash c231839c06651bfe744a8cde3e9a7a5b!
Bad MD5 hash for package http://pypi.python.org/packages/source/P/PrettyTable/prettytable-0.6.tar.gz#md5=c231839c06651bfe744a8cde3e9a7a5b (from http://pypi.python.org/simple/PrettyTable/)

---

I don't think we actually need prettytable anymore so perhaps we can just remove it?

Dan Prince (dan-prince)
Changed in keystone:
assignee: nobody → Dan Prince (dan-prince)
importance: Undecided → Medium
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Revision history for this message
Pádraig Brady (p-draigbrady) wrote :

I downloaded the package from your link and http://pypi.python.org/pypi/PrettyTable/0.6,
and both had an md5 of c231839c06651bfe744a8cde3e9a7a5b

where is 10dca492a089b9a3c42194b2fa8cceda coming from?

Revision history for this message
Pádraig Brady (p-draigbrady) wrote :

Oh I see this file has the 10...ceda md5sum:
https://launchpad.net/debian/+archive/primary/+files/prettytable_0.6.orig.tar.gz

So the file must have been regenerated with the same name,
and you're getting a cached version or something?

I see the difference is the 10...ceda one forgot to include licensing info

Revision history for this message
Dan Prince (dan-prince) wrote :

Yes. I'm using caches so there is a bad version out there in the mirror servers.

And apparently we do need the dependency...

As an alternate approach I've nailed it to 0.5.0 for now.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/7538
Committed: http://github.com/openstack/keystone/commit/56492c23746f38c373d9ead5af244dfe17fdaf29
Submitter: Jenkins
Branch: master

commit 56492c23746f38c373d9ead5af244dfe17fdaf29
Author: Dan Prince <email address hidden>
Date: Thu May 17 09:51:07 2012 -0400

    Nail prettytable test dependency at 0.5.0.

    Prettytable 0.6.0 has a bad md5sum in PyPi and as such causes tests
    to fail when creating a new venv. Nail the version to 0.5.0.

    Fixes LP Bug #1000757.

    Change-Id: I121ba8d411e47c2fe154370d9c3c941e25331ce6

Changed in keystone:
status: In Progress → Fix Committed
Joseph Heck (heckj)
Changed in keystone:
milestone: none → folsom-1
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: folsom-1 → 2012.2
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.