Skip to content

Fail in case version file can't be donwloaded. This is an indication …#504

Open
percona-mhansson wants to merge 1 commit into
Percona-Lab:8.0from
percona-mhansson:8.0
Open

Fail in case version file can't be donwloaded. This is an indication …#504
percona-mhansson wants to merge 1 commit into
Percona-Lab:8.0from
percona-mhansson:8.0

Conversation

@percona-mhansson

Copy link
Copy Markdown

…that

something went seriously wrong and any subsequent errors will point in the wrong direction.

Comment thread jenkins/pipeline-parallel-mtr.groovy Outdated
Comment thread jenkins/pipeline-parallel-mtr.groovy Outdated
…that

something went seriously wrong and any subsequent errors will point in the
wrong direction.
@inikep

inikep commented Feb 12, 2026

Copy link
Copy Markdown
Collaborator

If you will merge this PR it will change pipelines for 5.7 (parallel), 8.0, 8.4, and 9.7.
Without testing we had to revert some changes in the past.
It would be good if you would test your changes on a temp pipeline.

@inikep inikep left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@inikep

inikep commented Feb 12, 2026

Copy link
Copy Markdown
Collaborator

@inikep

inikep commented Feb 16, 2026

Copy link
Copy Markdown
Collaborator

@percona-mhansson My new PR improves error reporting and also improves on top of your changes:
46c6d0d

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.

2 participants