Skip to content

Commit

Permalink
Replace c54xlarge instance with c59xlarge on Jenkins Main Node (#347)
Browse files Browse the repository at this point in the history
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
  • Loading branch information
peterzhuamazon authored Sep 8, 2023
1 parent 4a3bf1e commit 6e57cb3
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion lib/compute/jenkins-main-node.ts
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ export class JenkinsMainNode {
this.EFS_ID = efs.fileSystemId;
}
this.mainNodeAsg = new AutoScalingGroup(stack, 'MainNodeAsg', {
instanceType: InstanceType.of(InstanceClass.C5, InstanceSize.XLARGE4),
instanceType: InstanceType.of(InstanceClass.C5, InstanceSize.XLARGE9),
machineImage: MachineImage.latestAmazonLinux({
generation: AmazonLinuxGeneration.AMAZON_LINUX_2,
}),
Expand Down
10 changes: 5 additions & 5 deletions resources/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,18 +11,18 @@ services:
- 50000:50000
container_name: jenkins
environment:
- JENKINS_JAVA_OPTS="-Xmx18g"
- JENKINS_JAVA_OPTS="-Xmx60g -Xss4m"
- CASC_RELOAD_TOKEN=reloadPasswordHere
volumes:
- /var/lib/jenkins:/var/jenkins_home
deploy:
resources:
limits:
cpus: '12'
memory: '20g'
cpus: '32'
memory: '64g'
reservations:
cpus: '12'
memory: '20g'
cpus: '32'
memory: '64g'
logging:
driver: awslogs
options:
Expand Down
2 changes: 1 addition & 1 deletion test/ci-stack.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ test('MainNode', () => {

// THEN
Template.fromStack(stack).hasResourceProperties('AWS::AutoScaling::LaunchConfiguration', {
InstanceType: 'c5.4xlarge',
InstanceType: 'c5.9xlarge',
SecurityGroups: [
{
'Fn::GetAtt': [
Expand Down

0 comments on commit 6e57cb3

Please sign in to comment.