Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add vTPM to VMware provisioning #3602

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Lennonka
Copy link
Contributor

@Lennonka Lennonka commented Jan 21, 2025

What changes are you introducing?

Adding Virtual Trusted Platform Module compute-profile option and user permissions to VMware provisioning

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

New feature

SAT-30464 (public)

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

Checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into: probably none

  • Foreman 3.13/Katello 4.15 (EL9 only)
  • Foreman 3.12/Katello 4.14 (Satellite 6.16)
  • Foreman 3.11/Katello 4.13 (orcharhino 6.11 on EL8 only; orcharhino 7.0 on EL8+EL9)
  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (Satellite 6.15; orcharhino 6.8/6.9/6.10)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • We do not accept PRs for Foreman older than 3.7.

@Lennonka Lennonka marked this pull request as draft January 21, 2025 12:23
@Lennonka Lennonka force-pushed the add-secureboot-vmware branch from fea7cc7 to efe6933 Compare January 22, 2025 17:40
@Lennonka Lennonka changed the title Add Secure Boot to VMware provisioning Add vTPM to VMware provisioning Jan 22, 2025
@Lennonka Lennonka force-pushed the add-secureboot-vmware branch from efe6933 to e493afd Compare January 22, 2025 17:45
@Lennonka Lennonka marked this pull request as ready for review January 22, 2025 17:45
@Lennonka Lennonka added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective labels Jan 24, 2025
@Lennonka
Copy link
Contributor Author

@nofaralfasi @Gauravtalreja1 Ack?

@Lennonka Lennonka added tech review done No issues from the technical perspective and removed Needs tech review Requires a review from the technical perspective labels Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs style review Requires a review from docs style/grammar perspective tech review done No issues from the technical perspective
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants