Exception from "queue reject"

Bug #51742 reported by Scott James Remnant (Canonical)
12
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Celso Providelo

Bug Description

lp_archive@drescher:/tmp/keybuk$ queue reject devede
Initialising connection to queue new
Running: "reject devede"
Rejecting ubuntu/edgy (NEW) 1/21
---------------------------------------------------------------------------
Rejecting devede
Traceback (most recent call last):
  File "/srv/launchpad.net/codelines/current/scripts/ftpmaster-tools/queue", line 893, in ?
    sys.exit(main())
  File "/srv/launchpad.net/codelines/current/scripts/ftpmaster-tools/queue", line 875, in main
    cmd_runner.execute(single_args, exact_match)
  File "/srv/launchpad.net/codelines/current/scripts/ftpmaster-tools/queue", line 794, in execute
    wrapper.run()
  File "/srv/launchpad.net/codelines/current/scripts/ftpmaster-tools/queue", line 544, in run
    self.send_email(message)
  File "/srv/launchpad.net/codelines/current/scripts/ftpmaster-tools/queue", line 311, in send_email
    sendmail(mail_message)
  File "/srv/launchpad.net/codelines/soyuz-production/scripts/ftpmaster-tools/../../lib/canonical/launchpad/mail/sendmail.py", line 164, in sendmail
    assert 'to' in message and bool(message['to']), 'No To: header'
AssertionError: No To: header

Tags: lp-soyuz
Revision history for this message
Scott James Remnant (Canonical) (canonical-scott) wrote :

devede was queue id #60838

when rejected with -M one gets:
lp_archive@drescher:/tmp/keybuk$ queue reject -M devede
Initialising connection to queue new
Running: "reject devede"
Rejecting ubuntu/edgy (NEW) 1/21
---------------------------------------------------------------------------
Rejecting devede
Would be sending a mail:
   Subject: devede_1.6-0ubuntu1_source.changes Rejected
   Sender: Ubuntu Installer <email address hidden>
   Recipients:
   Bcc: Launchpad Archiver <lp_archive@localhost>
   Body:
Rejected:
devede 1.6-0ubuntu1 was REJECTED.
        Component: universe Section: utils

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Wed, 10 May 2006 00:33:48 +0200
Source: devede
Binary: devede
Architecture: source
Version: 1.6-0ubuntu1
Distribution: edgy
Urgency: low
Maintainer: DUNNEWIND Maxence <email address hidden>
Changed-By: DUNNEWIND Maxence <email address hidden>
Description:
 devede - Video DVD creator
Changes:
 devede (1.6-0ubuntu1) edgy; urgency=low
 .
   * Initial release
Files:
 a641a58da578aa2b460179483f6d4ff4 588 utils optional devede_1.6-0ubuntu1.dsc
 e2b5d69e51d8497d62c76b317d0a7a40 173633 utils optional devede_1.6.orig.tar.gz
 db4efc7abb9d41dacb28d10ccea69e50 3224 utils optional devede_1.6-0ubuntu1.diff.gz

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.2 (GNU/Linux)

iD8DBQFEpmdZld4FnY1/ypERApGLAJ9DF88TGir7T9ebki89AY5Xn01hoACgi8v3
BcL2UK/T+rhawcc4lJn9msw=
=es2U
-----END PGP SIGNATURE-----

===

If you don't understand why your files were rejected, or if the
override file requires editing, reply to this email.

---------------------------------------------------------------------------

Revision history for this message
Celso Providelo (cprov) wrote :

Mr. DUNNEWIND Maxence <email address hidden> isn't an LP valid user/email.

Use -A you@whatever to ensure To: field isn't empty (lp_archive@production will be Bcc:, no worries).

I will find a way to avoid this situation automatically. Thank you for reporting this.

Changed in qprocd:
assignee: nobody → cprov
importance: Untriaged → Medium
status: Unconfirmed → Confirmed
Revision history for this message
Celso Providelo (cprov) wrote :

workaround coming soon, from my wip `small-fixes`

Changed in qprocd:
status: Confirmed → In Progress
Revision history for this message
Celso Providelo (cprov) wrote :

RF 3819

Changed in qprocd:
status: In Progress → Fix Committed
Revision history for this message
Colin Watson (cjwatson) wrote :

For the record, the -A workaround doesn't work for rejects.

Revision history for this message
Celso Providelo (cprov) wrote : Re: [Bug 51742] Re: Exception from "queue reject"

Colin Watson wrote:
> For the record, the -A workaround doesn't work for rejects.

Yes, unfortunately, it's right.

In common agreement on IRC, we decided to manually apply the 4 lines patch in
drescher current codeline, it will unblock Kamion until tomorrow's rollout.

--
Celso Providelo <email address hidden>
Canonical Ltd - http://www.canonical.com

Celso Providelo (cprov)
Changed in qprocd:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.