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

fix(ai): llm model moved to share state to guarantee consistency #13627

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

tysoekong
Copy link
Contributor

@tysoekong tysoekong commented Sep 5, 2024

Summary

Sets request model name consistently across AI plugins.

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[issue number]

@tysoekong tysoekong marked this pull request as ready for review September 6, 2024 08:07
@tysoekong tysoekong closed this Sep 6, 2024
@tysoekong tysoekong reopened this Sep 6, 2024
@tysoekong tysoekong force-pushed the fix/ai_model_name_consistency branch from 66a26f2 to c541fe8 Compare September 6, 2024 11:46
@pull-request-size pull-request-size bot added size/M and removed size/S labels Sep 6, 2024
@tysoekong tysoekong force-pushed the fix/ai_model_name_consistency branch from 895f678 to 54cd557 Compare September 6, 2024 13:46
@tysoekong
Copy link
Contributor Author

@fffonion Added integration test now for this, please verify once more

Copy link
Contributor

@fffonion fffonion left a comment

Choose a reason for hiding this comment

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

wait for CI

@locao locao added this to the 3.8.0 milestone Sep 6, 2024
@tysoekong tysoekong force-pushed the fix/ai_model_name_consistency branch 2 times, most recently from e3828ee to d3559e5 Compare September 6, 2024 16:24
@tysoekong
Copy link
Contributor Author

@fffonion Should pass now

@tysoekong tysoekong force-pushed the fix/ai_model_name_consistency branch from d3559e5 to 6d62fae Compare September 6, 2024 16:25
@fffonion
Copy link
Contributor

fffonion commented Sep 6, 2024

two lint errors, rerunning CI

@fffonion fffonion merged commit ad3e19a into master Sep 6, 2024
26 checks passed
@team-gateway-bot
Copy link
Collaborator

Cherry-pick failed for master, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git remote add upstream https://github.com/kong/kong-ee
git fetch upstream master
git worktree add -d .worktree/cherry-pick-13627-to-master-to-upstream upstream/master
cd .worktree/cherry-pick-13627-to-master-to-upstream
git checkout -b cherry-pick-13627-to-master-to-upstream
ancref=$(git merge-base b861d1fc030c43923151241d7e4f6710ab2a3a5a 2aa3a4f39804e56cffb7c6ed414697a025135df0)
git cherry-pick -x $ancref..2aa3a4f39804e56cffb7c6ed414697a025135df0

@team-gateway-bot
Copy link
Collaborator

Successfully created backport PR for release/3.8.x:

@github-actions github-actions bot added the incomplete-cherry-pick A cherry-pick was incomplete and needs manual intervention label Sep 6, 2024
@locao locao removed the incomplete-cherry-pick A cherry-pick was incomplete and needs manual intervention label Sep 6, 2024
guanlan pushed a commit that referenced this pull request Sep 9, 2024
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.

5 participants