feat: AIF Operator Build#86
Open
leooamaral wants to merge 10 commits into
Open
Conversation
Renames the Go source directory and updates the module path from github.com/SUSE/suse-ai-operator to github.com/SUSE/aif-operator. Updates all import paths, field owners, namespace defaults, and test constants.
Renames all chart template helpers, labels, and resource names. Fixes helm.sh/chart label format, deployment indentation, adds EXTENSION_NAMESPACE env var and imagePullSecrets to the deployment. Guards empty nodeSelector/tolerations/affinity/podAnnotations.
Contributor
Author
Discussion: Configurable operator coordinates in the UICurrently the UI extension has hardcoded constants for the operator namespace and service name in export const OPERATOR_NAMESPACE = 'aif-operator';
export const OPERATOR_SERVICE = 'aif-operator';These are used to build the Rancher proxy URL ( The problem is that users can install the operator Helm chart in any namespace they choose (e.g. Possible approaches
Would be good to align on the preferred approach before hardcoding assumptions about the operator's namespace. |
…stem Updates README docs, chart README, e2e tests, UI constants, and operator sample CR references to use aif-operator namespace.
- Fix Chart.yaml home/sources URLs to point to the actual repo - Fix imagePullSecrets helper indentation for manager branch - Remove stale build-extension.yml and release-operator.yml workflows
- Add nil-guards (with) for resources, securityContext, podSecurityContext - Add standard Helm labels to ClusterRoles and namespaced Role - Add explicit namespace to ServiceAccount, leader-election Role/RoleBinding, and metrics Service for consistency and template-pipe safety
- Use fullname helper in NOTES.txt instead of .Release.Name - Fix CRD name from singular to plural in operator README delete commands - Add missing labels to metrics-auth ClusterRoleBinding
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
suse-ai-operator/→aif-operator/and updates module path togithub.com/SUSE/aif-operatorcharts/suse-ai-operator/→charts/aif-operator/with all internal template references updatedhelm.sh/chartlabel format, deployment indentation, addsEXTENSION_NAMESPACEenv var andimagePullSecretsto deployment, guards empty scheduling fieldsOPERATOR_NAMESPACE,OPERATOR_SERVICE) toaif-operatorrelease-aif-operator.ymlCI pipelineTest plan
go build ./...compiles successfullyhelm templaterenders cleanlyhelm lintpasses with 0 failures🤖 Generated with Claude Code