Javiermateor commented on PR #2287:
URL: https://github.com/apache/systemds/pull/2287#issuecomment-3129194096

   
   Hi @christinadionysio ,
   
    I  ran the failing federated tests locally to rule out any issues with my 
changes. Here are the results:
   
   *   `mvn test -Dtest=...FederatedFullCumulativeTest` -> **BUILD SUCCESS** 
(16 tests passed)
   *   `mvn test -Dtest=...FederatedFullAggregateTest` -> **BUILD SUCCESS** (22 
tests passed)
   
   **Both of the test classes that are failing in the CI pipeline pass 
perfectly on my local machine.** The failures may be due to a flaky test in the 
CI environment and are not caused by my changes in our `nn` module or in our 
implementation for the PR.
   
   For completeness, the issue in the CI appears to be related to `FED°fedinit` 
instructions, which points towards other error sources, maybe environmental 
factors like networking or process timing.
   
   This pattern strongly suggests that these are flaky tests and the failures 
are due to the CI environment rather than a bug introduced by my PR.
   
   Could a maintainer please take a look or advise on how to proceed?


-- 
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: dev-unsubscr...@systemds.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to