Skip to content

feat: External ConfigMap Support for Cross-Operator Integration. #166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

feat: External ConfigMap Support for Cross-Operator Integration. #166

wants to merge 6 commits into from

Conversation

dmaniloff
Copy link

@dmaniloff dmaniloff commented Sep 26, 2025 •
edited
Loading

Add External ConfigMap Support for Cross-Operator Integration

Problem

LlamaStack distributions need to reference Konflux-built images from other operators (like TrustyAI) without manual user intervention.

Solution

Changes

  • API: New ExternalConfigMapSpec type with namespace/name/mapping fields
  • Controller: Fetches external ConfigMaps and injects values as environment variables
  • CRD: Updated schema with proper validation
  • Examples: Added sample YAML showing usage

@dmaniloff dmaniloff mentioned this pull request Sep 26, 2025
Merged
@dmaniloff dmaniloff marked this pull request as draft September 26, 2025 16:57
@dmaniloff
feat: External ConfigMap Support for Cross-Operator Integration.
1a2ff8d 
Signed-off-by: Diego Maniloff <[email protected]>
@dmaniloff
update docs.
e5b05a5 
Signed-off-by: Diego Maniloff <[email protected]>
@dmaniloff
cosmit: whitespace.
4ffc5b6 
Signed-off-by: Diego Maniloff <[email protected]>
@dmaniloff
sync release file.
064a8d1 
Signed-off-by: Diego Maniloff <[email protected]>
@dmaniloff
add unit test.
b50945f 
Signed-off-by: Diego Maniloff <[email protected]>
@dmaniloff
pre-commit/lint.
4c32db7 
Signed-off-by: Diego Maniloff <[email protected]>
@dmaniloff dmaniloff force-pushed the external-configmap-support branch from 95e7815 to 4c32db7 Compare September 26, 2025 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cdoern cdoern Awaiting requested review from cdoern cdoern is a code owner

@derekhiggins derekhiggins Awaiting requested review from derekhiggins derekhiggins is a code owner

@leseb leseb Awaiting requested review from leseb leseb is a code owner

@mfleader mfleader Awaiting requested review from mfleader mfleader is a code owner

@nathan-weinberg nathan-weinberg Awaiting requested review from nathan-weinberg nathan-weinberg is a code owner

@rhdedgar rhdedgar Awaiting requested review from rhdedgar rhdedgar is a code owner

@rhuss rhuss Awaiting requested review from rhuss rhuss is a code owner

@VaishnaviHire VaishnaviHire Awaiting requested review from VaishnaviHire VaishnaviHire is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@dmaniloff