Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Change devtoolset-7 (scl-rh) of CentOS6 #44

Merged
merged 4 commits into from
Oct 15, 2018

Conversation

thinker0
Copy link
Member

Copy link
Contributor

@ridv ridv left a comment

Choose a reason for hiding this comment

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

One minor question and one minor nit but looks good go to me. Thanks @thinker0 ! Will upload the mesos egg as soon as I get a chance to.

)
build_centos6() {
local mesos_version=$1 output_basedir=$2
local python_outdir=$output_basedir/centos/6/python
local image_name=centos:6

pushd "$TMPDIR"
docker run -i -v $(pwd):/mesos-egg $image_name bash <<EOF
docker run -m=2g -i -v $(pwd):/mesos-egg $image_name bash <<EOF
Copy link
Contributor

Choose a reason for hiding this comment

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

Any particular reason CentOS6 requires the docker container to be limited to 2gb's of memory? Would be worth putting this as a comment if that's the case 🙂

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok, Removed

set -e -u

# We need Software Collections for Python 2.7 (we dropped support for 2.6).
# wget must get installed before everything else to fetch the additional repos.
yum -y install centos-release-SCL wget

# Mesos requires gcc 4.8, which is available in devtools-2 on centos 6.
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: We should leave a comment somewhere about why we're installing devtools-7 that way we know where to start looking if things break.

@thinker0
Copy link
Member Author

@ridv ridv merged commit 547dc76 into apache:master Oct 15, 2018
@thinker0 thinker0 deleted the feature/mesos-executor-centos6 branch October 15, 2018 22:19
@thinker0 thinker0 restored the feature/mesos-executor-centos6 branch October 15, 2018 22:41
@thinker0 thinker0 deleted the feature/mesos-executor-centos6 branch October 15, 2018 22:43
@ridv ridv added this to the 0.22.0 milestone Oct 22, 2019
@ridv ridv added the 0.22.0 Belongs to the 0.22.0 release label Oct 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
0.22.0 Belongs to the 0.22.0 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants