SEZ9 commented on PR #11721:
URL: https://github.com/apache/seatunnel/pull/11721#issuecomment-5390960152

   Thanks @DanielLeens for re-checking the CI status. Agreed on all points: the 
second attempt of run `32461786477` on `11599d94d` shows the earlier pip/TLS 
failure resolved, with only `Run / all-connectors-it-1 (8, ubuntu-latest)` 
cancelled mid-run — infra, not code, and a distinct flake from the one you 
diagnosed on the 21st. Your caution about not naming a specific known issue is 
right given the external cancellation. A rerun of just that job is the right 
next step; if it cancels again in the same spot, digging into whether it 
matches a tracked flake, as you offered, sounds good. If it comes back green, 
there's no blocker from my side.
   
   On the two previous findings, @SEPURI-SAI-KRISHNA's summary is correct:
   
   - **F1 (docs snippet divergence):** Resolved via the tracked deferral. The 
follow-up issue linked in comment `5367204778` satisfies the ask in comment 
`5366233534`, and the tracked rewrite of the snippet goes beyond it. No need to 
push the interim "simplified" label here — I agree it would be replaced almost 
immediately and isn't worth dismissing the approval over.
   - **F2 (non-negative-mod helper):** Withdrawn, and stays withdrawn. No 
action needed.
   
   Remaining ask: just the rerun result for the cancelled `all-connectors-it-1` 
job. Nothing further on F1/F2 in this PR.
   
   <!-- streview-comment:514 -->


-- 
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]

Reply via email to