Opened 11 months ago

Closed 11 months ago

#649 closed defect (fixed)

Testing email on ticket manipulation

Reported by: rjsparks@nostrum.com Owned by:
Priority: medium Milestone:
Component: v3 vocabulary Version:
Keywords: Cc:

Description

Config had changed such that email wasn't getting sent. Seeing if that's been corrected.

Change History (13)

comment:1 Changed 11 months ago by rjsparks@nostrum.com

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [3984]:

Testing commit hook email. Fixes #649.

comment:2 Changed 11 months ago by rjsparks@nostrum.com

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:3 Changed 11 months ago by rjsparks@nostrum.com

  • Resolution set to fixed
  • Status changed from reopened to closed

Fixed in [3985]:

Testing commit hook email. Fixes #649.

comment:4 Changed 11 months ago by krathnayake@ietf.org

Do you have ​krathnayake@ietf.org in commit hook email list?
I didn't receive commit hook email for that commit.

comment:5 Changed 11 months ago by rjsparks@nostrum.com

  • Resolution fixed deleted
  • Status changed from closed to reopened

There's a definite issue with the trac ticket manipulation code that is keeping the trac-post-commit-hook from sending email when an svn commit alters a ticket.
The error in the log looks like this:

2021-06-24T16:03:02.264136-07:00 ietfa svn post-commit[20587]: Unexpected error while processing ticket ID 649: 'int' object has no attribute 'rsplit'

and it's coming out of a call into the Trac libraries, almost certainly this one:

                tn = TicketNotifyEmail(self.env)
                tn.notify(ticket, newticket=0, modtime=self.now)

I'll chase this further tomorrow.

comment:6 Changed 11 months ago by rjsparks@nostrum.com

From [4001]:

Testing svn-trac integration. Related to #649.

comment:7 Changed 11 months ago by rjsparks@nostrum.com

From [4002]:

Testing svn-trac integration. Related to #649.

comment:8 Changed 11 months ago by rjsparks@nostrum.com

From [4003]:

Testing svn-trac integration (next steps). Related to #649.

comment:9 Changed 11 months ago by rjsparks@nostrum.com

From [4004]:

Testing svn-trac integration (step 3). Related to #649.

comment:10 Changed 11 months ago by rjsparks@nostrum.com

From [4005]:

Testing svn-trac integration (step 4). Related to #649.

comment:11 Changed 11 months ago by rjsparks@nostrum.com

From [4006]:

Testing svn-trac integration (step 5). Related to #649.

comment:12 Changed 11 months ago by rjsparks@nostrum.com

From [4007]:

Testing svn-trac integration (step 6). Related to #649.

comment:13 Changed 11 months ago by rjsparks@nostrum.com

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.