Skip to content
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

HADOOP-17354. Move Jenkinsfile outside of the root directory. #2647

Merged
merged 1 commit into from
Feb 1, 2021

Conversation

aajisaka
Copy link
Member

JIRA: https://issues.apache.org/jira/browse/HADOOP-17354

The Jenkinsfile is placed under the project root directory, so when the Jenkinsfile is changed, all the Hadoop unit tests will run and it wastes a lot of time and resources. Let's move the file outside of the root directory.

If approved, I'll commit this to all the active branches and update the config of hadoop-multibranch job to specify the new location of the Jenkinsfile.

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, but not sure why the Jenkins job didn't run for this PR?

@aajisaka
Copy link
Member Author

aajisaka commented Jan 26, 2021

The hadoop-multibranch job first searches ./Jenkinsfile but failed to find it because it is moved to ./dev-support/Jenkinsfile. That way the job didn't run.

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanx @aajisaka for confirming. I guess that we can change, Do give a check once you merge this.

@aajisaka
Copy link
Member Author

aajisaka commented Feb 1, 2021

Thank you @ayushtkn

@aajisaka aajisaka merged commit 84b154e into apache:trunk Feb 1, 2021
@aajisaka aajisaka deleted the HADOOP-17354 branch February 1, 2021 09:51
aajisaka added a commit that referenced this pull request Feb 1, 2021
aajisaka added a commit that referenced this pull request Feb 1, 2021
aajisaka added a commit that referenced this pull request Feb 1, 2021
aajisaka added a commit that referenced this pull request Feb 1, 2021
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.

2 participants