Skip to content

CI: Improve Jenkins GitHub/GitLab integration#11147

Merged
dpressle merged 6 commits intoopenucx:masterfrom
Alexey-Rivkin:GH_Jenkins_jobs_fix
Jan 28, 2026
Merged

CI: Improve Jenkins GitHub/GitLab integration#11147
dpressle merged 6 commits intoopenucx:masterfrom
Alexey-Rivkin:GH_Jenkins_jobs_fix

Conversation

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor

@Alexey-Rivkin Alexey-Rivkin commented Jan 25, 2026

What?

Improve Jenkins CI to work reliably for both GitHub PRs and GitLab MRs.

Why?

  • Fork PRs fail with 404 when posting GitHub status
  • Codestyle checks only worked for GitLab, missing target branch info for GitHub
  • Job naming inconsistent between platforms

How?

  • Detect PR range from merge commit parents (GitHub) or target branch vars (GitLab)
  • Enable shallow clone for faster GitHub checkout
  • Rename GitLab jobs to ucx-*-gitlab for consistency with ucx-*-github
  • Wrap GitHub status updates in try/catch

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin Alexey-Rivkin force-pushed the GH_Jenkins_jobs_fix branch 2 times, most recently from 578db94 to 1c12c98 Compare January 26, 2026 13:14
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin Alexey-Rivkin force-pushed the GH_Jenkins_jobs_fix branch 2 times, most recently from e45a415 to 74921fb Compare January 26, 2026 14:57
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin Alexey-Rivkin changed the title CI/GH: Handle fork PR status updates CI: Improve Jenkins GitHub/GitLab integration Jan 26, 2026
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

1 similar comment
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

1 similar comment
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

Wrap GitHub status updates in try/catch blocks to make failures
non-fatal. Fork PRs cannot post status to the base repository,
causing 404 errors that would otherwise fail the entire job.
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

2 similar comments
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

Detect PR diff range using merge commit parents for GitHub PRs,
falling back to target branch variables for GitLab MRs.
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

2 similar comments
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

Signed-off-by: Alexey Rivkin <arivkin@nvidia.com>
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

2 similar comments
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

Signed-off-by: Alexey Rivkin <arivkin@nvidia.com>
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

5 similar comments
@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor Author

/build

@dpressle dpressle merged commit fc113a9 into openucx:master Jan 28, 2026
153 checks passed
@Alexey-Rivkin Alexey-Rivkin deleted the GH_Jenkins_jobs_fix branch January 28, 2026 17:16
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