<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>8394</bug_id>
          
          <creation_ts>2015-09-24 21:25:28 +0000</creation_ts>
          <short_desc>patchwork: ensure that patches received during a PW server outage are processed</short_desc>
          <delta_ts>2022-05-12 15:25:00 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>6</classification_id>
          <classification>Yocto Project Subprojects</classification>
          <product>Patchwork/Patchtest</product>
          <component>Patchwork</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard> </status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>Future</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jose Lamego">jose.a.lamego</reporter>
          <assigned_to name="Unassigned">unassigned</assigned_to>
          <cc>benjamin.esquivel</cc>
    
    <cc>bluelightning</cc>
    
    <cc>damien.lespiau</cc>
    
    <cc>jose.perez.carranza</cc>
    
    <cc>mhalstead</cc>
    
    <cc>randy.macleod</cc>
          
          <qa_contact name="Apoorv">apoorv.sangal</qa_contact>
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>54652</commentid>
    <comment_count>0</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2015-09-24 21:25:28 +0000</bug_when>
    <thetext>All patches that are received at the mailing list during a Patchwork server outage (either planned or not) are obviously missing at Patchwork when the service is restored. When the service is available again, it continues processing any new patch email, but currently there is no automated process to be aware of such missing patches, nor to re-parse the emails, so they and all the subsequent related comments are not displayed in Patchwork.

Probably a daily-nightly job to review the mail list content could ensure that any missed patch is processed, or (if the resulting overhead is unacceptable) at least provide a report that could be used for a manual fix.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>55775</commentid>
    <comment_count>1</comment_count>
    <who name="Damien Lespiau">damien.lespiau</who>
    <bug_when>2015-10-26 16:53:10 +0000</bug_when>
    <thetext>Patches are delivered through SMTP, so if the patchwork server is down, there&apos;s some wiggle room as SMTP servers will try to deliver the message again later on.

Another option is to use a gmail account to receive mails from a mailing list and then fetchmail + procmail from the patchwork server to inject the mails into the system from the gmail account.

So, I don&apos;t think there&apos;s any actionable item here and I&apos;d close this bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>56208</commentid>
    <comment_count>2</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2015-11-04 18:45:25 +0000</bug_when>
    <thetext>In response to comment 1:

The option to wait for SMTP servers to finally deliver any message that didn&apos;t make it during a Patchwork server down time does not always work:

An example of a patch message sent during a Patchwork server down time is [1]. This patch is already merged in master [2], but since original message was not received, neither this, the subsequent messages nor the merge are displayed in patchwork.

Regarding the Gmail account option, we did have that in the past, but we confirmed that some emails were blocked by the email account provider due to high traffic considerations (reference in[3]), so we decided to use another one.

If I understand correctly, what you are suggesting is to use this Gmail account + fetchmail/procmail as a backup process for mail injection to ensure retrieval of any patch that was skipped during the &quot;main&quot; process. Am I right?

[1] http://lists.openembedded.org/pipermail/openembedded-core/2015-September/110947.html
[2] commit 4562f3f7713f836e6060a23357ff805c38983390
[3] https://bugzilla.yoctoproject.org/show_bug.cgi?id=7657</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>56212</commentid>
    <comment_count>3</comment_count>
    <who name="Damien Lespiau">damien.lespiau</who>
    <bug_when>2015-11-04 22:01:48 +0000</bug_when>
    <thetext>What I&apos;m saying is: as long as you have reliable mail delivery to the patchwork email address, you can use that mailbox as buffer until the patchwork service comes back up and start processing mails again. It&apos;s all on the side of mail configuration and how patches are delivered to parsemail.sh.

I don&apos;t think this problem is high priority, but your project may think differently.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>56213</commentid>
    <comment_count>4</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2015-11-04 22:23:25 +0000</bug_when>
    <thetext>In response to Comment 3:

Agree on the low priority of this issue given the few occurrences for a server outage. So we can prioritize other issues and when appropriate, return to this and evaluate if an alternate email delivery account/provider would be required.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>93261</commentid>
    <comment_count>5</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2022-05-12 15:25:00 +0000</bug_when>
    <thetext>WE are working off lore.kernel.org now</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>