Conversation
- Add file writing logic to backend for finalOutput and messages - Update operator to handle state directory management Co-Authored-By: Claude <noreply@anthropic.com>
- Add kustomize configuration for clean deployment management - Create PVC for persistent state storage with RWO compatibility - Implement HTTPS route with TLS edge termination - Fix RBAC permissions and namespace references - Update deploy script with uninstall capability and better error handling Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: sallyom <somalley@redhat.com>
- Update README with common setup section and streamlined deployment paths - Reset all image references to default quay.io/ambient_code/* names - Comment out kustomize images section to preserve override capability Co-Authored-By: Claude <noreply@anthropic.com>
- Handle case where secret patch fails on first attempt - Add retry logic with brief delay for secret creation timing - Ensure API key is properly populated even if secret was deleted - Add explicit namespace parameter to patch commands Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: sallyom <somalley@redhat.com>
Signed-off-by: sallyom <somalley@redhat.com>
Collaborator
|
/lgtm |
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
3 participants
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.
Updates OPenShift manifests, adds route, updates docs, adds PVC
@cooktheryan will follow-up with rbac small fix & might need to manually create the secret in your ns - that substitution is not working afaict