Skip to content

Conversation

@stdnt-xiao
Copy link
Contributor

@stdnt-xiao stdnt-xiao commented Sep 29, 2022

https://issues.apache.org/jira/browse/AMBARI-25746

What changes were proposed in this pull request?

An error is reported when run ./dev-support/docker/centos7/build-containers.sh
as follows:
No package /root/ambari-agent.rpm available. Error: Nothing to do Setting up mariadb-server Created symlink from /etc/systemd/system/multi-user.target.wants/mariadb.service to /usr/lib/systemd/system/mariadb.service. ERROR at line 1: Failed to open file '/var/lib/ambari-server/resources/Ambari-DDL-MySQL-CREATE.sql', error: 2 Setting up ambari-server bash: ambari-server: command not found bash: ambari-server: command not found

reason:
I am building in a new server environment, the server does not have mvn installed by default

solve:
Install mvn before compiling

How was this patch tested?

Use Docker cluster environment for tests(scripts are under ./dev-support/docker/centos7/build-containers.sh)

@stdnt-xiao stdnt-xiao closed this Sep 29, 2022
@stdnt-xiao stdnt-xiao reopened this Sep 29, 2022
@stdnt-xiao stdnt-xiao changed the title fix the question of dev-support build docker containers AMBARI-25746: fix the question of dev-support build docker containers Sep 29, 2022
@kevinw66
Copy link
Contributor

kevinw66 commented Oct 1, 2022

Still have problem when running on my machine

[root@bigdata centos7]# ./build-containers.sh
Compiling ambari
[root@1d5c15c3e947 ambari]#

@stdnt-xiao
Copy link
Contributor Author

stdnt-xiao commented Oct 2, 2022

hi @kevinw66
I modified the compilation process and kill maven when rebuild.

@stdnt-xiao
Copy link
Contributor Author

stdnt-xiao commented Oct 2, 2022

update build-ambari.sh file
build-ambari.sh can only be executed after build-containers.sh and already exists ambari-rpm-build container

Copy link
Contributor

@kevinw66 kevinw66 left a comment

Choose a reason for hiding this comment

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

+1 LGTM, thanks @stdnt-xiao

@kevinw66 kevinw66 merged commit d320c19 into apache:trunk Oct 2, 2022
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.

2 participants