Re: Getting new tracker service code to go live

2010-01-03 Thread Thijs Kinkhorst
On snein 3 Jannewaris 2010, Michael Gilbert wrote:
 I've updated the sql logic to workaround a bug in lenny's aspw (and
 the code is actually now a bit cleaner...for sql anyway).  Please push
 this new commit to the live tracker.  

Ulib/python/security_db.py
Updated to revision 13701.


-- 
To UNSUBSCRIBE, email to debian-security-tracker-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Re: Getting new tracker service code to go live

2010-01-03 Thread Michael Gilbert
On Sun, 03 Jan 2010 13:51:59 + security tracker role wrote:

 The error message was:
 
 Traceback (most recent call last):
   File bin/update-db, line 73, in module
 warnings = db.calculateVulnerabilities(cursor)
   File 
 /srv/security-tracker.debian.org/website/secure-testing/lib/python/security_db.py,
  line 1159, in calculateVulnerabilities
 self._calcTesting(c, bug_name, 'oldstable', 'etch')
   File 
 /srv/security-tracker.debian.org/website/secure-testing/lib/python/security_db.py,
  line 1288, in _calcTesting
 (bug_name, suite, status, pkgs))
   File apsw.c, line 4238, in resetcursor
 apsw.ConstraintError: ConstraintError: constraint failed
 make: *** [all] Error 1

I can't reproduce this problem with the local tracker data that I have
(on lenny or sid). My best guess is that there is a corner case in
the live tracker where 'status' is not getting assigned, but I don't see
the hole for that to happen.

I've commited a fix that may resolve the issue.  It may also be that
the current database needs to be fully regenerated due to the new
code.  So maybe try deleting data/security.db and then running 'make
all'.

If not, can someone insert the following line above line 1285 in
security_db.py and send me the output of 'make all' on the live tracker?

print bug_name , suite , status , pkgs

Thanks,
Mike


-- 
To UNSUBSCRIBE, email to debian-security-tracker-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Re: Getting new tracker service code to go live

2010-01-03 Thread Nico Golde
Hi,
* Michael Gilbert michael.s.gilb...@gmail.com [2010-01-03 19:20]:
 If someone can push the latest updates, I think I've solved the
 problem with the latest commit.

I updated the tracker svn because I think your fix looks good. Apart from 
that... I know I haven't been very active recently, still I wonder why you 
need to implement undiscussed (excuse me if I missed this) tracker features 
and Thijs is blindly committing them. This is not how we should work in my 
opinion.

Cheers
Nico
-- 
Nico Golde - http://www.ngolde.de - n...@jabber.ccc.de - GPG: 0xA0A0
For security reasons, all text in this mail is double-rot13 encrypted.


pgptY8Z6lGf2v.pgp
Description: PGP signature


Re: Getting new tracker service code to go live

2010-01-03 Thread Nico Golde
Hi,
* Nico Golde n...@ngolde.de [2010-01-03 22:58]:
 * Michael Gilbert michael.s.gilb...@gmail.com [2010-01-03 19:20]:
  If someone can push the latest updates, I think I've solved the
  problem with the latest commit.
 
 I updated the tracker svn because I think your fix looks good.
 [...]

Or not, I can't sudo into the sectracker account as I just updated my ldap 
password and it seems to take some time before it updates on soler... So if 
anyone is fast, please pull the trigger.
Cheers
Nico
-- 
Nico Golde - http://www.ngolde.de - n...@jabber.ccc.de - GPG: 0xA0A0
For security reasons, all text in this mail is double-rot13 encrypted.


pgppTClnmvj6C.pgp
Description: PGP signature


Re: Getting new tracker service code to go live

2010-01-02 Thread Thijs Kinkhorst
On sneon 2 Jannewaris 2010, Michael Gilbert wrote:
 It appears that new commits to the tracker service do not
 automatically go live (based on the above syntax checker message
 recieved from sectrac...@soler.debian.org).  Anyway, can someone with
 appropriate permissions update the repo there

This is what I did:

sectrac...@soler:/srv/security-tracker.debian.org/website/secure-testing$ svn 
up
Ulib/python/security_db.py
Ulib/python/bugs.py
Ubin/tracker_service.py
Abin/generate-sequential-cve-ids
Ubin/test-web-server
Abin/generate-sequential-bugs
Ubin/check-new-issues
Updated to revision 13699.



Thijs


-- 
To UNSUBSCRIBE, email to debian-security-tracker-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Re: Getting new tracker service code to go live

2010-01-02 Thread Raphael Geissert
Thijs Kinkhorst wrote:
 svn up
 Ulib/python/security_db.py
 Ulib/python/bugs.py
 Ubin/tracker_service.py
 Abin/generate-sequential-cve-ids
 Ubin/test-web-server
 Abin/generate-sequential-bugs
 Ubin/check-new-issues
 Updated to revision 13699.
 
 

Which now completely broke it due to bad SQL.
Michael, please fix it soon
.

Regards,
-- 
Raphael Geissert - Debian Developer
www.debian.org - get.debian.net



-- 
To UNSUBSCRIBE, email to debian-security-tracker-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org