Skip to content

Allow mysqltuner to be downloaded via a proxy #917

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

Merged
merged 3 commits into from
Dec 23, 2016

Conversation

Boran
Copy link
Contributor

@Boran Boran commented Dec 7, 2016

Extend mysql::server::mysqltuner with an $environment, which is then passed onto staging::file, so that a proxy variable can be passed that allows mysqltuner to be downloaded. Example usage.

  class { 'mysql::server::mysqltuner':
    environment => 'https_proxy=http://myproxy.example.ch:80',    # for download
  }

Extend mysql::server::mysqltuner with an $environment, which is then passed onto staging::file, so that a proxy variable can be passed that alllows mysqltuner to be downloaded. Example usage.
'''
  class { 'mysql::server::mysqltuner':
    environment => 'https_proxy=http://myproxy.example.ch:80',    # for download
  }
'''
bmjen
bmjen previously requested changes Dec 15, 2016
@@ -37,6 +38,7 @@

staging::file { "mysqltuner-${_version}":
source => $_source,
environment => $environment,
Copy link
Contributor

Choose a reason for hiding this comment

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

Indentation needs to be fixed for this in order to pass lint.

@bmjen
Copy link
Contributor

bmjen commented Dec 15, 2016

@Boran Thanks for the PR! The new parameter needs to be added to the README. Also puppet-lint check is failing and needs to be addressed.

@tphoney tphoney dismissed bmjen’s stale review December 23, 2016 13:22

Subsequent action has been taken

@tphoney
Copy link
Contributor

tphoney commented Dec 23, 2016

Thanks for the PR, this seems to be a common issue.

@tphoney tphoney merged commit 9442272 into puppetlabs:master Dec 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants