Skip to content
This repository was archived by the owner on Sep 4, 2025. It is now read-only.

Conversation

@russellb
Copy link

(cherry picked from commit 740374d)

The ideal fix here is to upgrade to at least v0.5.4. In case that turns out to be not practical for some reason, I wanted to stage this backport for inclusion in the instructlab branch.

https://issues.redhat.com/browse/RHELAI-1211

@github-actions
Copy link

👋 Hi! Thank you for contributing to the vLLM project.
Just a reminder: PRs would not trigger full CI run by default. Instead, it would only trigger fastcheck CI to run, which consists only a small and essential subset of tests to quickly catch errors with the flexibility to run extra individual tests on top (you can do this by unblocking test steps in the Buildkite run).

Full CI run is still required to merge this PR so once the PR is ready to go, please make sure to run it. If you need all test signals in between PR commits, you can trigger full CI as well.

To run full CI, you can do one of these:

  • Comment /ready on the PR
  • Add ready label to the PR
  • Enable auto-merge.

🚀

@n1hility
Copy link

n1hility commented Oct 2, 2024

/lgtm

@n1hility
Copy link

n1hility commented Oct 2, 2024

/approve

@openshift-ci
Copy link

openshift-ci bot commented Oct 2, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dtrifiro, n1hility, russellb

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@n1hility n1hility merged commit e3f32d4 into opendatahub-io:instructlab Oct 2, 2024
1 of 2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants