Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add workaround for pending gitlab benchmark jobs #2772

Merged
merged 1 commit into from
Apr 11, 2023

Conversation

ivoanjo
Copy link
Member

@ivoanjo ivoanjo commented Apr 11, 2023

What does this PR do?:

This PR adds a workaround for the GitLab benchmark jobs configuration that was causing them to show up on GitHub as "Pending" and never completing for every PR.

Screenshot without fix
image (6)

Motivation:

Currently the jobs don't run on every PR so this fixes the "Pending" issue.

Hopefully soon we'll start running the benchmarks on every PR and so we'll be able to remove this workaround.

Additional Notes:

Thanks @ddyurchenko for the help setting this so far.

How to test the change?:

Verify that you no longer see "Pending" jobs on the GitHub CI status.

**What does this PR do?**:

This PR adds a workaround for the GitLab benchmark jobs configuration
that was causing them to show up on GitHub as "Pending" and never
completing for every PR.

**Motivation**:

Currently the jobs don't run on every PR so this fixes the "Pending"
issue.

Hopefully soon we'll start running the benchmarks on every PR and
so we'll be able to remove this workaround.

**Additional Notes**:

Thanks @ddyurchenko for the help setting this so far.

**How to test the change?**:

Verify that you no longer see "Pending" jobs on the GitHub CI status.
@ivoanjo ivoanjo requested a review from a team April 11, 2023 15:29
Copy link
Contributor

@ddyurchenko ddyurchenko left a comment

Choose a reason for hiding this comment

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

LGTM ✔️

@ivoanjo ivoanjo merged commit c038cdd into master Apr 11, 2023
@ivoanjo ivoanjo deleted the ivoanjo/fix-pending-gitlab-jobs branch April 11, 2023 16:10
@github-actions github-actions bot added this to the 1.11.0 milestone Apr 11, 2023
@lloeki lloeki modified the milestones: 1.11.0, 1.11.0.beta1 Apr 14, 2023
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