Skip to content

gh #14: Log monitor for the callback functions#15

Closed
bhanucbp wants to merge 1 commit intodevelopfrom
feature/gh14_log_monitor_for_callbacks
Closed

gh #14: Log monitor for the callback functions#15
bhanucbp wants to merge 1 commit intodevelopfrom
feature/gh14_log_monitor_for_callbacks

Conversation

@bhanucbp
Copy link
Contributor

@bhanucbp bhanucbp commented Oct 7, 2024

Added file monitor class

@bhanucbp bhanucbp added the enhancement New feature or request label Oct 7, 2024
@bhanucbp bhanucbp requested a review from Ulrond October 7, 2024 18:00
@bhanucbp bhanucbp self-assigned this Oct 7, 2024
@bhanucbp bhanucbp linked an issue Oct 7, 2024 that may be closed by this pull request
from interactiveShell import InteractiveShell

class utFileMonitor:
"""This Class monitors the log files. If there are any new lines added to the file, then reports back
Copy link
Contributor

Choose a reason for hiding this comment

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

highlight that the class will monitor a file on the device connected to the session.

@@ -0,0 +1,119 @@
import sys
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Goal: For the class

  • Monitor for events coming into a file on the target.

  • Wait for an event to arrive, then inform the caller.

  • Set up an option start event marker, and end event marker.

  • if the full message is received within the timeout, then the callback is triggered

  • if the message timeout, then the callback is not triggered.

  • Each instance of the class will create a monitoring thread

    • In the case where multiple sessions are required, the session class must support multiple instance.

Copy link
Contributor

@Ulrond Ulrond left a comment

Choose a reason for hiding this comment

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

minor changes.

@@ -0,0 +1,119 @@
import sys
Copy link
Contributor

Choose a reason for hiding this comment

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

RDK (c) must be present.

Copy link
Contributor

Choose a reason for hiding this comment

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

add a module description on what it's for.

Copy link
Contributor

Choose a reason for hiding this comment

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

@Ulrond
Copy link
Contributor

Ulrond commented Oct 15, 2024

can you remove [ut_menu_navigator.py](https://github.com/rdkcentral/ut-raft/blob/main/ut_menu_navigator.py) as part of this change, it seems to have sneaked in on develop and just needs removing.

@CLAassistant
Copy link

CLAassistant commented Oct 21, 2024

CLA assistant check
All committers have signed the CLA.

@Ulrond
Copy link
Contributor

Ulrond commented Jun 24, 2025

@bhanucbp - this is still pending?

@kanjoe24 kanjoe24 closed this Dec 4, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2025
@kanjoe24
Copy link
Contributor

kanjoe24 commented Dec 4, 2025

Closing this as not required.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

Status: Resolved

Development

Successfully merging this pull request may close these issues.

utFileMonitor: Log monitor for the callback functions

4 participants