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

[fix][build] Include Pulsar shell in the released artifacts #18583

Merged
merged 1 commit into from
Nov 24, 2022

Conversation

nicoloboschi
Copy link
Contributor

Motivation

The current release process doesn't stage the pulsar shell artifacts. Pulsar shell is going to be released in 2.11.0

Modifications

  • Add the .tar.gz and .zip pulsar shell artifacts to the stage-artifacts procedure
  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

note: we should update the release process in the pulsar-site repository.

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Nov 23, 2022
@nicoloboschi nicoloboschi reopened this Nov 23, 2022
@tisonkun tisonkun requested a review from Technoboy- November 23, 2022 17:06
@tisonkun
Copy link
Member

It seems the release action of 2.11.0 is performed against branch-2.11 so we need a cherry-pick.

Also, cc @Technoboy- as the release manager of 2.11.0 FYI.

@codecov-commenter
Copy link

codecov-commenter commented Nov 23, 2022

Codecov Report

Merging #18583 (1060d2f) into master (be1d07e) will increase coverage by 8.81%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #18583      +/-   ##
============================================
+ Coverage     36.82%   45.63%   +8.81%     
- Complexity     7842     9863    +2021     
============================================
  Files           698      672      -26     
  Lines         68060    65824    -2236     
  Branches       7277     7034     -243     
============================================
+ Hits          25062    30042    +4980     
+ Misses        39675    32365    -7310     
- Partials       3323     3417      +94     
Flag Coverage Δ
unittests 45.63% <ø> (+8.81%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...sar/broker/stats/metrics/ManagedLedgerMetrics.java 23.88% <0.00%> (-76.12%) ⬇️
...rvice/schema/KeyValueSchemaCompatibilityCheck.java 21.62% <0.00%> (-45.95%) ⬇️
...ar/broker/loadbalance/impl/BundleSplitterTask.java 60.00% <0.00%> (-20.00%) ⬇️
.../org/apache/pulsar/broker/lookup/LookupResult.java 56.66% <0.00%> (-20.00%) ⬇️
...ulsar/broker/namespace/NamespaceEphemeralData.java 68.18% <0.00%> (-18.19%) ⬇️
...e/pulsar/broker/stats/metrics/AbstractMetrics.java 36.48% <0.00%> (-17.57%) ⬇️
...pulsar/broker/service/PulsarCommandSenderImpl.java 65.64% <0.00%> (-12.83%) ⬇️
...ce/schema/validator/StructSchemaDataValidator.java 52.38% <0.00%> (-9.53%) ⬇️
...g/apache/pulsar/broker/stats/MetricsGenerator.java 90.47% <0.00%> (-9.53%) ⬇️
...roker/service/persistent/MessageDeduplication.java 46.28% <0.00%> (-8.30%) ⬇️
... and 153 more

@Technoboy- Technoboy- added this to the 2.12.0 milestone Nov 24, 2022
Copy link
Member

@tisonkun tisonkun left a comment

Choose a reason for hiding this comment

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

Merging...

@nicoloboschi @Technoboy- how do you think this patch should be cherry-picked to branch-2.11?

@tisonkun tisonkun merged commit 38af2f3 into apache:master Nov 24, 2022
@Technoboy-
Copy link
Contributor

Merging...

@nicoloboschi @Technoboy- how do you think this patch should be cherry-picked to branch-2.11?

I will do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants