Yikun commented on pull request #35179:
URL: https://github.com/apache/spark/pull/35179#issuecomment-1011881076


   @HyukjinKwon After this PR, looks like update_build_status work unexpectly. 
Such as this PR: https://github.com/apache/spark/pull/35183, it is stucked in 
"queued".
   
   I also take a look on it [raised 404 in 
update_build_status](https://github.com/apache/spark/runs/4800075844?check_suite_focus=true#step:2:231),
 it was wrong to get the run_id on 
https://api.github.com/repos/Yikun/spark/actions/runs/4797718278, but the right 
api should be https://api.github.com/repos/Yikun/spark/actions/runs/1690601631.
   
   There are 2 potential workaround I guess:
   
   Workaround 1, still return the old runID in here:
   
https://github.com/apache/spark/blob/21ad3da08214980f776f17c605774a249192ed4a/.github/workflows/notify_test_workflow.yml#L125
   
   Workaround 2, Get right action id according new runID
   
https://github.com/apache/spark/blob/21ad3da08214980f776f17c605774a249192ed4a/.github/workflows/update_build_status.yml#L68


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to