You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/06/17 19:58:10 UTC

[GitHub] [flink] ferenc-csaky opened a new pull request, #20012: [FLINK-27667][yarn-tests][NO_MERGE] Attempt to stabilize flaky YARN IT tests

ferenc-csaky opened a new pull request, #20012:
URL: https://github.com/apache/flink/pull/20012

   ## What is the purpose of the change
   
   Stabilizing YARN IT test cases. Currently, they tend to fail, because of a concurrent temp folder cleanup.
   
   ## Brief change log
   
   Added separate JUnit temp dirs for test cases to avoid concurrent cleanup under the same root dir.
   
   ## Verifying this change
   
   _in progress_
   
   ## Does this pull request potentially affect one of the following parts:
   
     - Dependencies (does it add or upgrade a dependency): no
     - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: no
     - The serializers: no
     - The runtime per-record code paths (performance sensitive): no
     - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
     - The S3 file system connector: no
   
   ## Documentation
   
     - Does this pull request introduce a new feature? no
   


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] MartijnVisser commented on pull request #20012: [FLINK-27667][yarn-tests] Stabilize flaky YARN integration tests

Posted by GitBox <gi...@apache.org>.
MartijnVisser commented on PR #20012:
URL: https://github.com/apache/flink/pull/20012#issuecomment-1161884244

   If they look good we should merge it imho. Getting stable tests makes it worthwhile 😅


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] mbalassi commented on pull request #20012: [FLINK-27667][yarn-tests][NO_MERGE] Attempt to stabilize flaky YARN IT tests

Posted by GitBox <gi...@apache.org>.
mbalassi commented on PR #20012:
URL: https://github.com/apache/flink/pull/20012#issuecomment-1160460541

   @flinkbot run azure


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] bgeng777 commented on pull request #20012: [FLINK-27667][yarn-tests] Stabilize flaky YARN integration tests

Posted by GitBox <gi...@apache.org>.
bgeng777 commented on PR #20012:
URL: https://github.com/apache/flink/pull/20012#issuecomment-1162724423

   LGTM as well. Thanks @ferenc-csaky !


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] ferenc-csaky commented on pull request #20012: [FLINK-27667][yarn-tests][NO_MERGE] Attempt to stabilize flaky YARN IT tests

Posted by GitBox <gi...@apache.org>.
ferenc-csaky commented on PR #20012:
URL: https://github.com/apache/flink/pull/20012#issuecomment-1160311433

   @flinkbot run azure


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] MartijnVisser commented on pull request #20012: [FLINK-27667][yarn-tests] Stabilize flaky YARN integration tests

Posted by GitBox <gi...@apache.org>.
MartijnVisser commented on PR #20012:
URL: https://github.com/apache/flink/pull/20012#issuecomment-1162857409

   You too @mbalassi  :)


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] flinkbot commented on pull request #20012: [FLINK-27667][yarn-tests][NO_MERGE] Attempt to stabilize flaky YARN IT tests

Posted by GitBox <gi...@apache.org>.
flinkbot commented on PR #20012:
URL: https://github.com/apache/flink/pull/20012#issuecomment-1159190152

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "be30fe0f9954f673ac599e9e3149edb98ee67c9b",
       "status" : "UNKNOWN",
       "url" : "TBD",
       "triggerID" : "be30fe0f9954f673ac599e9e3149edb98ee67c9b",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * be30fe0f9954f673ac599e9e3149edb98ee67c9b UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run azure` re-run the last Azure build
   </details>


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] ferenc-csaky commented on pull request #20012: [FLINK-27667][yarn-tests][NO_MERGE] Attempt to stabilize flaky YARN IT tests

Posted by GitBox <gi...@apache.org>.
ferenc-csaky commented on PR #20012:
URL: https://github.com/apache/flink/pull/20012#issuecomment-1160310161

   recheck


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] mbalassi commented on pull request #20012: [FLINK-27667][yarn-tests] Stabilize flaky YARN integration tests

Posted by GitBox <gi...@apache.org>.
mbalassi commented on PR #20012:
URL: https://github.com/apache/flink/pull/20012#issuecomment-1162846936

   Thanks @MartijnVisser.


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] MartijnVisser merged pull request #20012: [FLINK-27667][yarn-tests] Stabilize flaky YARN integration tests

Posted by GitBox <gi...@apache.org>.
MartijnVisser merged PR #20012:
URL: https://github.com/apache/flink/pull/20012


-- 
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: issues-unsubscribe@flink.apache.org

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