Fix streaming partial lines in overallLog#520
Conversation
Adopt AnnotatedLargeText#writeHtmlToFilter to do the heavy lifting of the fix.
|
This is ready for review, but tests will fail until I adopt the incremental from jenkinsci/jenkins#26871. I've hit a flakey run there and don't want to wait for CI to pass today. Will take another look tomorrow. |
|
What's missing for adopting the incremental? (Locally |
|
I had hoped https://ci.jenkins.io/job/Plugins/job/workflow-api-plugin/job/PR-520/3/execution/node/39/log/ would be sufficiently discoverable but apparently not. 😞 BTW please keep the PR in draft while consuming an incremental version, just as a reminder (it would not be mergeable in any case). |
|
Thanks for the hint!
Ah, that's a recent change from jenkins-infra/pipeline-library#991. I did not scroll all the way up and inspect the "successful" part of the pipeline. WDYT about failing the build at that echo step when an - echo 'Forbidding use...'
+ String msg = 'Forbidding use ...'
+ if (readFile('pom.xml').contains('-rc')) {
+ error msg
+ } else {
+ echo msg
+ }I can send a PR your way if you are happy with this approach. |
This PR is fixing the streaming of partial lines in the overallLog by adopt AnnotatedLargeText#writeHtmlToFilter to do the heavy lifting.
Testing done
See jenkinsci/jenkins#26871
Inspect network, see
consoleAnnotatorinmetamulti-part field of theprogressiveHtmlresponse when looking at the overall log.Submitter checklist