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

BIGTOP-4056. Support Detect Python Path #1223

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vivostar
Copy link
Contributor

@vivostar vivostar commented Dec 26, 2023

Description of PR

According to the descussion on the email,we should add a python path detecting script for all bigtop componets.

How was this patch tested?

  1. build the bigtop-utils package
docker run --rm -v `pwd`:/ws --workdir /ws bigtop/slaves:trunk-centos-7 bash -l -c './gradlew allclean ; ./gradlew bigtop-utils'
  1. make repo
./gradlew repo
  1. test bigtop-utils package by docker-hadoop.sh
./docker-hadoop.sh -dcp -d -k 'bigtop-utils' -L -r file:///bigtop-home/output -c 1
  1. login a container, and install python3 package from default distro repo or from source

(1) install from default repo

yum install -y python3

(2) install from source

# install build dependencies
yum install -y zlib-devel bzip2-devel openssl-devel ncurses-devel sqlite-devel readline-devel
yum install -y tk-devel gdbm-devel db4-devel libpcap-devel xz-devel libffi-devel
# get python3 source code
wget https://www.python.org/ftp/python/3.7.6/Python-3.7.6.tgz
tar xzvf Python-3.7.6.tgz
cd Python-3.7.6
./configure --prefix=/opt/python3
make && make install
  1. check the python3 path detecting
. /usr/lib/bigtop-utils/bigtop-detect-pythonpath
echo $PAYTHON3_PATH

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
  • Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

@vivostar
Copy link
Contributor Author

test it on centos-7 and ubuntu-22.04

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.

1 participant