Skip to content

Commit

Permalink
Disable e2e tests for now
Browse files Browse the repository at this point in the history
Disable the vagrant based e2e tests until
actions/runner-images#8730 is resolved.

Signed-off-by: Sascha Grunert <[email protected]>
  • Loading branch information
saschagrunert committed Nov 15, 2023
1 parent c33f280 commit 0c5f0a7
Show file tree
Hide file tree
Showing 4 changed files with 54 additions and 41 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/helm-chart-lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
python-version: 3.7

- name: Set up chart-testing
uses: helm/chart-testing-action@e8788873172cb653a90ca2e819d79d65a66d4e76 # v2.4.0
uses: helm/chart-testing-action@b43128a8b25298e1e7b043b78ea6613844e079b1 # v2.6.0

- name: Run chart-testing (lint)
run: >
Expand Down
81 changes: 45 additions & 36 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,10 @@ jobs:
path: image.tar

e2e-fedora:
# TODO(saschagrunert): re-enable when
# https://github.com/actions/runner-images/issues/8730 is resolved
if: false
needs: image

runs-on: macos-12
timeout-minutes: 90
env:
Expand Down Expand Up @@ -86,8 +88,10 @@ jobs:
$RUN hack/ci/e2e-fedora.sh
e2e-ubuntu:
# TODO(saschagrunert): re-enable when
# https://github.com/actions/runner-images/issues/8730 is resolved
if: false
needs: image

runs-on: macos-12
timeout-minutes: 90
env:
Expand Down Expand Up @@ -116,42 +120,47 @@ jobs:
$RUN hack/ci/e2e-ubuntu.sh
e2e-flatcar:
needs: image

runs-on: macos-12
timeout-minutes: 90
env:
RUN: ./hack/ci/run-flatcar.sh
steps:
- uses: actions/checkout@8ade135a41bc03ea155e62e844d188df1ea18608 # v3.5.2
- name: Vagrant box version
id: vagrant-box
run: |
echo "version=$(curl -s https://stable.release.flatcar-linux.net/amd64-usr/current/flatcar_production_vagrant.json | jq '.versions[0].version' | tr -d '".')" >> $GITHUB_OUTPUT
shell: bash
- name: Upgrade vagrant box
run: |
ln -sf hack/ci/Vagrantfile-flatcar Vagrantfile
vagrant box update
- uses: actions/download-artifact@9bc31d5ccc31df68ecc42ccf4149144866c47d8a
with:
name: image
path: .
- name: Boot Virtual Machine
run: make vagrant-up-flatcar
- name: Show environment information
run: |
$RUN kubectl wait --for=condition=ready --timeout=600s node localhost
$RUN kubectl get nodes -o wide
- name: Run E2E tests
run: $RUN hack/ci/e2e-flatcar-dev-container.sh
- name: Run Flaky E2E tests
continue-on-error: true
run: |
$RUN "echo \"export E2E_TEST_FLAKY_TESTS_ONLY=true\" >> /vagrant/hack/ci/env-flatcar.sh"
$RUN hack/ci/e2e-flatcar-dev-container.sh
# TODO(saschagrunert): re-enable when
# https://github.com/actions/runner-images/issues/8730 is resolved
if: false
needs: image
runs-on: macos-12
timeout-minutes: 90
env:
RUN: ./hack/ci/run-flatcar.sh
steps:
- uses: actions/checkout@8ade135a41bc03ea155e62e844d188df1ea18608 # v3.5.2
- name: Vagrant box version
id: vagrant-box
run: |
echo "version=$(curl -s https://stable.release.flatcar-linux.net/amd64-usr/current/flatcar_production_vagrant.json | jq '.versions[0].version' | tr -d '".')" >> $GITHUB_OUTPUT
shell: bash
- name: Upgrade vagrant box
run: |
ln -sf hack/ci/Vagrantfile-flatcar Vagrantfile
vagrant box update
- uses: actions/download-artifact@9bc31d5ccc31df68ecc42ccf4149144866c47d8a
with:
name: image
path: .
- name: Boot Virtual Machine
run: make vagrant-up-flatcar
- name: Show environment information
run: |
$RUN kubectl wait --for=condition=ready --timeout=600s node localhost
$RUN kubectl get nodes -o wide
- name: Run E2E tests
run: $RUN hack/ci/e2e-flatcar-dev-container.sh
- name: Run Flaky E2E tests
continue-on-error: true
run: |
$RUN "echo \"export E2E_TEST_FLAKY_TESTS_ONLY=true\" >> /vagrant/hack/ci/env-flatcar.sh"
$RUN hack/ci/e2e-flatcar-dev-container.sh
baseprofiles:
# TODO(saschagrunert): re-enable when
# https://github.com/actions/runner-images/issues/8730 is resolved
if: false
needs: image
runs-on: macos-12
timeout-minutes: 90
Expand Down
11 changes: 7 additions & 4 deletions test/e2e_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,13 @@ func (e *e2e) TestSecurityProfilesOperator() {
"Seccomp: Verify base profile merge",
e.testCaseBaseProfile,
},
{
"Seccomp: Verify base profile merge from OCI registry",
e.testCaseBaseProfileOCI,
},
// TODO: re-enable when we found a workaround to the flaky GitHub registry connection
/*
{
"Seccomp: Verify base profile merge from OCI registry",
e.testCaseBaseProfileOCI,
},
*/
{
"Seccomp: Allowed syscalls",
e.testCaseAllowedSyscalls,
Expand Down
1 change: 1 addition & 0 deletions test/tc_base_profiles_oci_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"time"
)

//nolint:unused // test is flaky and therefore got deactivated
func (e *e2e) testCaseBaseProfileOCI([]string) {
e.seccompOnlyTestCase()

Expand Down

0 comments on commit 0c5f0a7

Please sign in to comment.