Opened 9 years ago

Closed 8 years ago

#1307 closed defect (fixed)

mail from failed jobs is highly incomplete

Reported by: dlove Owned by: Dave Love <d.love@…>
Priority: normal Milestone:
Component: sge Version: 6.2u5
Severity: minor Keywords:
Cc:

Description

Admin mail from failed jobs just contains the

Shepherd pe_hostfile:

information, which is a regression at least from 6.0.
The logic in admin_mail.c looks wrong, and I'm not sure what
the semantics are of the double use of sge_mail_body_total
in the sprintf:

      strcpy(sge_mail_body_total, sge_mail_body);

      
      for (i=0; i<num_files; i++) {
         if (shepherd_files[i].exists) {
            sprintf(sge_mail_body_total, "%s\nShepherd %s:\n", 
                      sge_mail_body_total, shepherd_filenames[i]);

Attachments (1)

admin_mail.c.diff (752 bytes) - added by dlove 9 years ago.
potential fix (untested)

Download all attachments as: .zip

Change History (4)

comment:1 Changed 9 years ago by dlove

By coincidence, I just got another one where the user etc. were present,
but the hostfile wasn't.

Changed 9 years ago by dlove

potential fix (untested)

comment:2 Changed 9 years ago by dlove

Partially fixed by [3698]. The shepherd error and trace files still aren't
being printed, or perhaps are empty when the mail is sent.

comment:3 Changed 8 years ago by Dave Love <d.love@…>

  • Owner set to Dave Love <d.love@…>
  • Resolution set to fixed
  • Status changed from new to closed

In [3999/sge]:

(The changeset message doesn't reference this ticket)

Note: See TracTickets for help on using tickets.