Skip to content

Drop llm-d Private Pull Token & Release on Push#124

Merged
elevran merged 2 commits intomainfrom
llm-d-token-drop
May 20, 2025
Merged

Drop llm-d Private Pull Token & Release on Push#124
elevran merged 2 commits intomainfrom
llm-d-token-drop

Conversation

@vMaroon
Copy link
Member

@vMaroon vMaroon commented May 20, 2025

Summary

  • removed all private-llm-d repos pull support
  • removed image-build on push (gha CI)

Fixes #105

@vMaroon vMaroon requested review from elevran and nirrozenbaum May 20, 2025 07:57
@vMaroon vMaroon added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 20, 2025
- removed image-build on push (gha CI)

Signed-off-by: Maroon Ayoub <maroon.ayoub@ibm.com>
@vMaroon vMaroon force-pushed the llm-d-token-drop branch from 4684657 to 69dba78 Compare May 20, 2025 09:20
elevran
elevran previously approved these changes May 20, 2025
nirrozenbaum
nirrozenbaum previously approved these changes May 20, 2025
Copy link
Collaborator

@nirrozenbaum nirrozenbaum left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

Signed-off-by: Etai Lev Ran <elevran@gmail.com>
@elevran elevran dismissed stale reviews from nirrozenbaum and themself via 91b5199 May 20, 2025 11:58
@elevran elevran removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 20, 2025
@elevran elevran merged commit 55c58aa into main May 20, 2025
1 check passed
@elevran elevran deleted the llm-d-token-drop branch May 20, 2025 12:02
Jooho referenced this pull request in Jooho/llm-d-inference-scheduler Sep 17, 2025
Signed-off-by: konflux-internal-p02 <170854209+konflux-internal-p02[bot]@users.noreply.github.com>
Co-authored-by: konflux-internal-p02[bot] <170854209+konflux-internal-p02[bot]@users.noreply.github.com>
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.

Remove llm-d-kv-cache-manager Access Token

4 participants