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.

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

Jump to bottom

feat: Adding github action to run the operator end-to-end tests. #4762

Merged
merged 1 commit into from
Nov 15, 2024
Merged

feat: Adding github action to run the operator end-to-end tests. #4762

merged 1 commit into from
Nov 15, 2024

Conversation

lokeshrangineni
Copy link
Contributor

@lokeshrangineni lokeshrangineni commented Nov 14, 2024 •
edited
Loading

  • Adding github action to do the operator end-to-end testing. This action will create a kind cluster and runs the tests on it.

@lokeshrangineni lokeshrangineni changed the title Adding github action to run the end-to-end tests. feat: Adding github action to run the end-to-end tests. Nov 14, 2024
@lokeshrangineni lokeshrangineni force-pushed the feature/operator-action-e2e-test branch from 25c59f4 to a9ae211 Compare November 14, 2024 21:21
@lokeshrangineni lokeshrangineni changed the title feat: Adding github action to run the end-to-end tests. feat: Adding github action to run the operator end-to-end tests. Nov 14, 2024
@lokeshrangineni lokeshrangineni force-pushed the feature/operator-action-e2e-test branch from bda0d8c to 7dbc394 Compare November 14, 2024 21:34
@lokeshrangineni lokeshrangineni marked this pull request as ready for review November 14, 2024 21:35
@lokeshrangineni lokeshrangineni requested a review from a team as a code owner November 14, 2024 21:35
@lokeshrangineni lokeshrangineni self-assigned this Nov 14, 2024
@lokeshrangineni lokeshrangineni force-pushed the feature/operator-action-e2e-test branch 2 times, most recently from 48ca378 to 29d6d7f Compare November 14, 2024 21:55
dmartinol
dmartinol reviewed Nov 15, 2024
View reviewed changes
Copy link
Contributor

@dmartinol dmartinol left a comment

Choose a reason for hiding this comment

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

lgtm

franciscojavierarceo
franciscojavierarceo requested changes Nov 15, 2024
View reviewed changes
Copy link
Member

@franciscojavierarceo franciscojavierarceo left a comment

Choose a reason for hiding this comment

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

Can we separate the actions upgrades from adding the operator? There have been side effects to the release process from upgrading stuff in the actions, so I'd like us to be able to revert or adjust easily without removing the operator.

@lokeshrangineni
Adding github action to do the operator end-to-end testing. This acti…
3638676 
…on will create a kind cluster and runs the tests on it.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
@lokeshrangineni lokeshrangineni force-pushed the feature/operator-action-e2e-test branch from ed61651 to 3638676 Compare November 15, 2024 16:00
@lokeshrangineni
Copy link
Contributor Author

Can we separate the actions upgrades from adding the operator? There have been side effects to the release process from upgrading stuff in the actions, so I'd like us to be able to revert or adjust easily without removing the operator.

I have separated out the warning fixes into separate PR

franciscojavierarceo
franciscojavierarceo reviewed Nov 15, 2024
View reviewed changes
.github/workflows/operator-e2e-integration-tests.yml

services:
kind:
# Specify the Kubernetes version
Copy link
Member

@franciscojavierarceo franciscojavierarceo Nov 15, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

is this the Kind version or the Kubernetes version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

kind version, my understanding is each kind image version is having associated k8 version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have revalidated again, each kind image version is mapped to the same kubernetes version.

dmartinol
dmartinol reviewed Nov 15, 2024
View reviewed changes
Copy link
Contributor

@dmartinol dmartinol left a comment

Choose a reason for hiding this comment

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

lgtm

franciscojavierarceo
franciscojavierarceo approved these changes Nov 15, 2024
View reviewed changes
Copy link
Member

@franciscojavierarceo franciscojavierarceo left a comment

Choose a reason for hiding this comment

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

Lgtm

