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: Limit integration runs back to a single python version due to rate limiting errors #4169

Merged

Conversation

jeremyary
Copy link
Collaborator

Even with recent workflow improvements, the recent change to increase scope of integration tests from only python v3.9 to v's 3.9-3.11 has caused us to begin hitting rate limiting errors with google cloud resources due to 3 simultaneous executions per action trigger. This PR reduces integration tests back to a single version scope using v3.11 so that actions won't require constant monitoring and re-triggers.

…rate limiting errors

Signed-off-by: Jeremy Ary <jary@redhat.com>
Copy link
Collaborator

@tokoko tokoko left a comment

Choose a reason for hiding this comment

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

lgtm

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 b0c5ffc into feast-dev:master May 2, 2024
25 checks passed
tmihalac pushed a commit to tmihalac/feast that referenced this pull request May 3, 2024
…rate limiting errors (feast-dev#4169)

Signed-off-by: Jeremy Ary <jary@redhat.com>
tmihalac pushed a commit to tmihalac/feast that referenced this pull request May 3, 2024
…rate limiting errors (feast-dev#4169)

Signed-off-by: Jeremy Ary <jary@redhat.com>
Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
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.

3 participants