rebase #138
Merged
rebase #138
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
) Reason for change: During initial setup, the device is attempting to download the SAT token and timing out, which is holding up the ctrlm message queue processing, which allows BLE pairing to fail Test Procedure: boot and attempt to pair BLE remote, keeping an eye on ctrlm loggging. Look for "CTRLM : ERROR: call_plugin: Thunder call failed <getServiceAccessToken> <11>, attempt 1 of 1" We expect BLE pairing to succeed rather than timeout. If the call_plugin log error occurs and BLE pairing succeed then the code change has been exercised and the issue is resolved. Risks: Low Priority: P1 Signed-off-by: Jason Thomson <jason_thomson@comcast.com>
When an RDK device is factory reset, controlMgr will send a message to the remote to also factory reset itself. Once controlMgr gets notified from the remote of a successful RCU factory reset, it needs to be requested to bluez to remove the device. This prevents a connection attempt from happening to the just factory-reset RCU before the RDK device reboots. This connection attempt will prevent the RCU from autopairing during the activation flow after the RDK factory reset.
* RDKEMW-8133: Optional param name for voiceSessionRequest
Reason for change: Adding optional name param for voiceSessionRequest
which is needed to track metadata about voice
sessions from various ipcontrol clients
Test Procedure: Use VoiceControl voiceSessionRequest method with name
param
Risks: Low
Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com>
* RDKEMW-8133: Optional param name for voiceSessionRequest
Reason for change: remove the required conditional of name
Test Procedure:
Risks:
Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com>
* RDKEMW-8133: Optional param name for voiceSessionRequest
Reason for change: Clean up log messaging
Test Procedure:
Risks:
Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com>
* RDKEMW-8133: Optional param name for voiceSessionRequest
Reason for change: Move obj != NULL block to prevent null dereference
Test Procedure:
Risks:
Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com>
---------
Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com>
* RDKEMW-8354: ctrlm-main crash while holding standby during OTA Reason for change: crash due to null reference on repeating timer event Test Procedure: see ticket Risks: low Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com> * RDKEMW-8354: ctrlm-main crash on timer Reason for change: add comment for clarity Test Procedure: Risks: Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com> --------- Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com>
Deploy cla action
Deploy fossid_integration_stateless_diffscan_target_repo action
…t least 1 code." (#124)
…1 code. (#125) It can often happen that the IRDB returns success but no codes are returned, this leads to confusing UI screens. So even if the IR database returned successfully, only return success to the plugin API if there is at least 1 code is present
* RDKEMW-8929: Refactor ctrlm_voice_ipc_t to inherit ctrlm_ipc_iarm_t Reason for change: Inherit ctrlm_ipc_iarm_t Test Procedure: Verify behavior of events before & after no diff Risks: Low Signed-off-by: Kelvin Lu <Kelvin_Lu@comcast.com>
if BLE pairing fails, retry 3 times or up to the pairing timeout (currently configured at 20 seconds), whichever comes first.
egalla204
merged commit 828ec71
into
feature/RDKEMW-9474_multiple_IRDBs
12 of 13 checks passed
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.