<?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>9942</bug_id>
          
          <creation_ts>2016-07-14 15:16:42 +0000</creation_ts>
          <short_desc>patchwork: ensure patches are not marked as Accepted when a newer Series revision is modified</short_desc>
          <delta_ts>2017-12-06 17:03:13 +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>normal</bug_severity>
          <target_milestone>2.3 M1</target_milestone>
          <dependson>10153</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jose Lamego">jose.a.lamego</reporter>
          <assigned_to name="Jose Lamego">jose.a.lamego</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>leonardo.sandoval.gonzalez</cc>
    
    <cc>sgw</cc>
          
          
          <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>63987</commentid>
    <comment_count>0</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2016-07-14 15:16:42 +0000</bug_when>
    <thetext>Patches from a previous revision (v1) were marked as Accepted after two patches in a newer series revision (v2) were changed to Accepted state:


https://patchwork.openembedded.org/series/271/

Wrongly marked patches: 126549, 126551, 126553, 126555, 126557, 126559.

Find root cause and verify that there are no other affected series.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>65540</commentid>
    <comment_count>1</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2016-08-25 21:57:35 +0000</bug_when>
    <thetext>Attempted to replicate the issue at a staging instance with the following conditions:

- Same Patchwork commit as in OE-Patchwork (fc3c66a1ac3e94ce26ca29b4296f1196b869cbac)
- Created an initial patch Series from patch email messages created with create-pull-request script
- Created both a subsequent (v2) patch Series and a second revision (for original series) from patch email messages created with create-pull-request script
- Updated some patches from second revision/series

But reported behavior was not detected.
 
This seems to be a symptom and result of a bad series creation, which is fixed in bug 9904. After this fix is implemented, further testing will support this assumption.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>65938</commentid>
    <comment_count>2</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2016-09-08 17:07:59 +0000</bug_when>
    <thetext>This wont be ready for M3 due to the dependency (not met). Moving to M4.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67356</commentid>
    <comment_count>3</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2016-10-19 13:54:29 +0000</bug_when>
    <thetext>This bug is still depending on 10153 but I believe we can move both to 2.3 M1. 10153 is launching a PW testing instance so this is definitely something can be done post 2.2.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68990</commentid>
    <comment_count>4</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2016-12-09 17:07:49 +0000</bug_when>
    <thetext>Couldn&apos;t replicate issue at patchwork-staging instance with series 3895 [1], adding to the assumption that the reported issue was caused by a wrongly-created series, which is already fixed in production.

[1] https://patchwork-staging.openembedded.org/series/3895/</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>