Skip to content

Initial folder and SPEC file creation for PMA#327

Merged
sfonn merged 18 commits into3.0-devfrom
pma-dev
Jul 31, 2025
Merged

Initial folder and SPEC file creation for PMA#327
sfonn merged 18 commits into3.0-devfrom
pma-dev

Conversation

@jagratac
Copy link
Copy Markdown
Contributor

@jagratac jagratac commented Jul 21, 2025

Merge Checklist

All boxes should be checked before merging the PR

  • The changes in the PR have been built and tested
  • cgmanifest file has been updated if required
  • Ready to merge

Description

Initial commit establishing the PMA repository structure, comprising the directory layout and foundational files.

Any Newly Introduced Dependencies

Any Newly Introduced Dependencies

PMA should be added as part of EMT image.
No 3rd part dependency

How Has This Been Tested?

Tested with local system deployment.
complete automated flow tested for EMT
image

image image image

@jagratac jagratac requested a review from a team as a code owner July 21, 2025 05:27
@jagratac jagratac changed the title Intial commit for PMA Initial folder and SPEC file creation for PMA Jul 21, 2025
anujm1
anujm1 previously approved these changes Jul 21, 2025
@sfonn sfonn requested a review from andy-vm July 21, 2025 05:38
Copy link
Copy Markdown
Contributor

@sfonn sfonn left a comment

Choose a reason for hiding this comment

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

missing intel copyright for sh script

@sfonn
Copy link
Copy Markdown
Contributor

sfonn commented Jul 21, 2025

Could you share how has this been tested and output of "sudo systemd-analyze" on tested EMF core/xeon platforms with this change included?

Copy link
Copy Markdown
Contributor

@sfonn sfonn left a comment

Choose a reason for hiding this comment

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

Missing change to add this new PMA package for vPRO AMT inclusion in
https://github.com/open-edge-platform/edge-microvisor-toolkit/blob/3.0/toolkit/imageconfigs/packagelists/vpro-amt-packages.json?

How is this rpm verified to be building successfully and working?

Copy link
Copy Markdown
Contributor

@sfonn sfonn left a comment

Choose a reason for hiding this comment

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

Build is still failing.

@jagratac jagratac marked this pull request as draft July 22, 2025 03:26
Copy link
Copy Markdown
Contributor

@andy-vm andy-vm left a comment

Choose a reason for hiding this comment

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

please follow the steps below to test if the spec file works:
1: build the pma spec locally
2: build EMT image with pma included, boot up the image and make sure the pma service is up and running
3: attach the test result screenshots

@sfonn
Copy link
Copy Markdown
Contributor

sfonn commented Jul 25, 2025

@jagratac please confirm with developer build provided earlier with the changes in PR whether PMA functionality in this PR is working as expected.

@jagratac jagratac marked this pull request as ready for review July 29, 2025 04:48
Copy link
Copy Markdown
Contributor

@andy-vm andy-vm left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@sfonn sfonn left a comment

Choose a reason for hiding this comment

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

LGTM

@sfonn sfonn merged commit 366cb14 into 3.0-dev Jul 31, 2025
16 of 19 checks passed
@sfonn sfonn deleted the pma-dev branch July 31, 2025 10:43
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.

6 participants