Skip to content

Conversation

@Narayana-CT
Copy link
Contributor

@Narayana-CT Narayana-CT commented Dec 30, 2025

This pull request introduces automated YouTube streaming support for Android devices through youtube_android_test.py, integrating Android execution into the generic endpoint workflow and enabling device discovery using ADB interop data. The device and endpoint architecture has been restructured by centralizing device, resource, and port correlation logic, simplifying generic endpoint creation, and extending support to LANforge devices alongside real clients. Metric collection has been strengthened with improved port-matching logic for MAC, RSSI, link rate, and SSID data. WebUI handling has been consolidated into a single method, eliminating duplicate logic and ensuring consistent reporting of configured devices and OS summaries. Additional cleanup includes using absolute paths for Android test execution, removing legacy code paths, and enhancing documentation for improved clarity and maintainability.
Command to execute youtube_android_test.py
python3 /home/lanforge/lanforge-scripts/py-scripts/real_application_tests/youtube/youtube_android_test.py
--url https://youtu.be/-SQop2bI8Eg
--duration 1
--devices R9ZTC01333R,RZCTA09CTXF
--upstream_port 192.168.200.198
NOTE: This Command is executing through the generic endpoint created on lanforge device

- Initialize device-related tracking lists in Youtube constructor
- Remove legacy execute_youtube_test monitoring logic
- Simplify create_generic_endp interface by removing arguments

Signed-off-by: Narayana-CT <narayana.pinapatruni@candelatech.com>
…data

- Introduce get_android_device_data helper
- Fetch Android device details via /adb endpoint
- Map users to device serial numbers and resources
- Store serials and ports for downstream test setup

Signed-off-by: Narayana-CT <narayana.pinapatruni@candelatech.com>
…t device data flow

- Simplify create_generic_endp by removing resource and port discovery logic
- Add detailed docstring describing endpoint creation behavior and dependencies
- Move device, resource, and port correlation logic into get_device_data

Signed-off-by: Narayana-CT <narayana.pinapatruni@candelatech.com>
…device data handling

- Remove OS and hostname extraction from selection logic
- Add process_device_data to derive hostnames, OS types, and device counts

Signed-off-by: Narayana-CT <narayana.pinapatruni@candelatech.com>
…point execution

- Add Android-specific command execution using youtube_android_test.py
- Assign Android commands to the appropriate generic endpoints

Signed-off-by: Narayana-CT <narayana.pinapatruni@candelatech.com>
…UI updates

- Added generic endpoint creation for LANforge devices
- Centralized WebUI update logic into update_webui()
- Removed duplicate WebUI handling from main execution flow
- Improved port matching logic while collecting device metrics
- Updated Android test command to use absolute script path
- Minor cleanup and structural improvements for readability

Verified CLI: python3 lf_interop_youtube.py --mgr 192.168.204.75 --url https://youtu.be/-SQop2bI8Eg --duration 1 --res 1080p --upstream_port 192.168.200.198

Signed-off-by: Narayana-CT <narayana.pinapatruni@candelatech.com>
…devices

Signed-off-by: Narayana-CT <narayana.pinapatruni@candelatech.com>
Copy link
Collaborator

@smileyrekiere smileyrekiere left a comment

Choose a reason for hiding this comment

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

Appreciate the comments on each file in the review

@goyalsaurabh06 goyalsaurabh06 merged commit 60241dd into greearb:master Jan 8, 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.

3 participants