Skip to content

readme: fix logo#1666

Merged
mattklein123 merged 1 commit intomasterfrom
fix_logo
Sep 15, 2017
Merged

readme: fix logo#1666
mattklein123 merged 1 commit intomasterfrom
fix_logo

Conversation

@mattklein123
Copy link
Copy Markdown
Member

Fixes #1662

@mattklein123 mattklein123 merged commit 19c4771 into master Sep 15, 2017
@mattklein123 mattklein123 deleted the fix_logo branch September 15, 2017 19:34
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Signed-off-by: Rafal Augustyniak <raugustyniak@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Signed-off-by: Rafal Augustyniak <raugustyniak@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
**Description**

This adds a new field 'prefix' as discussed in #1639. tldr is that the
existing "version" field has been overused in various and confusing
ways. Notably for OpenAI, it was literally used for expressing the
prefix of the endpoints and not really a version in any sense.

**Related Issues/PRs (if applicable)**

Closes #1639 
Closes #1629

---------

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
**Description**

This commit adds an additional safeguard for older deployment regarding
the OpenAI prefix configuration added in #1666. More specifically, this
prevents an issue that could happen in a small window when the old
extproc is running while the extproc config gets updated by the new
controller.

---------

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
**Description**
This adds an additional backward compatible pass for OpenAI's prefix
field handling

**Related Issues/PRs (if applicable)**
Follow up on #1666 and #1674

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
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