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

Separate macOS builder #207

Merged
merged 12 commits into from
Oct 4, 2021
Merged

Separate macOS builder #207

merged 12 commits into from
Oct 4, 2021

Conversation

britzl
Copy link
Contributor

@britzl britzl commented Sep 28, 2021

No description provided.

@@ -50,7 +49,8 @@ start_service() {
fi
fi

nohup java -Xmx1g -XX:MaxDirectMemorySize=512m -jar ${PATH_TO_JAR} --spring.profiles.active=${PROFILE} >> ${STDOUT_LOG} 2>> ${ERROR_LOG} < /dev/null &
echo "Running: java -Xmx1g -XX:MaxDirectMemorySize=512m -jar ${PATH_TO_JAR} --spring.profiles.active=${PROFILE} >> ${STDOUT_LOG} 2>> ${ERROR_LOG} < /dev/null &"
java -Xmx1g -XX:MaxDirectMemorySize=512m -jar ${PATH_TO_JAR} --spring.profiles.active=${PROFILE} >> ${STDOUT_LOG} 2>> ${ERROR_LOG} < /dev/null &
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got "nohup: can't detach from console: Inappropriate ioctl for device"

Recommendation seems to be to remove nohup. The server is still running after the session has ended.

@@ -35,7 +40,7 @@ if [[ ! -e ${LOGS_DIR} ]]; then
echo "[setup] Created logs directory at ${LOGS_DIR}."
fi

WGET_CMD=/usr/local/bin/wget
CURL_CMD=/usr/bin/curl
Copy link
Contributor Author

Choose a reason for hiding this comment

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

wget is not installed by default on the instance. curl is.

@britzl britzl requested a review from JCash September 28, 2021 11:46
Comment on lines +5 to +11
* Create macOS instance in AWS Console
* 100 GB storage (default is 60 GB)
* Key pair: `defold2_ec2`
* VPC: `vpc-f5e04690` Subnet: `subnet-38e0725d`
* Public IPV4
* Security Groups: `sg-bd9926da`, `sg-f9734c9f`
* Add instance to EC2 Target Group `prod-darwin-extender` (`arn:aws:elasticloadbalancing:eu-west-1:378120250811:targetgroup/prod-darwin-extender/4a33b7844a4ad624`)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it an issue putting this info online?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No. I wouldn't say so. The information uniquely identifies the load balancer, a virtual private cloud and subnet. The security groups are also of no use to anyone. And access to detailed information is only available to someone with access to our AWS account.

Follow [these setup instructions](README_SETUP_RELEASE.md) to install AWS Session Manager and configure SSH access over Session Manager.

```
ssh -i ~/.ssh/defold2_ec2.pem ec2-user@i-05303831267023ab0
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here?

Previously, we've used google drive documents for this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope, ec2-user is the default user on any mac1.metal host (it is public information easily found in the AWS documentation). And knowing the instance id is of no help to anyone at all, and it is not a security concern either.

Comment on lines +28 to +35
# From Dockerfile
export XCODE_12_VERSION=12.5
export MACOS_10_15_VERSION=10.15
export IOS_14_VERSION=14.5
export LIB_TAPI_1_6_PATH=/usr/local/tapi1.6/lib
export MACOS_11_VERSION=11.3
export XCODE_12_CLANG_VERSION=12.0.5
export SWIFT_5_VERSION=5.0
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd like a comment in the Dockerfile referencing this file as well, since it's duplicated code and easy for it to diverge

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point. I've updated the PR with this addition to the Dockerfile

enabled: false
url: https://build-darwin-stage.defold.com
enabled: true
url: https://build-darwin.defold.com
Copy link
Contributor

Choose a reason for hiding this comment

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

We really have to be extra careful now, since we cannot test things as we used to.

@JCash JCash merged commit 9053031 into dev Oct 4, 2021
@JCash JCash deleted the dev-build-darwin branch October 4, 2021 07:38
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