Skip to content

fork synchronization#2

Merged
mspasiano merged 181 commits intoTrasparenzAI:mainfrom
conductor-oss:main
Oct 15, 2025
Merged

fork synchronization#2
mspasiano merged 181 commits intoTrasparenzAI:mainfrom
conductor-oss:main

Conversation

@mspasiano
Copy link
Member

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

Describe the new behavior from this PR, and why it's needed
Issue #

Alternatives considered

Describe alternative implementation you have considered

za-liyongchao002 and others added 27 commits September 3, 2025 16:44
…rk, readTimeOut always max to default 3 minutes
Fix(http-task): readTimeOut configuration from task input does not wo…
Minor wiki formatting so prerequisites render clearly
Add ssl mode and client name in redis cluster configuration
Added UI feature to manage event handlers - create/update
…efresh

UI Feature: Complete human task for running execution with refresh after api call
Introduce ExecutionMetadata model to capture execution level metrics in task and taskresult
Fix issue: Return 404 instead of 2xx codes for methods handling non-existent tasks#38
fix(elasticsearch-dao): wait for index refresh and guarantee changes are visible for search
…read_safety

Address bug where the Kafka event queue is resuing the consumer client.id for each queue created
…pply

Ensures docker-compose builds even when code would fail Spotless check.
…ctor-oss-v32119-the-connection-to-aws-sqs-broken

add @JsonProperty annotations to fix AWS policy JSON format
add username config to redis lock
…ta-test

Fix flaky postgres poll data test
Fixes 4 test failures related to NotFoundException behavior changes:

1. SQSEventQueueE2ESpec.groovy:420
   - Convert Groovy GString to String when passing to AWS SDK
   - Fixes ClassCastException in setQueueAttributes call

2. QueueResiliencySpec.groovy:517 - getTaskLogs test
   - Add thrown(NotFoundException) expectation
   - TaskResource.getTaskLogs now throws NotFoundException for missing tasks

3. QueueResiliencySpec.groovy:525 - getTask test
   - Add thrown(NotFoundException) expectation
   - TaskResource.getTask now throws NotFoundException for missing tasks

4. AbstractHttpEndToEndTest.java:451 - testTaskByTaskId
   - Update error message assertion from "No such task found by" to "Task not found for"
   - Matches new error message format from PR #69

Related to issue #602

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
…lures

Fix test failures stemming from exception handling changes
@mspasiano mspasiano self-assigned this Oct 15, 2025
@mspasiano mspasiano merged commit 12bd860 into TrasparenzAI:main Oct 15, 2025
mspasiano pushed a commit that referenced this pull request Feb 22, 2026
…#735)

* Fix flaky tests: PostgresGrpcEndToEndTest and PostgresPollDataDAONoCacheTest

1. PostgresGrpcEndToEndTest - Fixed two race conditions:
   - Search indexing: Replaced Thread.sleep(1000) with await().atMost(5, SECONDS)
     for Elasticsearch/OpenSearch indexing (8 search operations)
   - Task polling: Added awaitility retry for workflow task scheduling
   Both race conditions caused "expected:<1> but was:<0>" failures

2. PostgresPollDataDAONoCacheTest - Fixed test isolation issue:
   - Added @DirtiesContext(classMode = AFTER_EACH_TEST_METHOD) to force
     Spring context reload between tests
   - Added verification in @before to detect if table truncation fails
   - Prevents data leakage between tests in CI environments with
     connection pooling

Dependencies:
- Added awaitility test dependency to test-util module

These fixes address intermittent CI failures by replacing fixed delays
with robust retry mechanisms and ensuring proper test isolation.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* Trigger CI to verify flaky test fixes

* Trigger CI build #2 for validation

* Trigger CI build #3 for validation

---------

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
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.