Bug 1392257

Summary: job page: race condition between waiving and auto-fetch
Product: [Retired] Beaker Reporter: Dan Callaghan <dcallagh>
Component: web UIAssignee: beaker-dev-list
Status: CLOSED WONTFIX QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 23CC: mastyk, mjia
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-06-02 11:39:49 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Dan Callaghan 2016-11-07 01:39:32 UTC
Version-Release number of selected component (if applicable):
23.3

How reproducible:
timing sensitive so not readily

Steps to Reproduce:
1. Open the job page for a large job (e.g. workflow-selftest) which has some recipe sets finished but other still running (therefore the job page is still auto-fetching every 20 seconds)
2. Examine a completed recipe set and decide you want to waive it
3. Hit the Waive button in the modal at the same time as an AJAX request is in progress to fetch the job

Actual results:
Comment and Waived label appear against the recipe set briefly, but then when the AJAX request completes and the page is re-rendered the comment and Waived label will disappear again (because the server prepared the AJAX response before the Waiving change was committed).

Expected results:
Comment and Waived label should not disappear after they are submitted.

Comment 1 Martin Styk 2020-06-02 11:39:49 UTC
Hello,

thank you for opening issue in Beaker project.
This issue was marked with component "web ui".
As we are not planning to address any further issues in current UI, due to technical stack and not being able to work with Python 3 codebase, I'm closing this issue as WONTFIX.
New UI will be reimplemented within new versions of Beaker.

If you have any questions feel free to reach out to me.

Best regards,
Martin <martin.styk>