@franciscojavierarceo franciscojavierarceo merged commit d8ccb00 into feast-dev:master Nov 15, 2024
21 checks passed
shuchu pushed a commit to shuchu/feast that referenced this pull request Nov 21, 2024
@lokeshrangineni @shuchu
feat: Adding github action to run the operator end-to-end tests. (fea…
2f18527 
…st-dev#4762)
franciscojavierarceo pushed a commit that referenced this pull request Dec 5, 2024
@feast-ci-bot
chore(release): release 0.42.0
aac334a 
# [0.42.0](v0.41.0...v0.42.0) (2024-12-05)

### Bug Fixes

* Add adapters for sqlite datetime conversion ([#4797](#4797)) ([e198b17](e198b17))
* Added grpcio extras to default feature-server image ([#4737](#4737)) ([e9cd373](e9cd373))
* Changing node version in release ([7089918](7089918))
* Feast create empty online table when FeatureView attribute online=False ([#4666](#4666)) ([237c453](237c453))
* Fix db store types in Operator CRD ([#4798](#4798)) ([f09339e](f09339e))
* Fix the config issue for postgres ([#4776](#4776)) ([a36f7e5](a36f7e5))
* Fixed example materialize-incremental and improved explanation ([#4734](#4734)) ([ca8a7ab](ca8a7ab))
* Fixed SparkSource docstrings so it wouldn't used inhereted class docstrings ([#4722](#4722)) ([32e6aa1](32e6aa1))
* Fixing PGVector integration tests ([#4778](#4778)) ([88a0320](88a0320))
* Incorrect type passed to assert_permissions in materialize endpoints ([#4727](#4727)) ([b72c2da](b72c2da))
* Issue of DataSource subclasses using parent abstract class docstrings ([#4730](#4730)) ([b24acd5](b24acd5))
* Operator envVar positioning & tls.SecretRef.Name ([#4806](#4806)) ([1115d96](1115d96))
* Populates project created_time correctly according to created ti… ([#4686](#4686)) ([a61b93c](a61b93c))
* Reduce feast-server container image size & fix dev image build ([#4781](#4781)) ([ccc9aea](ccc9aea))
* Removed version func from feature_store.py ([#4748](#4748)) ([f902bb9](f902bb9))
* Support registry instantiation for read-only users ([#4719](#4719)) ([ca3d3c8](ca3d3c8))
* Syntax Error in BigQuery While Retrieving Columns that Start wit… ([#4713](#4713)) ([60fbc62](60fbc62))
* Update release version in a pertinent Operator file ([#4708](#4708)) ([764a8a6](764a8a6))

### Features

* Add api contract to fastapi docs ([#4721](#4721)) ([1a165c7](1a165c7))
* Add Couchbase as an online store ([#4637](#4637)) ([824859b](824859b))
* Add Operator support for spec.feastProject & status.applied fields ([#4656](#4656)) ([430ac53](430ac53))
* Add services functionality to Operator ([#4723](#4723)) ([d1d80c0](d1d80c0))
* Add TLS support to the Operator ([#4796](#4796)) ([a617a6c](a617a6c))
* Added feast Go operator db stores support ([#4771](#4771)) ([3302363](3302363))
* Added support for setting env vars in feast services in feast controller  ([#4739](#4739)) ([84b24b5](84b24b5))
* Adding docs outlining native Python transformations on singletons ([#4741](#4741)) ([0150278](0150278))
* Adding first feast operator e2e test. ([#4791](#4791)) ([8339f8d](8339f8d))
* Adding github action to run the operator end-to-end tests. ([#4762](#4762)) ([d8ccb00](d8ccb00))
* Adding ssl support for registry server. ([#4718](#4718)) ([ccf7a55](ccf7a55))
* Adding SSL support for the React UI server and feast UI command. ([#4736](#4736)) ([4a89252](4a89252))
* Adding support for native Python transformations on a single dictionary  ([#4724](#4724)) ([9bbc1c6](9bbc1c6))
* Adding TLS support for offline server. ([#4744](#4744)) ([5d8d03f](5d8d03f))
* Building the feast image ([#4775](#4775)) ([6635dde](6635dde))
* File persistence definition and implementation ([#4742](#4742)) ([3bad4a1](3bad4a1))
* Object store persistence in operator ([#4758](#4758)) ([0ae86da](0ae86da))
* OIDC authorization in Feast Operator ([#4801](#4801)) ([eb111d6](eb111d6))
* Operator will create k8s serviceaccount for each feast service ([#4767](#4767)) ([cde5760](cde5760))
* Printing more verbose logs when we start the offline server  ([#4660](#4660)) ([9d8d3d8](9d8d3d8))
* PVC configuration and impl ([#4750](#4750)) ([785a190](785a190))
* Qdrant vectorstore support ([#4689](#4689)) ([86573d2](86573d2))
* RBAC Authorization in Feast Operator ([#4786](#4786)) ([0ef5acc](0ef5acc))
* Support for nested timestamp fields in Spark Offline store ([#4740](#4740)) ([d4d94f8](d4d94f8))
* Update the go feature server from Expedia code repo. ([#4665](#4665)) ([6406625](6406625))
* Updated feast Go operator db stores ([#4809](#4809)) ([2c5a6b5](2c5a6b5))
* Updated sample secret following review ([#4811](#4811)) ([dc9f825](dc9f825))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmartinol dmartinol dmartinol left review comments

@franciscojavierarceo franciscojavierarceo franciscojavierarceo approved these changes

Assignees

@lokeshrangineni lokeshrangineni

Labels
ok-to-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lokeshrangineni @franciscojavierarceo @dmartinol