Skip to content

Update VPro ADR with ACM mode activation user flow#1483

Open
Ram-srini wants to merge 3 commits intomainfrom
acm_adr
Open

Update VPro ADR with ACM mode activation user flow#1483
Ram-srini wants to merge 3 commits intomainfrom
acm_adr

Conversation

@Ram-srini
Copy link
Copy Markdown
Contributor

Description

Please include a summary of the changes and the related issue. List any dependencies that are required for this change.

Fixes # (issue)

Any Newly Introduced Dependencies

Please describe any newly introduced 3rd party dependencies in this change. List their name, license information and how they are used in the project.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes
  • I have not introduced any 3rd party dependency changes
  • I have performed a self-review of my code

Comment on lines +182 to +188
activate DM
Note over DM: Subscribe and listen for tenant creation events.
DM ->> DM: TenantCreation event received
DM ->> RPS: Create CIRA configuration
DM ->> RPS: Create the CCM profile
DM ->> RPS: Create the ACM profile
deactivate DM
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should also include this in the previous sequence diagram, since CIRA configurations apply to both CCM and ACM.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

CIRA configurations are created by DM manager and user input are not required. Yes they are common to both the modes.

Note over PMA: User has update the EN BIOS with DNS suffix(ACM mode only) and MEBx Password

US ->> CLI: Register the EN(serialNo or hardwareUUID, activationMode)
Note over CLI: Default CCM mode will chosen if user doesn't specify the action mode
Copy link
Copy Markdown
Contributor

@damiankopyto damiankopyto Feb 20, 2026

Choose a reason for hiding this comment

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

@Ram-srini setting activation mode at registration time seems like a new feature for 2026.0

In 2025.2 we have reverted for AMT/vPRO to be an opt-in day2 feature, if we have to set a mode for it during registration that kind of negates the opt-in philosophy

Today we already have a CLI command implemented to set the activation mode post onboarding/provisioning that can be used just before the activation command

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

AMT/Vpro is opt-in feature, and user can activate/deactivate anytime. Choosing the CCM/ACM is the user choice and while registering the device it can be opted. If user doesn't provide the mode, then CCM is default.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does this need to be cleaned up?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, it needs to be cleaned up

Signed-off-by: Srinivasamurthy, Ramakrishna <ramakrishna.srinivasamurthy@intel.com>
Signed-off-by: Srinivasamurthy, Ramakrishna <ramakrishna.srinivasamurthy@intel.com>
Signed-off-by: Srinivasamurthy, Ramakrishna <ramakrishna.srinivasamurthy@intel.com>
@jagratac
Copy link
Copy Markdown
Contributor

Need to add orch-cli domain creation part in current flow.

@github-actions
Copy link
Copy Markdown

This pull request is stale because it has been open 30 days with no activity. Make a comment or update the PR to avoid closing PR after 15 days.

@github-actions github-actions bot added the stale label Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants