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

chore: Fix python-sdk workflow #4958

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

tchughesiv
Copy link
Contributor

What this PR does / why we need it:

Which issue(s) this PR fixes:

Misc

Signed-off-by: Tommy Hughes <tohughes@redhat.com>
@tchughesiv tchughesiv marked this pull request as ready for review January 22, 2025 17:10
@franciscojavierarceo franciscojavierarceo merged commit e93d164 into feast-dev:v0.43-branch Jan 22, 2025
22 checks passed
franciscojavierarceo added a commit that referenced this pull request Jan 23, 2025
* fix: Update workflows to also use workflow_call

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Also adding workflow_call to release version

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Adding type to release version input

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* chore: Fix operator release workflow (v0.43-branch) (#4937)

fix operator release workflow

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* fix: Updated type in yaml

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Updating publish.yml

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Update type in publish_images

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Updating publish helm charts

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Updating publish java sdk

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Updating environment variable in the build_wheels

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updating input parameters to build wheels

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* formatting issue

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* chore: Fix operator release for 0.43.0 (v0.43-branch) (#4939)

fix operator release for 0.43.0

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* renaming args and adding more

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Adding outputs to build_wheels workflow

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* reverting changes

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* chore: Add ability to change default feature-server image during Operator deploy (v0.43-branch) (#4941)

chore: Add ability to change default feature-server image during Operator deploy (#4940)

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* chore: CR image no longer needed for e2e (v0.43-branch) (#4943)

CR image no longer needed for e2e

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* chore: Fix publish, versions issue (#4944)

fix publish, versions issue

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* chore: Fix publish, versions issue (#4945)

fix publish, versions issue

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* chore: Fix publish, versions issue (#4946)

fix publish, versions issue

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* chore: Fix publish, versions issue (#4947)

fix publish, versions issue

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* fix publish, versions issue (#4948)

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* fix: adding checkout to publish sdk

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* chore: Change get version workflow to an action (#4951)

change get version workflow to an action

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* fix: Release Fix - Added the workflow_call element in missing workflows  (#4954)

Added the workflow_call element in all the workflows wherever there is a workflow_dispatch element to see if that fixes the problem to be able to call the workflow from another workflow.

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

* chore: Build wheels fix (#4957)

dev

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* chore: Fix python-sdk workflow (#4958)

dev

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* fix: adding type and workflow_call to build_wheels.yml

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* chore: Remove checkout from get-version action (#4959)

dev

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* chore: Fixing build wheels pre-release image build (#4960)

dev

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

* fix: Updating pypi upload to use legacy repo url (#4962)

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Updating repostiry to repostory url

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* converting to dash to see if that works

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Updating pypi publish action and adding name

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: moving back to kebab case

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Trying to hardcode gh-action version

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* trying new version with oidc false

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* trying without attestations

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* reverting again to old action and removing checkout action

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* reverting code to existing master without actions checkout

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* logging token length to validate it and adding quotes to password

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* adding logging to the build wheels to see if secret is present there

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* adding secrets to jobs keyword

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* adding input parameters to build wheels and publish python sdk

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* addding inherit

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* commenting out the publish python sdk and adding inherit

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* commenting out the needs

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removing commented out code and logging to incorporate code that worked

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* adding one more change to remove logging

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removing changes outside of actions

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removing makefile change

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removing yamls

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removing changes files and reverting move of file

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* reverting changed files

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

---------

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
Signed-off-by: Tommy Hughes <tohughes@redhat.com>
Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
Co-authored-by: Tommy Hughes IV <tohughes@redhat.com>
Co-authored-by: lokeshrangineni <19699092+lokeshrangineni@users.noreply.github.com>
franciscojavierarceo pushed a commit that referenced this pull request Feb 4, 2025
# [0.44.0](v0.43.0...v0.44.0) (2025-02-04)

### Bug Fixes

* Adding periodic check to fix the sporadic failures of the operator e2e tests.  ([#4952](#4952)) ([1d086be](1d086be))
* Adding the feast-operator/bin to the .gitignore directory. Somehow it… ([#5005](#5005)) ([1a027ee](1a027ee))
* Changed Env Vars for e2e tests ([#4975](#4975)) ([fa0084f](fa0084f))
* Fix GitHub Actions to pass authentication ([#4963](#4963)) ([22b9138](22b9138)), closes [#4937](#4937) [#4939](#4939) [#4941](#4941) [#4940](#4940) [#4943](#4943) [#4944](#4944) [#4945](#4945) [#4946](#4946) [#4947](#4947) [#4948](#4948) [#4951](#4951) [#4954](#4954) [#4957](#4957) [#4958](#4958) [#4959](#4959) [#4960](#4960) [#4962](#4962)
* Fix showing selected navigation item in UI sidebar ([#4969](#4969)) ([8ac6a85](8ac6a85))
* Invalid column names in get_historical_features when there are field mappings on join keys ([#4886](#4886)) ([c9aca2d](c9aca2d))
* Read project data from the 'projects' key while loading the registry state in the Feast UI ([#4772](#4772)) ([cb81939](cb81939))
* Remove grpcurl dependency from Operator ([#4972](#4972)) ([439e0b9](439e0b9))
* Removed the dry-run flag to test and we will add it back later. ([#5007](#5007)) ([d112b52](d112b52))
* Render UI navigation items as links instead of buttons ([#4970](#4970)) ([1267703](1267703))
* Resolve Operator CRD bloat due to long field descriptions ([#4985](#4985)) ([7593bb3](7593bb3))
* Update manifest to add feature server image for odh ([#4973](#4973)) ([6a1c102](6a1c102))
* Updating release workflows to refer to yml instead of yaml ([#4935](#4935)) ([02b0a68](02b0a68))
* Use locally built feast-ui package in dev feature-server image ([#4998](#4998)) ([0145e55](0145e55))

### Features

* Added OWNERS file for OpenshiftCI ([#4991](#4991)) ([86a2ee8](86a2ee8))
* Adding Milvus demo to examples ([#4910](#4910)) ([2daf852](2daf852))
* Adding retrieve_online_documents endpoint ([#5002](#5002)) ([6607d3d](6607d3d))
* Adding support to return additional features from vector retrieval for Milvus db ([#4971](#4971)) ([6ce08d3](6ce08d3))
* Creating/updating the stable branch after the release. ([#5003](#5003)) ([e9b53cc](e9b53cc))
* Implementing online_read for MilvusOnlineStore ([#4996](#4996)) ([92dde13](92dde13))
* Improve exception message for unsupported Snowflake data types ([#4779](#4779)) ([5992364](5992364))
* Operator add feast ui deployment ([#4930](#4930)) ([b026d0c](b026d0c))
* Updating documents to highlight v2 api for Vector Similarity Se… ([#5000](#5000)) ([32b82a4](32b82a4))
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.

2 participants