Apply suggestions from code review

Co-authored-by: Aaron Pham <[email protected]>

Author: okdshin

vllm/entrypoints/openai/cli_args.py

@@ -227,7 +227,8 @@ def make_arg_parser(parser: FlexibleArgumentParser) -> FlexibleArgumentParser:
         "--expand-tools-even-if-tool-choice-none",
         action="store_true",
         default=False,
-        help="[DEPRECATED] Include tool definitions in prompts "
+        deprecated=True,
+        help="Include tool definitions in prompts "
         "even when tool_choice='none'. "
         "This is a transitional option that will be removed in v0.10.0. "
         "In v0.10.0, tool definitions will always be included regardless of "

vllm/entrypoints/openai/serving_chat.py

@@ -181,7 +181,7 @@ async def create_chat_completion(
                   and not self.expand_tools_even_if_tool_choice_none):
                 assert request.tools is not None
                 if len(request.tools) > 0:
-                    logger.warning(
+                    logger.warning_once(
                         "Tools are specified but tool_choice is set to 'none' "
                         "and --expand-tools-even-if-tool-choice-none is not "
                         "enabled. Tool definitions will be excluded from the "