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

Create Source tarball using Default ASF Configuration #9184

Merged
merged 5 commits into from
Jan 21, 2021

Conversation

eolivelli
Copy link
Contributor

@eolivelli eolivelli commented Jan 12, 2021

With this patch we are now using the common ASF configuration to create the "release tarball" to be VOTEd and release during the official VOTE.

Currently in 2.6+ we are

Changes:

Results:
With this change you will find the apache-pulsar-VERSION-src.tar.gz file inside the main "target" directory.
It contains the dump of the source tree, according to the default Apache rules (basically it strips out only the GIT releated files).

The source tarball that we are going to release and VOTE will be a copy of the git tag from which it has been created

Notes:
in ASF projects usually you are using the -Papache-release profile but it looks like we are not using it always here https://github.com/apache/pulsar/wiki/Release-process

it would be better to activate this package only with a Maven profile, in order to speed up the build when you are not performing a release.

pom.xml Show resolved Hide resolved
@eolivelli
Copy link
Contributor Author

@codelipenghui this is the patch for the release

@eolivelli
Copy link
Contributor Author

/pulsarbot run-failure-checks

1 similar comment
@codelipenghui
Copy link
Contributor

/pulsarbot run-failure-checks

Copy link
Member

@sijie sijie left a comment

Choose a reason for hiding this comment

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

@eolivelli I'd prefer fixing that problem first because 2.7.1 is a minor release, not a major release. I'd prefer not to change the existing release workflow. This change can go into master.

@eolivelli
Copy link
Contributor Author

@sijie this change does not impact the binaries so it is pretty safe.
IMHO it is up to the Release manager and finally the PMC to validate the release.

I suggest to commit this patch to master branch and to 2.7 branch.
As you said probably it is not worth to pick it to 2.6 branch, I guess that 2.6.3 will be the last release cut out of that branch.

btw I am fixing the CI error

@eolivelli
Copy link
Contributor Author

/pulsarbot run-failure-checks

@eolivelli
Copy link
Contributor Author

/pulsarbot run-failure-checks

@sijie
Copy link
Member

sijie commented Jan 14, 2021

I don't think we should commit this change to a branch release. I understand you fix the build script but I am not sure how will that impact the whole release process. Hence I'd prefer fixing the current build process to include missing files for minor releases. We can use this PR for 2.8 release and onwards.

@eolivelli
Copy link
Contributor Author

@sijie agreed.
let's do it only for master branch

@eolivelli
Copy link
Contributor Author

@sijie @codelipenghui CI passed can you please merge this patch ?

@codelipenghui codelipenghui merged commit 0770ae6 into apache:master Jan 21, 2021
@codelipenghui
Copy link
Contributor

@eolivelli Sorry for the late response, I have merged the PR.

merlimat pushed a commit to merlimat/pulsar that referenced this pull request Apr 6, 2021
With this patch we are now using the common ASF configuration to create the "release tarball" to be VOTEd and release during the official VOTE.

Currently in 2.6+ we are 

Changes:
- Use ASF apache-source-release-assembly-descriptor
- Configure the final name of the artifact
- Update the release script src/stage-release.sh used during the release process https://github.com/apache/pulsar/wiki/Release-process
- Remove the old "src" descriptor 

Results:
With this change you will find the apache-pulsar-VERSION-src.tar.gz file inside the main "target" directory.
It contains the dump of the source tree, according to the default Apache rules (basically it strips out only the GIT releated files).

The source tarball that we are going to release and VOTE will be a copy of the git tag from which it has been created

Notes:
in ASF projects usually you are using the -Papache-release profile but it looks like we are not using it always here https://github.com/apache/pulsar/wiki/Release-process

it would be better to activate this package only with a Maven profile, in order to speed up the build when you are not performing a release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants