Skip to content
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.

Sign up for GitHub

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

Jump to bottom

workflow: Reenable L7 tests on EKS + IPsec #22617

Merged
merged 2 commits into from
Dec 9, 2022
Merged

workflow: Reenable L7 tests on EKS + IPsec #22617

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
095b029
workflow: Reenable coverage for EKS+IPsec+L7
pchaigno Dec 8, 2022
d4b695c
workflow: Bump timeout for EKS
pchaigno Dec 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 2 additions & 6 deletions .github/workflows/conformance-eks-v1.10.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ jobs:
github.event_name == 'schedule' ||
github.event_name == 'pull_request'
runs-on: ubuntu-latest
timeout-minutes: 45
timeout-minutes: 60
steps:
- name: Checkout master branch to access local actions
uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8
Expand Down Expand Up @@ -297,11 +297,7 @@ jobs:

- name: Run connectivity test
run: |
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} --force-deploy \
--test '!client-egress-l7,!echo-ingress-l7'
# workaround for L7 tests issues on EKS with IPsec enabled
# TODO: remove both test exceptions once:
# - https://github.com/cilium/cilium/issues/17139 is fixed
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} --force-deploy

- name: Post-test information gathering
if: ${{ !success() }}
Expand Down
8 changes: 2 additions & 6 deletions .github/workflows/conformance-eks-v1.11.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ jobs:
github.event_name == 'schedule' ||
github.event_name == 'pull_request'
runs-on: ubuntu-latest
timeout-minutes: 45
timeout-minutes: 60
steps:
- name: Checkout master branch to access local actions
uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8
Expand Down Expand Up @@ -311,11 +311,7 @@ jobs:

- name: Run connectivity test
run: |
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} --force-deploy \
--test '!client-egress-l7,!echo-ingress-l7'
# workaround for L7 tests issues on EKS with IPsec enabled
# TODO: remove both test exceptions once:
# - https://github.com/cilium/cilium/issues/17139 is fixed
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} --force-deploy

- name: Post-test information gathering
if: ${{ !success() }}
Expand Down
8 changes: 2 additions & 6 deletions .github/workflows/conformance-eks-v1.12.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ jobs:
github.event_name == 'schedule' ||
github.event_name == 'pull_request'
runs-on: ubuntu-latest
timeout-minutes: 45
timeout-minutes: 60
steps:
- name: Checkout master branch to access local actions
uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8
Expand Down Expand Up @@ -311,11 +311,7 @@ jobs:

- name: Run connectivity test
run: |
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} --force-deploy \
--test '!client-egress-l7,!echo-ingress-l7'
# workaround for L7 tests issues on EKS with IPsec enabled
# TODO: remove both test exceptions once:
# - https://github.com/cilium/cilium/issues/17139 is fixed
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} --force-deploy

- name: Post-test information gathering
if: ${{ !success() }}
Expand Down
8 changes: 2 additions & 6 deletions .github/workflows/conformance-eks-v1.13.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ jobs:
github.event_name == 'schedule' ||
github.event_name == 'pull_request'
runs-on: ubuntu-latest
timeout-minutes: 45
timeout-minutes: 60
steps:
- name: Checkout master branch to access local actions
uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8
Expand Down Expand Up @@ -311,11 +311,7 @@ jobs:

- name: Run connectivity test
run: |
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} \
--force-deploy --test '!client-egress-l7,!echo-ingress-l7'
# workaround for L7 tests issues on EKS with IPsec enabled
# TODO: remove both test exceptions once:
# - https://github.com/cilium/cilium/issues/17139 is fixed
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} --force-deploy

- name: Post-test information gathering
if: ${{ !success() }}
Expand Down
8 changes: 2 additions & 6 deletions .github/workflows/conformance-eks.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ jobs:
github.event_name == 'schedule' ||
github.event_name == 'pull_request'
runs-on: ubuntu-latest
timeout-minutes: 45
timeout-minutes: 60
steps:
- name: Checkout master branch to access local actions
uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8
Expand Down Expand Up @@ -314,11 +314,7 @@ jobs:

- name: Run connectivity test
run: |
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} \
--force-deploy --test '!client-egress-l7,!echo-ingress-l7'
# workaround for L7 tests issues on EKS with IPsec enabled
# TODO: remove both test exceptions once:
# - https://github.com/cilium/cilium/issues/17139 is fixed
cilium connectivity test ${{ steps.vars.outputs.connectivity_test_defaults }} --force-deploy

- name: Post-test information gathering
if: ${{ !success() }}
Expand Down