Skip to content

Conversation

@rajkamal-cv
Copy link
Contributor

Reason for change: The code changes in #123
are not meant for 'Sky' Gateways. Skip the respective models
Test Procedure: Check resolv.conf without RF and ensure brlan0 IP is updated Risks: Low
Priority: P1

Reason for change: The code changes in rdkcentral#123
    are not meant for 'Sky' Gateways. Skip the respective models
Test Procedure: Check resolv.conf without RF and ensure brlan0 IP is updated
Risks: Low
Priority: P1
@rajkamal-cv rajkamal-cv requested review from a team as code owners December 23, 2025 13:33
Reason for change: The code changes in rdkcentral#123
     are not meant for 'Sky' Gateways. Skip the respective models
Test Procedure: Check resolv.conf without RF and ensure brlan0 IP is updated
Risks: Low
Priority: P1
@rajkamal-cv
Copy link
Contributor Author

  1. Is this a Bug or a User Story (US)?

SHARMAN-3885 is a bug

  1. If it is a User Story:
    • Have all dependent PRs from other components been listed (if any)?
    • Does the commit message include both the User Story ticket and the Subtask ticket?
    • Will be all changes related to the User Story squashed and merged in a single commit?
    • Has the PR been raised only after completing all changes for the User Story (no partial changes)?
    • Has code development for the User Story been completed?
      • If yes, has the Gerrit topic or list of all dependent PRs across components (including meta-layer changes) been shared?
  2. Is there a validation log available in the Jira ticket for verifying builds with the updated generic-srcrev.inc across all platforms?
    • If yes, have the links to validation comments been shared?

Yes. Gerrit verification buids ( https://gerrit.teamccp.com/#/c/941773/ ) was triggered and the results are available here - https://ccp.sys.comcast.net/browse/SHARMAN-3885?focusedId=24308442&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-24308442

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