-
Notifications
You must be signed in to change notification settings - Fork 194
(PA-7222) Add Amazon FIPS 2023 (x86_64) support in task acceptance tests #767
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
Conversation
6dc83e0
to
7750793
Compare
Hi @bastelfreak, @joshcooper |
@shubhamshinde360 could you update your commit message to describe what this change is doing? It's somewhat unexpected that we're "adding support" for this new platform, by skipping the upgrade test (since there isn't a previous version to upgrade from). Were there really no other changes needed for Amazon 2023 FIPS? |
7750793
to
e499c6f
Compare
Thanks @joshcooper, |
This commit adds Amazon FIPS 2023 (x86_64) support to task acceptance tests. Support for puppet7 to puppet8 upgrade tests has not been added since puppet7 is EOL and its packages were not built for the platform.
e499c6f
to
dd32e84
Compare
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.
If you haven't already, could you verify that the task acceptance tests pass with this change? The test already installs "latest_sources" from artifactory, since we point yum_source
there:
puppetlabs-puppet_agent/task_spec/spec/acceptance/init_spec.rb
Lines 53 to 72 in 14b1844
def latest_sources | |
{ | |
'yum_source' => 'https://artifactory.delivery.puppetlabs.net:443/artifactory/internal_nightly__local/yum', | |
'apt_source' => 'https://artifactory.delivery.puppetlabs.net:443/artifactory/internal_nightly__local/apt', | |
'mac_source' => 'https://artifactory.delivery.puppetlabs.net:443/artifactory/internal_nightly__local/downloads', | |
'windows_source' => 'https://artifactory.delivery.puppetlabs.net:443/artifactory/internal_nightly__local/downloads', | |
} | |
end | |
it 'works with version and install tasks' do | |
case target_platform | |
when latest_platform_list | |
# Here we only install puppet-agent 8.x from nightlies as we don't support 7.x | |
# We have to consider tests to upgrade puppet 8.x to latest nightlies in future | |
# Install an puppet8 nightly version | |
results = run_task('puppet_agent::install', 'target', { 'collection' => 'puppet8-nightly', | |
'version' => 'latest', | |
'stop_service' => true }.merge(latest_sources)) | |
Yes, Josh. I have already tested the task acceptance tests as part of https://perforce.atlassian.net/wiki/spaces/AGENT/pages/369365152/How+to+add+a+platform+for+puppet-agent#Howtoaddaplatformforpuppet-agent-3.Updatepuppet_agentmoduletaskacceptancepe_puppet_agent
|
This commit adds Amazon FIPS 2023 (x86_64) support to task acceptance tests. Support for puppet7 to puppet8 upgrade tests has not been added since puppet7 is EOL and its packages were not built for the platform.