Skip to content

Remove ragImage configuration from CR#8

Merged
lpiwowar merged 1 commit into
openstack-k8s-operators:mainfrom
umago:remove-ragImage
Jul 9, 2026
Merged

Remove ragImage configuration from CR#8
lpiwowar merged 1 commit into
openstack-k8s-operators:mainfrom
umago:remove-ragImage

Conversation

@umago

@umago umago commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

For tech preview, the RAG images are built with specific requirements (e.g., bundled with OKP embedding model) that are not yet available to customers. Removing the field prevents users from supplying custom RAG images that would lack these dependencies and break things.

The operator continues to use the image pinned via the RELATED_IMAGE_OPENSTACK_LIGHTSPEED_IMAGE_URL_DEFAULT environment variable, which is set to a known image at operator build time.

@umago umago marked this pull request as draft July 3, 2026 16:09
@umago

umago commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator Author

Ran kuttl tests locally, it's good:

=== NAME  kuttl
    harness.go:403: run tests finished
    harness.go:510: cleaning up
    harness.go:567: removing temp folder: ""
--- PASS: kuttl (374.80s)
    --- PASS: kuttl/harness (0.00s)
        --- PASS: kuttl/harness/basic-openstack-lightspeed-configuration (69.58s)
        --- PASS: kuttl/harness/persistent-database (97.39s)
        --- PASS: kuttl/harness/update-openstacklightspeed (107.39s)
        --- PASS: kuttl/harness/okp-configuration (100.42s)
PASS

@umago umago marked this pull request as ready for review July 3, 2026 16:25
@omkarjoshi0304

Copy link
Copy Markdown
Collaborator

LGTM, thank you:)

@Akrog Akrog left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/lgtm

For tech preview, the RAG images are built with specific requirements
(e.g., bundled with OKP embedding model) that are not yet available to
customers. Removing the field prevents users from supplying custom RAG
images that would lack these dependencies and break things.

The operator continues to use the image pinned via the
RELATED_IMAGE_OPENSTACK_LIGHTSPEED_IMAGE_URL_DEFAULT environment variable,
which is set to a known image at operator build time.

Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
@umago umago force-pushed the remove-ragImage branch from 07484a3 to e9f5c30 Compare July 8, 2026 18:33
@umago umago requested a review from lpiwowar July 9, 2026 08:36
@lpiwowar

lpiwowar commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

/lgtm

Sorry, I missed this one.

@lpiwowar lpiwowar merged commit ed059d7 into openstack-k8s-operators:main Jul 9, 2026
3 checks passed
@umago umago deleted the remove-ragImage branch July 9, 2026 10:50
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.

4 participants