[ci][test] exclude model download time in server start time

[youkaichao]

fixes errors observed in https://buildkite.com/vllm/fastcheck/builds/3075#019182d5-5e41-408c-89e8-1d6bb86cc729

[github-actions]

👋 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 run fastcheck CI which consists a small and essential subset of CI tests to quickly catch errors. You can run other CI tests on top of default ones by unblocking the steps in your fast-check build on Buildkite UI.

Once the PR is approved and ready to go, please make sure to run full CI as it is required to merge (or just use auto-merge).

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

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

🚀

[DarkLight1337]

We should have ran entrypoints-tests before merging this. Since the model name is changed, test cases that check the model name now fail.

[mgoin]

snapshot_download can download many more files than we need for inference, such as duplicate .pt files when we prefer to use safetensors. We should try to use a common function with how vLLM usually pulls down files for models

[youkaichao]

can you come up with a fix for it?

I don't know if safetensors would be enough. but we can have atry.

[mgoin]

Basically I am proposing using DefaultModelLoader._prepare_weights

vllm/vllm/model_executor/model_loader/loader.py

Line 232 in ea9fa16

def _prepare_weights(self, model_name_or_path: str,

I can make a PR for this later today

[youkaichao]

please go ahead, I do see more files are downloaded in https://buildkite.com/vllm/fastcheck/builds/3094#01918525-98da-4596-8d31-f4e2c1172455