Bug 10973 - changing state of patch fails?
Summary: changing state of patch fails?
Status: RESOLVED FIXED
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchwork (show other bugs)
Version: unspecified
Hardware: x86 Multiple
: Medium normal
Target Milestone: 2.3 M3
Assignee: Jose Lamego
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2017-01-26 07:26 UTC by Patrick Ohly
Modified: 2017-12-06 17:25 UTC (History)
2 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Ohly 2017-01-26 07:26:38 UTC
I'm logged in as "pohly" and looking at the patch https://patchwork.openembedded.org/series/4071/#

That patch has been merged, but the status is still "New". When changing the status to "Accepted" and clicking "Update", I get a blank page with https://patchwork.openembedded.org/series/4071/# as URL.

Reloading shows that the status hasn't changed.

Should I be able to change the status? If not, a proper error should be displayed.
Comment 1 Jose Lamego 2017-01-26 16:44:16 UTC
I'm looking into this. Issue is present at Series view only. Patches status and bundles can be edited at patch view.
Comment 2 Jose Lamego 2017-01-30 23:33:49 UTC
Two patches were developed and can be seen at:
https://lists.yoctoproject.org/pipermail/yocto/2017-January/034255.html
https://lists.yoctoproject.org/pipermail/yocto/2017-January/034256.html

these will be tested in patchwork-staging before implementing in production.
Comment 3 Jose Lamego 2017-01-31 22:15:50 UTC
I prepared a version 2 of the patches (fixing some javascript syntax errors), but the POST method used to update the patch status seems to be blocked by the nginx server (used both at patchwork-staging and production. I will consult Michael H. for options.
Comment 4 Jose Lamego 2017-02-07 20:24:38 UTC
Version 2 of patches were successfully implemented in patchwork-staging and can be seen at:
https://lists.yoctoproject.org/pipermail/yocto/2017-February/034396.html
https://lists.yoctoproject.org/pipermail/yocto/2017-February/034396.html