You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by GitBox <gi...@apache.org> on 2022/09/29 15:25:41 UTC
[GitHub] [ambari] stdnt-xiao opened a new pull request, #3376: fix the question of dev-support build docker containers
stdnt-xiao opened a new pull request, #3376:
URL: https://github.com/apache/ambari/pull/3376
## 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)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org
[GitHub] [ambari] kevinw66 commented on a diff in pull request #3376: AMBARI-25746: fix the question of dev-support build docker containers
Posted by GitBox <gi...@apache.org>.
kevinw66 commented on code in PR #3376:
URL: https://github.com/apache/ambari/pull/3376#discussion_r984617279
##########
dev-support/docker/centos7/build-containers.sh:
##########
@@ -15,6 +15,12 @@
# See the License for the specific language governing permissions and
# limitations under the License.
+echo -e "\033[32mInstall mvn\033[0m"
+if [ ! "$(command -v mvn)" ]; then
+ echo "mvn not found and will be installed" >&2
+ yum install -y maven
Review Comment:
We can't use yum here because we should support as many distros as possible
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org
[GitHub] [ambari] kevinw66 commented on pull request #3376: AMBARI-25746: fix the question of dev-support build docker containers
Posted by GitBox <gi...@apache.org>.
kevinw66 commented on PR #3376:
URL: https://github.com/apache/ambari/pull/3376#issuecomment-1264398370
Still have problem when running on my machine
```
[root@bigdata centos7]# ./build-containers.sh
Compiling ambari
[root@1d5c15c3e947 ambari]#
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org
[GitHub] [ambari] stdnt-xiao commented on pull request #3376: AMBARI-25746: fix the question of dev-support build docker containers
Posted by GitBox <gi...@apache.org>.
stdnt-xiao commented on PR #3376:
URL: https://github.com/apache/ambari/pull/3376#issuecomment-1264553771
hi @kevinw66
I modified the compilation process and kill maven where rebuild.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org
[GitHub] [ambari] kevinw66 merged pull request #3376: AMBARI-25746: fix the question of dev-support build docker containers
Posted by GitBox <gi...@apache.org>.
kevinw66 merged PR #3376:
URL: https://github.com/apache/ambari/pull/3376
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org
[GitHub] [ambari] stdnt-xiao commented on pull request #3376: AMBARI-25746: fix the question of dev-support build docker containers
Posted by GitBox <gi...@apache.org>.
stdnt-xiao commented on PR #3376:
URL: https://github.com/apache/ambari/pull/3376#issuecomment-1264334536
@kevinw66
I have finished the adjustment and tested it locally.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org
[GitHub] [ambari] stdnt-xiao closed pull request #3376: fix the question of dev-support build docker containers
Posted by GitBox <gi...@apache.org>.
stdnt-xiao closed pull request #3376: fix the question of dev-support build docker containers
URL: https://github.com/apache/ambari/pull/3376
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org
[GitHub] [ambari] stdnt-xiao commented on a diff in pull request #3376: fix the question of dev-support build docker containers
Posted by GitBox <gi...@apache.org>.
stdnt-xiao commented on code in PR #3376:
URL: https://github.com/apache/ambari/pull/3376#discussion_r983695864
##########
dev-support/docker/centos7/build-containers.sh:
##########
@@ -15,6 +15,12 @@
# See the License for the specific language governing permissions and
# limitations under the License.
+echo -e "\033[32mInstall mvn\033[0m"
+if [ ! "$(command -v mvn)" ]; then
+ echo "mvn not found and will be installed" >&2
+ yum install -y maven
+fi
+
Review Comment:
Install mvn before compiling
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org
[GitHub] [ambari] stdnt-xiao commented on pull request #3376: AMBARI-25746: fix the question of dev-support build docker containers
Posted by GitBox <gi...@apache.org>.
stdnt-xiao commented on PR #3376:
URL: https://github.com/apache/ambari/pull/3376#issuecomment-1264559136
update build-ambari.sh file
build-ambari.sh can only be executed after build-comtains.sh and already exists ambari-rpm-build container
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org