Skip to content

RDK-60748: Add telemetry logging for iarmmgr with native build#170

Open
yuvaramachandran-gurusamy wants to merge 2 commits into
developfrom
feature/RDK-60748_2
Open

RDK-60748: Add telemetry logging for iarmmgr with native build#170
yuvaramachandran-gurusamy wants to merge 2 commits into
developfrom
feature/RDK-60748_2

Conversation

@yuvaramachandran-gurusamy
Copy link
Copy Markdown
Contributor

No description provided.

Update native_full_build.yml
Copilot AI review requested due to automatic review settings March 12, 2026 11:23
@yuvaramachandran-gurusamy yuvaramachandran-gurusamy requested a review from a team as a code owner March 12, 2026 11:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enables the existing “native full build” GitHub Actions workflow job for the iarmmgrs component by removing the hard-disabled condition, so CI will actually execute the native build steps on pushes/PRs.

Changes:

  • Removed if: false to allow the build-entservices-on-pr job to run.
  • Fixed the indentation/formatting of the cov_build.sh step line in the workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines 10 to 12
build-entservices-on-pr:
name: Build iarmmgrs component in github rdkcentral
if: false
runs-on: ubuntu-latest
Copy link

Copilot AI Mar 12, 2026

Choose a reason for hiding this comment

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

The PR title indicates adding telemetry logging for iarmmgr/iarmmgrs, but this change only enables the existing native build workflow (removes if: false). Please either update the PR title/description to match the actual change (CI enablement) or include the corresponding telemetry logging changes in this PR so the scope is consistent.

Copilot uses AI. Check for mistakes.
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.

2 participants