Skip to content

Scc updates tls#330

Open
rhdedgar wants to merge 3 commits into
ogx-ai:mainfrom
rhdedgar:scc_updates_tls
Open

Scc updates tls#330
rhdedgar wants to merge 3 commits into
ogx-ai:mainfrom
rhdedgar:scc_updates_tls

Conversation

@rhdedgar

Copy link
Copy Markdown
Collaborator
  • Avoids running with anyuid permissions
  • TLS is required for external access is enabled
  • Fixes a few config and build errors that I encountered along the way. e.g. llama->ogx renaming, duplicate boolPtr func, etc.

Closes RHAIENG-5584, RHAIENG-5588, and RHOAIENG-76479.

@VaishnaviHire

Copy link
Copy Markdown
Collaborator

Is the test failure expected?

@rhdedgar

Copy link
Copy Markdown
Collaborator Author

Not this one. The others that were blocking other k8s-operator PRs were resolved, and now there is a new issue, possibly with ollama. I'm looking into it now.

rhdedgar added 3 commits July 17, 2026 11:19
Signed-off-by: Doug Edgar <dedgar@redhat.com>
Signed-off-by: Doug Edgar <dedgar@redhat.com>
Signed-off-by: Doug Edgar <dedgar@redhat.com>
@rhdedgar

Copy link
Copy Markdown
Collaborator Author

Ok, so I found two sets of issues that were blocking this PR.

One was a new, unrelated issue with container image name formatting that I just fixed in #336

The other one is an upstream ogx server dependency, because the starter distribution image runs as root by default. I've created a PR here ogx-ai/ogx#6319 to address that.

After that merges, then the next starter distribution container build should resolve this error.

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.

2 participants