You are viewing an old version of this page. View the current version.

Compare with Current View Page History

Version 1 Next »

The following are a few steps for doing Comcast Downstreaming:

1, Incoming changes for component owner review

https://code.rdkcentral.com/r/q/status:open

When a community contribution is submitted to CMF:
First we check that the BD, Copyright and Component Build are all OK.
Then we check that the test procedure is detailed enough for someone in Comcast to perform the tests with expected results. This could be either in the commit-msg or in the JIRA ticket and referenced in the commit-msg.
When the changes fulfils both of them, then we submit the change to review by component owner:

  • If the topic has not been set, then we set it to `comcast-review`, `lnt-review`, `lgi-review`, etc, dependent on the owner of the component.
  • Set the Review-State label to +1 if not set
  • Notify Pradeepta if the component is mastered in Comcast and needs downstreaming later.


2, Component Owner approved the change and downstream to Comcast

https://code.rdkcentral.com/r/q/label:Review-state%253D1+status:open
https://wiki.rdkcentral.com/display/CMF/Contributions+Pending+Review

When the component owner has approved a change by either voting CR +1 or CR +2.

If CR +1, then vote CR +2. Voting CR +2 .

2.1 If Review-State is set to +2 by set review state script

Vote the original changeset Review-State +4 and remove the RS +2 and +1 votes.

After the RS is set to 4, and the BD, Copyright, CB are +1 and CR is ++2, then it can be downstreamed to Comcast using the job: https://jenkins.cmf.code.rdkcentral.com/view/SCM/job/scm-downstream/

The `CHANGE` parameter is the Gerrit change number of the original change, and the `START` parameter is the latest sprint in Comcast.

Please check the latest sprint branch in Comcast during the last week of the month, the sprint branch for the next month is usually created in the second last weekend of the month.

Then run the job and successful, the RS label of the original change would be set to +5.

Then the old RS +4 vote can be removed and any `review` topic also removed. 

Notify Pradeepta an email about the downstream with the original changeset in CMF and the new changeset in Comcast Gerrit.

Note if the automatic downstream works then a slack notification is sent to the RDKM #cmf-downstreaming channel.


2.2 Automatic downstream failed due to conflict

If the automatic downstream failed due to a conflict you need to manually downstream the change


git clone <comcast Gerrit url>  --no-tags --single-branch -b <sprint branch> <project>

.
git clone https://gerrit.teamccp.com/rdk/rdkb/components/opensource/ccsp/MeshAgent/generic  --no-tags --single-branch -b 24Q3_sprint MeshAgent


3, Rework required

https://code.rdkcentral.com/r/q/(topic:%2522contributor-review%2522+status:open)+OR+(label:Review-state%253D-2+status:open)

If a downstreamed change needs reworking for whatever reason, e.g. comment / rejection from component owner in Comcast, test failures in Comcast, etc. Then:

  • The comment / reject reason / test report is communicated back to the contributor as a comment in the changes and also in the JIRA ticket.
  • Set the topic to `contributor-review` if it is not set by the original contributor.
  • Set the Review-Stat label to -1 or -2.

TIPS

  • Review State can be reset by a new patchset which has new code, in this case you may need to reset RS +1 , it is not reset on a trivial rebase or commit message change
  • If a downstream change is updated post downstream the scm set review state jenkins job will fail, you need to manually set RS+1
  • No labels