Skip to content

RDKEMW-18857 : Remove RDKCM_RDKE from all workflows#177

Merged
srikanth-vv merged 5 commits into
developfrom
feature/RDKEMW-18857
Jul 9, 2026
Merged

RDKEMW-18857 : Remove RDKCM_RDKE from all workflows#177
srikanth-vv merged 5 commits into
developfrom
feature/RDKEMW-18857

Conversation

@balav08

@balav08 balav08 commented May 25, 2026

Copy link
Copy Markdown
Contributor

Reason for change: RDKCM_RDKE secret is introduced to access entservices-testframework when its private.
Now repo has been made as public. So, removing the RDKCM_RDKE secrets from all workflows and updated tag creation to on demand instead of automatic creation.
Replace RDKCM_RDKE token with RDKCM_DEPLOY_KEY in component-release.yml
Test Procedure: refer ticket.
Risks: Medium
Priority: P2
version: Patch

Reason for change: RDKCM_RDKE secret is introduced to access
entservices-testframework when its private. Now repo has been
made as public. So, removing the RDKCM_RDKE secrets from all workflows.
Test Procedure: refer ticket.
Risks: Medium
Priority: P2
version: Patch

Signed-off-by: balaji velmurugan <balaji_velmurugan@comcast.com>
Copilot AI review requested due to automatic review settings May 25, 2026 17:56
@balav08
balav08 requested a review from a team as a code owner May 25, 2026 17:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes the RDKCM_RDKE secret (previously needed to access the private entservices-testframework) from all workflows now that the repository is public. The secret is dropped from the L1 reusable workflow contract and its callers, from the native build job env, and from the component-release clone URL.

Changes:

  • Drop RDKCM_RDKE from tests-trigger.yml caller and from L1-tests.yml reusable-workflow secrets: block and its entservices-testframework checkout token.
  • Remove GITHUB_TOKEN: ${{ secrets.RDKCM_RDKE }} from the native_full_build.yml build step.
  • Replace the authenticated git clone URL in component-release.yml with an unauthenticated HTTPS clone.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/tests-trigger.yml Removes secrets: block passing RDKCM_RDKE to the reusable L1 workflow.
.github/workflows/L1-tests.yml Removes RDKCM_RDKE from workflow_call secrets and from the entservices-testframework checkout; cleans a stale commented token line.
.github/workflows/native_full_build.yml Removes the env: block that mapped GITHUB_TOKEN to RDKCM_RDKE.
.github/workflows/component-release.yml Switches the manual project clone to plain HTTPS without the embedded x-access-token.

Comment thread .github/workflows/component-release.yml Outdated
Comment thread .github/workflows/native_full_build.yml
@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Copilot AI review requested due to automatic review settings June 10, 2026 15:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread .github/workflows/component-release.yml Outdated
Comment thread .github/workflows/component-release.yml Outdated
Copilot AI review requested due to automatic review settings July 2, 2026 12:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/component-release-main.yml
@srikanth-vv
srikanth-vv merged commit 4677340 into develop Jul 9, 2026
10 of 11 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants