Skip to content

Commit 60348c7

Browse files
committed
Exclude system_tests group when running release prep in GH
GitHub runners don't have libcurl headers installed, which is needed by faraday 1.x -> patron. Omit the system_tests group when running release prep in GH (both GITHUB_ACTIONS will be set and GITHUB_WORKFLOW_REF will contain the absolute path to the workflow file).
1 parent 98b2860 commit 60348c7

1 file changed

Lines changed: 12 additions & 7 deletions

File tree

Gemfile

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -76,13 +76,18 @@ group :development, :release_prep do
7676
gem "puppetlabs_spec_helper", '~> 8.0', require: false
7777
gem "puppet-blacksmith", '~> 7.0', require: false
7878
end
79-
# We are overriding the default PDK template's Litmus logic in order to use the
80-
# latest Bolt in tests. See PA-7824.
81-
group :system_tests do
82-
gem "puppet_litmus", '~> 2.0', require: false, platforms: [:ruby, :x64_mingw]
83-
gem "CFPropertyList", '< 3.0.7', require: false, platforms: [:mswin, :mingw, :x64_mingw]
84-
gem "serverspec", '~> 2.41', require: false
85-
gem "voxpupuli-acceptance", '~> 3', require: false
79+
# On GitHub Actions, only resolve system_tests gems for the task_acceptance_tests
80+
# workflow to avoid native-extension (faraday-patron/libcurl4) failures in
81+
# workflows that don't need them (release_prep, unit_tests, static_code_analysis).
82+
unless ENV['GITHUB_ACTIONS'] && !ENV.fetch('GITHUB_WORKFLOW_REF', '').include?('task_acceptance_tests.yaml')
83+
# We are overriding the default PDK template's Litmus logic in order to use the
84+
# latest Bolt in tests. See PA-7824.
85+
group :system_tests do
86+
gem "puppet_litmus", '~> 2.0', require: false, platforms: [:ruby, :x64_mingw]
87+
gem "CFPropertyList", '< 3.0.7', require: false, platforms: [:mswin, :mingw, :x64_mingw]
88+
gem "serverspec", '~> 2.41', require: false
89+
gem "voxpupuli-acceptance", '~> 3', require: false
90+
end
8691
end
8792

8893
gems = {}

0 commit comments

Comments
 (0)