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

[AMBARI-21158] Eliminate Maven warning in Log Feeder Container Registry #2075

Merged
merged 1 commit into from Aug 15, 2018

Conversation

adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Get rid of Maven warning:

[WARNING] Some problems were encountered while building the effective model for org.apache.ambari:ambari-logsearch-logfeeder-container-registry:jar:2.0.0.0-SNAPSHOT
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-jar-plugin is missing. @ org.apache.ambari:ambari-logsearch-logfeeder-container-registry:[unknown-version], ambari-logsearch/ambari-logsearch-logfeeder-container-registry/pom.xml, line 73, column 15

How was this patch tested?

$ mvn -am -pl ambari-logsearch/ambari-logsearch-logfeeder-container-registry clean package
[INFO] Scanning for projects...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] ambari-logsearch                                                   [pom]
[INFO] Ambari Logsearch Log Feeder Container Registry                     [jar]
...
[INFO] BUILD SUCCESS

@asfgit
Copy link

asfgit commented Aug 15, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3610/
Test PASSed.

Copy link
Contributor

@oleewere oleewere left a comment

Choose a reason for hiding this comment

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

lgtm

@adoroszlai adoroszlai merged commit 47aa256 into apache:trunk Aug 15, 2018
@adoroszlai adoroszlai deleted the AMBARI-21158 branch August 15, 2018 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants