Skip to content

test: Resolve MemoryHub E2E URL from discovery endpoint#243

Open
rdwj wants to merge 1 commit into
kagenti:mainfrom
rdwj:242-memoryhub-discovery-url
Open

test: Resolve MemoryHub E2E URL from discovery endpoint#243
rdwj wants to merge 1 commit into
kagenti:mainfrom
rdwj:242-memoryhub-discovery-url

Conversation

@rdwj
Copy link
Copy Markdown
Contributor

@rdwj rdwj commented May 12, 2026

Summary

The MemoryHub E2E test now tries the MemoryHub discovery endpoint before falling back to the MEMORYHUB_E2E_URL repo secret. When the sandbox cluster rotates, the test self-heals on the next CI run without waiting for someone to update the secret.

  • Adds _discover_url() that fetches https://redhat-ai-americas.github.io/memory-hub/discovery.json with a 5s timeout
  • URL resolution order: MEMORYHUB_E2E_URL env var (explicit override) > discovery endpoint > skip
  • Fully backward-compatible: existing CI config is unaffected when the env var is set

Closes #242

Test plan

  • When MEMORYHUB_E2E_URL is set, it takes priority (no behavior change)
  • When MEMORYHUB_E2E_URL is unset, the discovery endpoint is tried
  • When both are unavailable, the test skips cleanly with an updated message
  • Discovery fetch failure (timeout, DNS, 404) is silently caught and falls through to skip

The E2E test now tries the MemoryHub discovery endpoint
(redhat-ai-americas.github.io) before falling back to the
MEMORYHUB_E2E_URL env var. This way the test self-heals when
the sandbox cluster rotates, without waiting for someone to
update the repo secret.

The env var still takes priority when set, so existing CI config
is unaffected.

Closes kagenti#242

Assisted-by: Claude Code (Opus 4.6)
Signed-off-by: rdwj <wjackson@redhat.com>
@rdwj rdwj force-pushed the 242-memoryhub-discovery-url branch from a970d1a to 1419f07 Compare May 12, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

test: E2E MemoryHub test should discover URL from discovery endpoint

2 participants