Skip to content

Commit

Permalink
feat: add tests to verify node label is added as expected (#2384)
Browse files Browse the repository at this point in the history
Co-authored-by: Ben Elferink <[email protected]>
  • Loading branch information
tamirdavid1 and BenElferink authored Feb 4, 2025
1 parent 3a7d303 commit 779ef25
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 0 deletions.
5 changes: 5 additions & 0 deletions tests/common/assert/node-odiglet-label.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
apiVersion: v1
kind: Node
metadata:
labels:
odigos.io/odiglet-oss-installed: "true"
6 changes: 6 additions & 0 deletions tests/e2e/helm-chart/chainsaw-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,12 @@ spec:
fi
- assert:
file: ../../common/assert/odigos-installed.yaml

- name: Verify Node Odiglet label has been added
try:
- assert:
file: ../../common/assert/node-odiglet-label.yaml

- name: Install Demo App
try:
- script:
Expand Down
5 changes: 5 additions & 0 deletions tests/e2e/multi-apps/chainsaw-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,11 @@ spec:
- assert:
file: ../../common/assert/odigos-installed.yaml

- name: Verify Node Odiglet label has been added
try:
- assert:
file: ../../common/assert/node-odiglet-label.yaml

- name: Install Demo App
try:
- script:
Expand Down
6 changes: 6 additions & 0 deletions tests/e2e/workload-lifecycle/chainsaw-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,12 @@ spec:
timeout: 60s
- assert:
file: ../../common/assert/odigos-installed.yaml

- name: Verify Node Odiglet label has been added
try:
- assert:
file: ../../common/assert/node-odiglet-label.yaml

- name: "01 - Install Test Apps"
try:
- apply:
Expand Down

0 comments on commit 779ef25

Please sign in to comment.