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

Fixed the CloudFormation template to complete the Jenkins setup #190

Merged
merged 3 commits into from
May 13, 2022

Conversation

chrismld
Copy link
Contributor

Issue #, if available: #157

Description of changes:
The existing CloudFormation template is installing the most recent version of Jenkins. The problem is that the userdata script is restoring a Jenkins back from an older version. I changed the install instructions to specify the Jenkins version. The one that the backup has is 2.138.4, and because it's a very old version, I had to add the --nogpgcheck. We should work on update the workshop to use a more recent version to avoid this problem.

I also had to change the instructions to download the CloudFormation template from GitHub and not from S3.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@chrismld chrismld changed the title Fixed the CloudFormation template to complete the Jenkins setup #157 Fixed the CloudFormation template to complete the Jenkins setup May 11, 2022
@jagpk
Copy link
Contributor

jagpk commented May 12, 2022

image

Health for Jenkins Master is failing. Please test at your end confirm if it works.

Copy link
Contributor

@jagpk jagpk left a comment

Choose a reason for hiding this comment

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

Approving PR with pending changes related to SLR and AMI to be covered in future PRs.

@jagpk jagpk merged commit 3d755f3 into awslabs:master May 13, 2022
@chrismld chrismld deleted the issue-157 branch July 21, 2022 11:41
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