Skip to content

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

Merged
tchughesiv merged 12 commits into
feast-dev:masterfrom
lokeshrangineni:feature/operator-remote-e2e-test
Dec 10, 2024
Merged

feat: Operator E2E test to validate FeatureStore custom resource using remote registry#4822
tchughesiv merged 12 commits into
feast-dev:masterfrom
lokeshrangineni:feature/operator-remote-e2e-test

Conversation

@lokeshrangineni

@lokeshrangineni lokeshrangineni commented Dec 6, 2024

Copy link
Copy Markdown
Contributor

We have added the operator E2E test to cover following scenario.

  • Deploy the feature store custom resource in one of of the kubernetes namespace.
  • Deploy the another feature store custom resource in another namespace but should point to the registry using remote setting which should refer to the registry service created in the previous step.
  • Increase the go test timeout so that github ci will succeed without time out issue. Also added the kind cluster logs when there is a failure of operator e2e test github action so it will help in troubleshooting the issue.

Issue related to 4823

lrangine added 2 commits December 5, 2024 16:42
abstracted the code to do the featurestore testing.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
…cute only once for all the test cases.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
@lokeshrangineni lokeshrangineni changed the title Operator E2E test to validate FeatureStore custom resource using remote registry feat: Operator E2E test to validate FeatureStore custom resource using remote registry Dec 6, 2024
@lokeshrangineni lokeshrangineni self-assigned this Dec 6, 2024
@lokeshrangineni
lokeshrangineni marked this pull request as ready for review December 6, 2024 06:30
@lokeshrangineni
lokeshrangineni requested a review from a team as a code owner December 6, 2024 06:30
lrangine added 6 commits December 5, 2024 23:34
Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
… the github CI timeout issue.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
…when there is a failure.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
…est action.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
…est action.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
…est action.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
lrangine added 2 commits December 10, 2024 12:06
Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
Comment thread infra/feast-operator/test/e2e/test_util.go Outdated
lrangine added 2 commits December 10, 2024 15:02
…truct object rather than map.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>

ghost left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@tchughesiv
tchughesiv enabled auto-merge (squash) December 10, 2024 22:19
@tchughesiv
tchughesiv merged commit d558ef7 into feast-dev:master Dec 10, 2024
@lokeshrangineni
lokeshrangineni deleted the feature/operator-remote-e2e-test branch January 22, 2025 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants