Skip to content

Fix: Remove hardcoded debug logs, fix test script and config - #130

Merged
aas008 merged 1 commit into
dev/reusable-components+benchmark-tunablesfrom
fix/remove-debug-logs-and-add-k8s-example
Jun 12, 2026
Merged

Fix: Remove hardcoded debug logs, fix test script and config#130
aas008 merged 1 commit into
dev/reusable-components+benchmark-tunablesfrom
fix/remove-debug-logs-and-add-k8s-example

Conversation

@aas008

@aas008 aas008 commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove hardcoded debug log write from cli/main.py that causes FileNotFoundError on any machine other than the original developer's
  • Suppress urllib3 InsecureRequestWarning and DeprecationWarning for cleaner output with self-signed cluster certs
  • Fix test_installation.py to use current TrialConfig API (study_name/trial_id instead of removed study_id)
  • Fix examples/study_config_local_exec.yaml: max_concurrentmax_concurrent_trials to match OptimizationConfig field name

Test plan

  • python test_installation.py passes
  • auto-tune-vllm validate --config examples/study_config_local_exec.yaml passes
  • K8s backend optimization runs without crashing on FileNotFoundError

Made with Cursor

- Remove hardcoded debug log writes from cli/main.py that cause
  FileNotFoundError on machines other than the original developer's
- Suppress urllib3 InsecureRequestWarning and DeprecationWarnings
- Fix test_installation.py to use current TrialConfig API (study_name/trial_id)
- Fix study_config_local_exec.yaml: max_concurrent → max_concurrent_trials
- Add examples/study_config_h200_k8s.yaml for direct K8s backend testing

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d8614ed6-dde6-403b-86b7-3d5814c244f3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/remove-debug-logs-and-add-k8s-example

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aas008
aas008 requested a review from thameem-abbas June 12, 2026 14:34
@aas008 aas008 self-assigned this Jun 12, 2026
@aas008
aas008 merged commit 15bac01 into dev/reusable-components+benchmark-tunables Jun 12, 2026
1 of 3 checks passed
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.

1 participant