Skip to content

try to test TLS engine config#1030

Merged
pjfanning merged 2 commits into
apache:mainfrom
pjfanning:copilot/update-application-conf-for-tls-engine-config
May 8, 2026
Merged

try to test TLS engine config#1030
pjfanning merged 2 commits into
apache:mainfrom
pjfanning:copilot/update-application-conf-for-tls-engine-config

Conversation

@pjfanning
Copy link
Copy Markdown
Member

@pjfanning pjfanning requested a review from He-Pin May 8, 2026 13:18
@pjfanning pjfanning merged commit 915446e into apache:main May 8, 2026
5 checks passed
pjfanning pushed a commit that referenced this pull request May 9, 2026
Motivation:
After #1030, the nightly job has been hanging for 5+ hours per matrix
entry until canceled (e.g. run 25588927669). The `run: |` literal block
turns each indented line into a separate shell command, so the first
line `sbt -D... -D...` launches sbt with no tasks and drops into the
interactive REPL, while the following `-D...` / `"++ ..." ... test`
lines never reach sbt.

Modification:
Switch the block scalar from `|` (literal, preserves newlines) to `>`
(folded, joins lines with spaces) so the whole sbt invocation runs as
a single command. Trailing whitespace on the first line is also
removed.

Result:
The nightly job runs sbt with all `-D` flags and the `test` task in
one process, restoring normal completion time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants