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

HDDS-5077. Include dev-support in ozone source distribution #2134

Merged
merged 1 commit into from Apr 8, 2021

Conversation

smengcl
Copy link
Contributor

@smengcl smengcl commented Apr 8, 2021

https://issues.apache.org/jira/browse/HDDS-5077

This should include dev-support in the ozone-src dist.

Change-Id: Ifccebd5d7f723fc6b5c0a303514ecd1009c38a78
@smengcl smengcl self-assigned this Apr 8, 2021
@smengcl smengcl added the build Pull request that modifies the build process label Apr 8, 2021
@smengcl
Copy link
Contributor Author

smengcl commented Apr 8, 2021

-Psrc now yields a tarball including dev-support:

$ tar tf hadoop-ozone-1.1.0-SNAPSHOT-src.tar.gz | grep "/dev-support"
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/hcfs-read.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/ratis-no-flush.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/hcfs-write.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/ratis-mock-followers.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/watchforcommit_all.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/README.md
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/watchforcommit.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/hadooprpc.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/ratis-flush.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/appendlog.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/dev-support/byteman/mock-scm.btm
hadoop-ozone-1.1.0-SNAPSHOT-src/hadoop-hdds/interface-client/dev-support/

Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

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

Thanks @smengcl for the fix, LGTM.

@prashantpogde
Copy link
Contributor

thanks @smengcl. We will need this for 1.1.0-RC2

@smengcl
Copy link
Contributor Author

smengcl commented Apr 8, 2021

Thanks @adoroszlai @prashantpogde for reviewing.

@prashantpogde prashantpogde merged commit ccd046e into apache:master Apr 8, 2021
prashantpogde pushed a commit that referenced this pull request Apr 9, 2021
Change-Id: Ifccebd5d7f723fc6b5c0a303514ecd1009c38a78
(cherry picked from commit ccd046e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Pull request that modifies the build process
Projects
None yet
3 participants