New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[hotfix] Update nightly master cron jobs #7045
Conversation
@@ -43,12 +43,16 @@ echo "Flink distribution directory: $FLINK_DIR" | |||
|
|||
# run_test "<description>" "$END_TO_END_DIR/test-scripts/<script_name>" | |||
|
|||
run_test "Flink CLI end-to-end test" "$END_TO_END_DIR/test-scripts/test_cli.sh" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please also update split_misc_hadoopfree
to include all tests independent of hadoop. I've copied split_misc.sh
for now since it's the easiest&fastest option.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will copy some tests but I think we need to further split split_misc_hadoopfree
because we run out of time otherwise.
nightly.sh
Outdated
@@ -43,7 +43,7 @@ mkdir -p $ARTIFACTS_DIR || { echo "FAILURE: cannot create log directory '${ARTIF | |||
LOG4J_PROPERTIES=${FLINK_DIR}/tools/log4j-travis.properties | |||
|
|||
MVN_LOGGING_OPTIONS="-Dlog.dir=${ARTIFACTS_DIR} -Dlog4j.configuration=file://$LOG4J_PROPERTIES -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn" | |||
MVN_COMMON_OPTIONS="-nsu -B -Dflink.forkCount=2 -Dflink.forkCountTestPackage=2 -Dfast" | |||
MVN_COMMON_OPTIONS="-T1C -nsu -B -Dflink.forkCount=2 -Dflink.forkCountTestPackage=2 -Dfast" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be in MVN_COMPILE_OPTIONS
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise we're executing (java) e2e tests in parallel
012778e
to
87db17a
Compare
87db17a
to
782a5a9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
split_misc_hadoopfree
still contains the kubernetes test.
Thanks @zentol. I will remove the kubernetes test and merge this. Once Travis gives green light. |
The changes have been applied to |
What is the purpose of the change
This PR updates the nightly master cron jobs to be in sync with the available end-to-end tests.
Brief change log
Verifying this change
Not verified yet as builds are failing. We might need to further stabilize the tests and might further split the
misc
tests.Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: noDocumentation