-
Notifications
You must be signed in to change notification settings - Fork 495
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
(FACT-1281) Simplify acceptance tests #1255
(FACT-1281) Simplify acceptance tests #1255
Conversation
Previously, `rake ci:test:packages` was the default rake target, but was unusable, because we don't build non-AIO packages anymore. This commit removes support for that rake task and changes the default to `aio`.
Previously, the presuite for AIO called `install_repos_on` for all host platforms, which was implemented in facter's install_utils helper. For redhat and debian platforms the method installed the corresponding repo_config. For other platforms, e.g. Windows, OSX, it installed puppet-agent using `install_puppet_agent_dev_repo_on`. For redhat and debian platforms, the presuite then called Beaker's `install_packages_on` method to execute `yum install puppet-agent`, etc. This commit modifies the presuite to install puppet-agent using Beaker's `install_puppet_agent_dev_repo_on` on all platforms. On rpm/deb platforms, it installs the repo_config and uses the package manager to install puppet-agent, e.g. `yum install -y puppet-agent`. The previous commit removed `packages` style acceptance testing, and `aio` no longer calls `install_repos_on`, so this commit removes it along with related helper code. This commit bumps our Beaker dependency to use the latest `install_puppet_agent_dev_repo` method.
verified using:
|
@er0ck @johnduarte @MikaelSmith similar changes to facter |
@@ -10,7 +10,7 @@ def location_for(place, fake_version = nil) | |||
end | |||
end | |||
|
|||
gem "beaker", *location_for(ENV['BEAKER_VERSION'] || "~> 2.27") | |||
gem "beaker", *location_for(ENV['BEAKER_VERSION'] || "~> 2.32") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are you trying to prevent this from updating Y version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wanting to ensure we install at least 2.32.0 (but less than 3.0) since 2.32 is the first version that contains the correct install_puppet_agent_dev_repo_on
implementation.
LGTM 👍 |
👍 |
…lify-acceptance-tests (FACT-1281) Simplify acceptance tests
Passed on AIX 6.1 |
This commit removes support for
packages
type of acceptance, bumps our beaker dependency, and uses beaker's updatedinstall_puppet_agent_dev_repo_on
method to install puppet-agent on all platforms.