Skip to content

[Build] Update management module version#8695

Merged
codelipenghui merged 1 commit intoapache:masterfrom
gaoran10:update-management-module-version
Nov 25, 2020
Merged

[Build] Update management module version#8695
codelipenghui merged 1 commit intoapache:masterfrom
gaoran10:update-management-module-version

Conversation

@gaoran10
Copy link
Contributor

@gaoran10 gaoran10 commented Nov 25, 2020

Modifications

Update package org.apache.pulsar:pulsar-package-management from 2.7.0-SNAPSHOT version to 2.8.0-SNAPSHOT.

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

@codelipenghui
Copy link
Contributor

/pulsarbot run-failure-checks

@codelipenghui
Copy link
Contributor

/pulsarbot run-failure-checks

1 similar comment
@gaoran10
Copy link
Contributor Author

/pulsarbot run-failure-checks

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

LGTM

@eolivelli
Copy link
Contributor

The MACOs test is failing with:

Run apache/pulsar-test-infra/diff-only@master
Error: Container action is only supported on Linux

IMHO we can ignore it and merge the patch

@codelipenghui codelipenghui merged commit 2fcb2f7 into apache:master Nov 25, 2020
@codelipenghui
Copy link
Contributor

@eolivelli Yes, we are still trying to find a solution to verify the build process on different os. So it's not a required CI test. I have merged this PR, and I will close #8698 first.

@gaoran10 gaoran10 deleted the update-management-module-version branch November 25, 2020 14:24
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.

3 participants