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.

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

test: improve e2e test script #320

Merged

Conversation

sthaha
Copy link
Collaborator

@sthaha sthaha commented Dec 5, 2023

This PR introduces the following changes

  • improves e2e tests to wait for operator to be ready by creating an invalid-kepler and waiting for the status to be set. This improves the stability of running the test and speeds up running the tests.
  • Allows additional args to be passed to go test so that accidental edits to go test ... can be avoided. Eg.g
./tests/run-e2e.sh --no-deploy -- -run Node

This skips operator deployment (only restarts the already running operator) and runs only Node test.

  • pins kubectl to 1.28.4 while fixing installation of oc overwriting kubectl
  • Prints all version of tools

@sthaha sthaha force-pushed the feat-improve-e2e-script branch 7 times, most recently from cec51a3 to a7ba7e1 Compare December 5, 2023 15:27
@sthaha sthaha requested a review from vprashar2929 December 5, 2023 15:37
@sthaha sthaha force-pushed the feat-improve-e2e-script branch 2 times, most recently from 5bae3e7 to 50c63ab Compare December 6, 2023 01:54
Makefile Show resolved Hide resolved
hack/tools.sh Outdated Show resolved Hide resolved
@@ -87,13 +142,23 @@ install_operator-sdk() {
ok "operator-sdk was installed successfully"
}

version_govulncheck() {
warn govulncheck - latest
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be warn right?

Copy link
Collaborator Author

@sthaha sthaha Dec 6, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it should be warn because the version isn't pinned.

tests/run-e2e.sh Outdated Show resolved Hide resolved
@sthaha sthaha force-pushed the feat-improve-e2e-script branch from 50c63ab to 956e663 Compare December 6, 2023 07:00
Signed-off-by: Sunil Thaha <sthaha@redhat.com>
Signed-off-by: Sunil Thaha <sthaha@redhat.com>
Signed-off-by: Sunil Thaha <sthaha@redhat.com>
@sthaha sthaha force-pushed the feat-improve-e2e-script branch 2 times, most recently from 6e92cf1 to c30f489 Compare December 6, 2023 10:02
@vimalk78
Copy link
Collaborator

vimalk78 commented Dec 6, 2023

/lgtm

Signed-off-by: Sunil Thaha <sthaha@redhat.com>
@sthaha sthaha force-pushed the feat-improve-e2e-script branch from c30f489 to aa5a84c Compare December 6, 2023 20:56
Copy link
Collaborator

@vprashar2929 vprashar2929 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vprashar2929 vprashar2929 merged commit 07e5b38 into sustainable-computing-io:v1alpha1 Dec 7, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants