Skip to content

Enhancements and Basic OCP Support #572

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

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

NormJohnIV
Copy link
Contributor

  1. Removed the need to set vdm: true when calling overlay_facts
  2. Added new vdm helper add_if_exists that can add overlays if a defined file exists
  3. Granted users more control over ordering their overlays via prefixing files with two digit number. The kustomization.yaml output has also been updated to display the priorites of all overlays and their source (either vdm or user)
    ex. 57-my-example-overlay.yaml
  4. Add basic support for Openshift. This includes routes, tls, remove seccomp profile, and optionally SecurityContextConstraints with the dependent RBAC permissions.

@saschjmil
Copy link
Contributor

Thanks for the contribution @NormJohnIV. I'm opening a PM feature request for the OpenShift support. While we wait for that, would you be willing to move the enhancements into a new PR? That would help us merge them into the main branch quicker.

@saschjmil saschjmil added the PM Feature Request Requires Project Management approval label Mar 31, 2025
@saschjmil
Copy link
Contributor

OpenShift support is currently on hold. I'll keep the PM Feature Request label on this PR while we wait. Meanwhile, you are welcome to move the enhancements into another PR.

Copy link

github-actions bot commented May 4, 2025

This PR is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale Open for 30 days with no activity label May 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PM Feature Request Requires Project Management approval stale Open for 30 days with no activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants