Repository navigation
Use aifoa namespace and complete NGC-based NMC installation instructions - #83
Merged
Merged
Conversation
Align public install, access and integration examples and the command builder on the aifoa namespace. Preserve Helm release and resource names, custom namespace selection and existing installations; explain that changing a namespace is not a migration. Refresh the bundled documentation and verify default and custom command generation. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Explain root and sudo login-shell requirements, Kubernetes access and the checkout directory required for the local inference chart. Verify authenticated model discovery and generation before agent installation, and document uninstall ownership, retained model caches, data loss and shared infrastructure across the admin and workload clusters. The readiness example was exercised against a running self-hosted inference pod. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Preserve the latest frontend image pins while keeping the namespace and operator documentation work based on current main. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Replace repository cloning and checkout-relative installation with the verified afoa-inference OCI chart and a pinned published version. Keep the existing inference values and authentication workflow and regenerate the bundled operator documentation. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Expand the new operator prose to Base Command Manager to satisfy the existing documentation vocabulary check. Preserve current main changes, align its new Redfish example with the aifoa namespace convention, and rebuild the documentation archive from the combined sources. Full local CI passes with 226 tests passed and two skipped; strict documentation build and Helm lint pass. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Preserve the frontend image updates from main while retaining the reviewed NMC onboarding instructions and CI wording fix. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Replace the operator account explanation with the agreed single sentence and refresh the bundled documentation. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Replace the embedded Python probe with an authenticated completion request through the configured HTTPS endpoint and explain the expected response. Refresh the bundled documentation. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Align installation, upgrade and uninstall examples and the command builder with the aifoa namespace convention. Preserve guidance for existing release names, validate rendered commands with the new release, and refresh the documentation bundle. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Remove the unnecessary final cluster switch and resource listing commands. Condense cleanup guidance while retaining backup, retained-storage and shared-resource ownership requirements, and refresh the documentation bundle. Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Member
Author
|
@coderabbitai can u see this? |
|
Tip For best results, initiate chat on the files or code changes. @RohanAdwankar Yes, I can see your comment. You are interacting with an AI system. |
Member
Author
|
@coderabbitai r u able to review the PR? |
|
Tip For best results, initiate chat on the files or code changes. @RohanAdwankar Yes, I can review PR I cannot confirm whether the You are interacting with an AI system. |
Member
Author
|
@coderabbitai review |
Action not completed
|
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.
Use aifoa for new-install namespace examples and the command builder while preserving custom namespaces, existing installations and release/resource names. Complete operator guidance for root/sudo login shells, inference verification and uninstall across both clusters with data and shared-resource ownership explained. Replace the inference checkout requirement with its now-published NGC OCI chart pinned to 0.0.2-main.20261030003406.gd14c5492; Git and a repository working directory are no longer required. Validation: namespace CI tests and browser checks passed, authenticated model discovery and generation succeeded against self-hosted inference, the new OCI artifact was pulled and rendered on aifoa2 and compared to source, and strict documentation builds, Bash syntax and bundle freshness checks passed. No existing deployment was migrated or uninstalled.
Summary by CodeRabbit
aifoaas the default namespace and Helm release name. Custom names remain supported; existing installations should keep their current names, as changing them does not migrate data.