tkaymak commented on PR #38966: URL: https://github.com/apache/beam/pull/38966#issuecomment-4711822980
Addressed @gemini-code-assist's two comments in `8387d94710b` (bare `except:` → `except Exception:`; the broker-start retry now stops a partially-started container to avoid leaks). The two red checks are both pre-existing **master** breakages, unrelated to this PR (which only adds the messaging expansion service + its IT/workflow): - **beam_PreCommit_Yaml_Xlang_Direct** — failing on the last several master scheduled runs with `io expansion-service shadowJar not found` across the Iceberg/Mongodb/Flatten yaml tests (none MQTT). Filed #38972. - **beam_PreCommit_Python_ML 3.13** — the Milvus RAG integration suites fail on every master run (container timeouts). Filed #38945. The MQTT integration tests in this PR run under `beam_PostCommit_Python_Xlang_Messaging_Direct` (PostCommit), which isn't part of the PreCommit set; I verified all three locally against the messaging shadowJar. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
