Skip to content

[e2e][python] Stabilize standalone nightly example tests - #975

Merged
wenjin272 merged 3 commits into
apache:mainfrom
wenjin272:standalone-test
Aug 7, 2026
Merged

[e2e][python] Stabilize standalone nightly example tests#975
wenjin272 merged 3 commits into
apache:mainfrom
wenjin272:standalone-test

Conversation

@wenjin272

Copy link
Copy Markdown
Contributor

Linked issue: #642

Purpose of change

This is a follow-up to #708 that makes the standalone nightly example test reliable enough to validate example execution instead of only successful submission.

The change:

  • builds and installs Flink Agents from the checked-out source and defaults the nightly run to Flink 2.3.0;
  • waits for submitted jobs to become stably running or finish, propagates discovery/setup/cleanup failures, and adds focused Bats coverage for the shell helpers;
  • shares the RAG Chroma data across the submitting process and TaskManager process;
  • synchronizes lazy Chroma client initialization to avoid the RustBindingsAPI initialization race;
  • uses a longer Ollama request timeout and disables thinking for the RAG example to keep CPU-only CI inference reliable; and
  • ignores repository-local .codex configuration.

Tests

  • uv run --no-sync pytest flink_agents/integrations/vector_stores/chroma/tests/test_chroma_vector_store.py flink_agents/examples/rag/tests/test_rag_agent.py -q — 23 passed, 1 skipped
  • bats tools/test/unit/verify_example_job.bats — 25 passed
  • Ruff check and format verification for the modified Python files
  • Nightly standalone E2E with Flink 2.3.0: 13 passed, 0 failed — https://github.com/wenjin272/flink-agents/actions/runs/31109016948

API

No public API changes.

Documentation

  • doc-needed
  • doc-not-needed
  • doc-included

Was this patch authored or co-authored using generative AI tooling?

  • Yes
  • No

Generated-by: Codex CLI 0.144.5

Generated-by: Codex CLI 0.144.5
Generated-by: Codex CLI 0.144.5
Generated-by: Codex CLI 0.144.5
@wenjin272
wenjin272 marked this pull request as ready for review August 6, 2026 14:55
@wenjin272 wenjin272 added feature [Issue Type] New features or improvements to existing features. priority/major Default priority of the PR or issue. doc-not-needed Your PR changes do not impact docs fixVersion/0.4.0 labels Aug 6, 2026
@github-actions github-actions Bot added doc-not-needed Your PR changes do not impact docs and removed doc-not-needed Your PR changes do not impact docs labels Aug 6, 2026
@wenjin272
wenjin272 merged commit 432e90d into apache:main Aug 7, 2026
54 of 56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs feature [Issue Type] New features or improvements to existing features. fixVersion/0.4.0 priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant