FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ZStack build failed for missing org.zstack:log4j2 dependency by lshmouse · Pull Request #1266 · zstackio/zstack · GitHub

/ zstack Public

ZStack build failed for missing org.zstack:log4j2 dependency - #1266

Open
lshmouse wants to merge 1 commit into
zstackio:masterfrom
MiComputingPlatform:build-failed
ConversationCommits (1)ChecksFiles changed
Open

ZStack build failed for missing org.zstack:log4j2 dependency#1266
lshmouse wants to merge 1 commit into
zstackio:masterfrom
MiComputingPlatform:build-failed

Conversation

lshmouse commented Apr 3, 2020

Copy link
Copy Markdown
Contributor

ZStack build failed for missing org.zstack:log4j2 dependency.

$mvn clean install package -DskipTests
...
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  01:11 min
[INFO] Finished at: 2020-04-03T12:43:32+08:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project build: Could not resolve dependencies for project org.zstack:build:war:3.9.0: Failure to find org.zstack:log4j2:jar:3.9.0 in http://nexus.d.xiaomi.net/nexus/content/groups/public was cached in the local repository, resolution will not be reattempted until the update interval of nexus has elapsed or updates are forced -> [Help 1

Solution: Move the org.zstack:log4j2 dependency under premium profile.

live4thee reviewed Apr 3, 2020
View reviewed changes
Comment thread build/pom.xml
<artifactId>zbox</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>

live4thee Apr 3, 2020

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

You moved the dependency to the "premium" profile - that is not expected.

lshmouse Apr 3, 2020

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

There is no org.zstack:log4j2 module under zstack and the maven can not find this dependency during building. So i guess the module is under premium version and move it there. Any suggestion about the placement of org.zstack:log4j2 dependency?

lshmouse Apr 3, 2020

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@live4thee Could we just remove this log4j2 dependency?

live4thee Apr 7, 2020

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I checked the code base and it turned out that your patch is correct. I will merge it. Thanks.

live4thee Apr 7, 2020

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Merged.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

live4thee live4thee left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL