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

Add direct download option for PE installers #97

Merged
merged 1 commit into from
May 28, 2020

Conversation

Sharpie
Copy link
Member

@Sharpie Sharpie commented May 21, 2020

This commit adds a new download_mode parameter to the peadm::provision
and peadm::upgrade plans. The default value for the parameter is
bolthost, which causes PE installation content to be downloaded to the
stagingdir of the workstation running bolt and then uploaded to the
PE hosts. Setting download_mode to direct causes PE installation content
to be downloaded directly to the PE hosts. The direct mode skips uploads,
which offers a significant speed boost when the PE hosts have a route to
the internet and the bolt host is connected by a slow link, such as VPN.

This commit adds a new download_mode parameter to the peadm::provision
and peadm::upgrade plans. The default value for the parameter is
bolthost, which causes PE installation content to be downloaded to the
stagingdir of the workstation running bolt and then uploaded to the
PE hosts. Setting download_mode to direct causes PE installation content
to be downloaded directly to the PE hosts. The direct mode skips uploads,
which offers a significant speed boost when the PE hosts have a route to
the internet and the bolt host is connected by a slow link, such as VPN.
@Sharpie Sharpie requested a review from a team as a code owner May 21, 2020 17:43
@ody ody self-requested a review May 28, 2020 19:57
@ody
Copy link
Member

ody commented May 28, 2020

I validated this on both gcp and aws builds through autope, works well. Investigated travis tests and they are passing, no idea why one of the builds are stuck.

Merging.

@ody ody merged commit 6a415d5 into puppetlabs:master May 28, 2020
Copy link
Member

@ody ody left a comment

Choose a reason for hiding this comment

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

Works. Forgot to use review button.

@Sharpie Sharpie deleted the add-direct-download-option branch May 29, 2020 17:16
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