Skip to content

lf_wifi_capacity_test.py: Support CLI Argument Overrides with --load_old_cfg - #362

Merged
a-gavin merged 3 commits into
greearb:masterfrom
goyalsaurabh06:LISP-191-handle_overwirte_args
Aug 14, 2026
Merged

lf_wifi_capacity_test.py: Support CLI Argument Overrides with --load_old_cfg#362
a-gavin merged 3 commits into
greearb:masterfrom
goyalsaurabh06:LISP-191-handle_overwirte_args

Conversation

@tarun-candela

Copy link
Copy Markdown
Collaborator

Description:

  • This change helps to overrides the arguments for existing wifi test config when using --load_old_cfg.

Verified CLIs:

  • With override:
python3 ./lf_wifi_capacity_test.py --mgr 192.168.244.45 --load_old_cfg --config_name Test --pull_report --duration 60000
  • Without override:
python3 ./lf_wifi_capacity_test.py --mgr 192.168.244.45 --load_old_cfg --config_name Test --pull_report

…d test config

Verified CLI:

python3 py-scripts/lf_wifi_capacity_test.py --help

Signed-off-by: Tarun <tarunkumar.madabathula@candelatech.com>
Verified CLIs:

python3 ./lf_wifi_capacity_test.py --mgr 192.168.204.60 --load_old_cfg --config_name new_config --pull_report

Signed-off-by: Tarun <tarunkumar.madabathula@candelatech.com>
…_old_cfg

Verified CLI:

python3 ./lf_wifi_capacity_test.py --mgr 192.168.244.45 --load_old_cfg --config_name Test --pull_report --duration 60000

Signed-off-by: Tarun <tarunkumar.madabathula@candelatech.com>
@tarun-candela tarun-candela added Ready for review Ready for code review and feedback. Hold Use when changes or additional testing are still required before merging. and removed Ready for review Ready for code review and feedback. labels Aug 14, 2026
@tarun-candela tarun-candela added Ready for review Ready for code review and feedback. Hold Use when changes or additional testing are still required before merging. and removed Hold Use when changes or additional testing are still required before merging. Ready for review Ready for code review and feedback. labels Aug 14, 2026
@a-gavin a-gavin removed the Hold Use when changes or additional testing are still required before merging. label Aug 14, 2026

@a-gavin a-gavin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Per-internal feedback approving and merging w/ understanding future change may adjust this behavior.

@a-gavin
a-gavin merged commit 9f8ef4e into greearb:master Aug 14, 2026
2 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.

2 participants