Skip to content

Commit

Permalink
CI Fixes
Browse files Browse the repository at this point in the history
Signed-off-by: Lance Albertson <lance@osuosl.org>
  • Loading branch information
ramereth committed Aug 3, 2024
1 parent 8bc7bcf commit 737fd5b
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 24 deletions.
8 changes: 7 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ jobs:

integration-smoke:
needs: lint-unit
runs-on: macos-12
runs-on: ubuntu-latest
strategy:
matrix:
os:
Expand All @@ -119,6 +119,12 @@ jobs:
steps:
- name: Check out code
uses: actions/checkout@v4
- name: Install VirtualBox & Vagrant
run: |
sudo apt update && sudo apt install virtualbox
wget -O- https://apt.releases.hashicorp.com/gpg | sudo gpg --dearmor -o /usr/share/keyrings/hashicorp-archive-keyring.gpg
echo "deb [signed-by=/usr/share/keyrings/hashicorp-archive-keyring.gpg] https://apt.releases.hashicorp.com $(lsb_release -cs) main" | sudo tee /etc/apt/sources.list.d/hashicorp.list
sudo apt update && sudo apt install vagrant
- name: Install Chef
uses: actionshub/chef-install@3.0.0
- name: Dokken
Expand Down
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
## Unreleased

- Add `none` as an option to `service_manager` to allow using the system defaults
- Switch to running vagrant+virtualbox on Ubuntu via nested virtualization for smoke tests
- Fix package installation tests

## 11.4.2 - *2024-07-16*

Expand Down
3 changes: 2 additions & 1 deletion kitchen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,11 @@ driver:
provisioner:
name: chef_infra
product_name: <%= ENV['CHEF_PRODUCT_NAME'] || 'chef' %>
product_version: <%= ENV['CHEF_VERSION'] || '17' %>
product_version: <%= ENV['CHEF_VERSION'] || '18' %>
enforce_idempotency: true
multiple_converge: 2
deprecations_as_errors: true
chef_license: accept-no-persist

verifier:
name: inspec
Expand Down
12 changes: 0 additions & 12 deletions test/cookbooks/docker_test/recipes/installation_package.rb
Original file line number Diff line number Diff line change
@@ -1,15 +1,3 @@
docker_ver =
# Include epoch on RHEL to fix idempotency issues
if platform_family?('rhel', 'fedora')
'3:20.10.11'
# Debian 9 does not include 20.10
elsif platform?('debian') && node['platform_version'].to_i == 9
'19.03.14'
else
'20.10.11'
end

docker_installation_package 'default' do
version docker_ver
action :create
end
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
if os.family == 'redhat' && os.release.to_i == 8
describe command('/usr/bin/docker --version') do
its(:exit_status) { should eq 0 }
its(:stdout) { should match(/26\.1\./) }
end
else
describe command('/usr/bin/docker --version') do
its(:exit_status) { should eq 0 }
its(:stdout) { should match(/27\.1\./) }
end
describe command('/usr/bin/docker --version') do
its(:exit_status) { should eq 0 }
its(:stdout) { should match(/2[0-9]\.[0-9]+\./) }
end

0 comments on commit 737fd5b

Please sign in to comment.