Skip to content

Freshjenkins#3

Open
fapestniegd wants to merge 4 commits intomasterfrom
freshjenkins
Open

Freshjenkins#3
fapestniegd wants to merge 4 commits intomasterfrom
freshjenkins

Conversation

@fapestniegd
Copy link
Owner

Added functionality to jenkins module so that it grabs the freshest version from jenkins.org.


# Execute line for Adding Jenkins gpg signature to host
exec {'add_jenkins_signature':
command => "/usr/bin/wget -q -O - http://pkg.jenkins-ci.org/debian/jenkins-ci.org.key | /usr/bin/apt-key add -"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you make this not run wget every time?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Sure. Could I get some help on re-writing the line? Not sure exactly how to go about doing that aside from possibly adding another file with the key directly.

Copy link
Collaborator

Choose a reason for hiding this comment

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

That's exactly what' I'd do. Add it as a file resource.

Copy link
Owner Author

Choose a reason for hiding this comment

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

But what if the file resource changes?

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