Skip to content

feat: Operator E2E test to validate FeatureStore custom resource using remote registry #4822

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

Merged
tchughesiv merged 12 commits into from
Dec 10, 2024
Merged

feat: Operator E2E test to validate FeatureStore custom resource using remote registry #4822

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
cbdf03d
Added new e2e test case to do the remote registry deployment.
lokeshrangineni Dec 5, 2024
c861b91
abstracted the code even further. Now the custom resource CR will exe…
lokeshrangineni Dec 6, 2024
8b52915
fixing the operator e2e test. this was commented before by mistake.
lokeshrangineni Dec 6, 2024
0c32bf7
increasing the remote registry deployment timeout to see if it solves…
lokeshrangineni Dec 6, 2024
cd72ce1
Trying to increase the timeout and also adding the debugging actions …
lokeshrangineni Dec 6, 2024
4db23f8
increased the go test timeout to 30m to fix the github operator e2e t…
lokeshrangineni Dec 6, 2024
1e551d3
increased the go test timeout to 30m to fix the github operator e2e t…
lokeshrangineni Dec 6, 2024
0070ac8
increased the go test timeout to 30m to fix the github operator e2e t…
lokeshrangineni Dec 6, 2024
60c10b1
incorporating the code review comments.
lokeshrangineni Dec 10, 2024
41acec6
incorporating the code review comments by using the feastRef.
lokeshrangineni Dec 10, 2024
7db4bf5
incorporating the code review comments by marshaling json to the go s…
lokeshrangineni Dec 10, 2024
2d84965
Fixing the lint error.
lokeshrangineni Dec 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
increased the go test timeout to 30m to fix the github operator e2e t… 
…est action.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
  • Loading branch information
@lokeshrangineni
lokeshrangineni committed Dec 7, 2024
commit 0070ac8ed095cbcc5d0d19f870398908c338b434
2 changes: 1 addition & 1 deletion infra/feast-operator/test/e2e/e2e_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ var _ = Describe("controller", Ordered, func() {

remoteFeatureStoreName := "simple-feast-remote-setup"

validateTheFeatureStoreCustomResource(remoteRegistryNs, remoteFeatureStoreName, 5*time.Minute)
validateTheFeatureStoreCustomResource(remoteRegistryNs, remoteFeatureStoreName, timeout)

By("deleting the feast remote registry deployment")
cmd = exec.Command("kubectl", "delete", "-f",
Expand Down