Skip to content

Commit

Permalink
workflows/e2e: Update status-defaults & smoke test
Browse files Browse the repository at this point in the history
The new alert and provider don't have any status. Remove any status
related checks from workflows/e2e tests.

Signed-off-by: Sunny <darkowlzz@protonmail.com>
  • Loading branch information
darkowlzz committed Jun 13, 2023
1 parent 1bc4642 commit ae9a55d
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 13 deletions.
5 changes: 1 addition & 4 deletions .github/workflows/e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ jobs:
- name: Run default status test
run: |
kubectl apply -f config/testdata/status-defaults
for crd in alert provider receiver ; do
for crd in receiver ; do
RESULT=$(kubectl get ${crd} status-defaults -o go-template={{.status}})
EXPECTED='map[observedGeneration:-1]'
if [ "${RESULT}" != "${EXPECTED}" ] ; then
Expand All @@ -87,9 +87,6 @@ jobs:
- name: Run smoke tests
run: |
kubectl -n notification-system apply -f ./config/samples
kubectl -n notification-system wait provider/slack-provider-sample --for=condition=ready --timeout=1m
kubectl -n notification-system wait provider/generic-provider-sample --for=condition=ready --timeout=1m
kubectl -n notification-system wait alert/alert-sample --for=condition=ready --timeout=1m
kubectl -n notification-system wait receiver/receiver-sample --for=condition=ready --timeout=1m
- name: Logs
run: |
Expand Down
5 changes: 0 additions & 5 deletions config/testdata/status-defaults/alert.yaml

This file was deleted.

4 changes: 0 additions & 4 deletions config/testdata/status-defaults/provider.yaml

This file was deleted.

0 comments on commit ae9a55d

Please sign in to comment.