Skip to content

OpenShift bootstrap manifests #317

Open
sats-23 wants to merge 4 commits intoIBM:mainfrom
sats-23:bs-manifests
Open

OpenShift bootstrap manifests #317
sats-23 wants to merge 4 commits intoIBM:mainfrom
sats-23:bs-manifests

Conversation

@sats-23
Copy link
Contributor

@sats-23 sats-23 commented Feb 12, 2026

Manifests that will be used in bootstrap configure process.

Contains:

  1. Dependent Operators
    --Node Feature Discovery w/ CR NFD
    --Cert-Manager w/ CR Self-Signed Issuer, CA Certificate, CA Issuer, Webhook Certificate
    --Service Mesh 3
    --Secondary Scheduler

  2. Machine Config (numbered for the ordering) - reference
    --Machine Config Pool
    --Actual MachineConfig for spyre configuration

  3. Core Operators
    --Spyre Operator w/ CR SpyreClusterPolicy (w/o externalDeviceReservation)

RHOAI Operator will be included eventually after v3.3.0 is GA'ed

Comment on lines +41 to +43
pfRunnerImage: quay.io/ibm-aiu/x86_64/release/dd2/spyredriver-image:latest
spyreFilter: .
vfRunnerImage: quay.io/ibm-aiu/x86_64/release/dd2/spyredriver-image:latest
Copy link
Member

Choose a reason for hiding this comment

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

Is it good to use latest?.
Isnt it better to use a specific version tag that we have tested so that we dont have to worry if there is any future release breaking changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

True @mayuka-c
Currently the repo is private and I can't see what the latest tag is pointing to.
When I create a SpyreClusterPolicy, it default to the latest tag

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mayuka-c
Copy link
Member

mayuka-c commented Feb 12, 2026

Can we have the path as ai-services/assets/bootstrap/openshift.

This would align with out app path ai-services/assets/application/

Signed-off-by: Sathvik <Sathvik.S@ibm.com>
Signed-off-by: Sathvik <Sathvik.S@ibm.com>
Signed-off-by: Sathvik <Sathvik.S@ibm.com>
Signed-off-by: Sathvik <Sathvik.S@ibm.com>
Copy link
Member

@mkumatag mkumatag left a comment

Choose a reason for hiding this comment

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

whats the relevant directory we need to name? is bootstrap a right thing? or can we rename to something else like internal, .system or something else which make sense for the tool itself.?

also we need to add an entry in here as well to include these newly added asset

var ApplicationFS embed.FS

sourceNamespace: openshift-marketplace
startingCSV: spyre-operator.v1.1.1
---
apiVersion: spyre.ibm.com/v1alpha1
Copy link
Member

Choose a reason for hiding this comment

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

we shouldn't do this, we should just add a patch to the existing resource gets deployed by default to remove that one unwanted entry from the existing resource.

Copy link
Member

Choose a reason for hiding this comment

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

let us names the files like 00-, 02- etc..

Comment on lines +5 to +6
kubernetes.io/arch: ppc64le
machineconfiguration.openshift.io/role: spyre
Copy link
Member

Choose a reason for hiding this comment

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

lets have some labels across all the machineconfigs, e.g: kubernetes.io/arch: ppc64le exist only in this and doesn't in other 2 machineconfigs

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.

3 participants