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

feat: add configurable resource values to router deployment #34

Closed
wants to merge 7 commits into from

Conversation

0xThresh
Copy link
Contributor

No description provided.

@ApostaC
Copy link
Collaborator

ApostaC commented Jan 29, 2025

Hey @0xThresh, thanks for creating the PR.
Just for your information, can you sign-off the commit by doing this in your local branch:

git commit --amend --sign-off

and force push to your fork?

More details can be found here: https://github.com/vllm-project/production-stack/pull/34/checks?check_run_id=36331245500

@0xThresh 0xThresh force-pushed the feat-resource-values branch from c41c3bc to f76478e Compare January 29, 2025 23:56
@0xThresh
Copy link
Contributor Author

I'm working on fixing the pre-commit issues now, will have those pushed shortly

0xThresh and others added 7 commits January 29, 2025 17:02
Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: 0xThresh.eth <[email protected]>
Updated tutorial minimal example filename.

Signed-off-by: dmatch01 <[email protected]>
Signed-off-by: 0xThresh.eth <[email protected]>
Updated tutorial file name reference for values minimal example.

Signed-off-by: dmatch01 <[email protected]>
Signed-off-by: 0xThresh.eth <[email protected]>
* Add pre-commit workflow

* Add actionlint

* Add generic hooks

* Add black, isort, shellcheck

* Add requirements and markdown linting

* Add toml

* Add Dockerfile

* Add codespell

* Use Node.js version of `markdownlint`

* Add `requirements-lint.txt`

* Use CLI version of Node.js `markdownlint`

* Add `pre-commit` instructions to `Contributing`

* `pre-commit run -a` automatic fixes

* Exclude helm templates from `check-yaml`

* Comment hooks that require installed tools

* Make `codespell` happy

* Make `actionlint` happy

* Disable `shellcheck` until it can be installed properly

* Make `markdownlint` happy

* Add note about running pre-commit

---------

Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: 0xThresh.eth <[email protected]>
Signed-off-by: 0xThresh.eth <[email protected]>
@0xThresh 0xThresh force-pushed the feat-resource-values branch from bbf73df to 95a0a29 Compare January 30, 2025 00:02
@0xThresh 0xThresh closed this Jan 30, 2025
@0xThresh 0xThresh deleted the feat-resource-values branch January 30, 2025 00:06
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.

6 participants