Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Platform] Add FailoverPlatform #1126

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
Merged
OskarStark merged 1 commit into from
Jan 7, 2026
Merged

[Platform] Add FailoverPlatform #1126

OskarStark merged 1 commit into from
Jan 7, 2026
+987 −0

Conversation

@Guikingone
Copy link
Contributor

@Guikingone Guikingone commented Dec 12, 2025 •
edited
Loading

Q A
Bug fix? no
New feature? yes
Docs? yes
Issues Discussed with @TimoBakx on Slack
License MIT

This platform was a POC that I launched a few weeks ago locally (mostly for Ollama and OpenAI as a backup in a context of another POC -more on the other one if I find time to finish the tests-), not totally convinced by the current implementation but debates can lead to improvements so here it is 😅

PS: No configuration for now as the options.php must be improved on my side to handle the platforms.

@Guikingone Guikingone changed the title [Platform] Add Failover platform [Platform] Add FailoverPlatform Dec 12, 2025
@Guikingone Guikingone force-pushed the platform/fallback branch 5 times, most recently from 03b2077 to 2216da4 Compare December 15, 2025 18:48
@Guikingone Guikingone marked this pull request as ready for review December 15, 2025 18:50
@carsonbot carsonbot added Feature New feature Platform Issues & PRs about the AI Platform component Status: Needs Review labels Dec 15, 2025
OskarStark
OskarStark reviewed Dec 16, 2025
View reviewed changes
@Guikingone
Copy link
Contributor Author

Guikingone commented Dec 17, 2025

Well, that's a weird one, no changes on my side and the class is not found by the bundle, what the funk is this behavior? 🤔

@OskarStark
Copy link
Contributor

OskarStark commented Dec 17, 2025

Please rebase

OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
@Guikingone Guikingone force-pushed the platform/fallback branch 2 times, most recently from 0c424e9 to dd990a7 Compare December 18, 2025 15:52
OskarStark
OskarStark reviewed Dec 18, 2025
View reviewed changes
OskarStark
OskarStark reviewed Dec 18, 2025
View reviewed changes
@OskarStark
Copy link
Contributor

OskarStark commented Dec 18, 2025

Please rebase

OskarStark
OskarStark reviewed Dec 19, 2025
View reviewed changes
OskarStark
OskarStark reviewed Dec 19, 2025
View reviewed changes
OskarStark
OskarStark reviewed Dec 19, 2025
View reviewed changes
OskarStark
OskarStark reviewed Dec 19, 2025
View reviewed changes
chr-hertel
chr-hertel reviewed Jan 2, 2026
View reviewed changes
chr-hertel
chr-hertel requested changes Jan 2, 2026
View reviewed changes
Copy link
Member

@chr-hertel chr-hertel left a comment

Choose a reason for hiding this comment

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

needs a rebase please

@OskarStark
Copy link
Contributor

OskarStark commented Jan 2, 2026

Path repository in composer.json should not be needed after the rebase

@Guikingone Guikingone requested a review from chr-hertel January 2, 2026 20:02
OskarStark
OskarStark reviewed Jan 5, 2026
View reviewed changes
@Guikingone Guikingone requested a review from OskarStark January 5, 2026 10:03
OskarStark
OskarStark reviewed Jan 6, 2026
View reviewed changes
OskarStark
OskarStark reviewed Jan 6, 2026
View reviewed changes
OskarStark
OskarStark approved these changes Jan 6, 2026
View reviewed changes
Copy link
Contributor

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

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

Good to merge after my comments

@Guikingone Guikingone requested a review from OskarStark January 6, 2026 19:59
@Guikingone Guikingone force-pushed the platform/fallback branch 3 times, most recently from 76ca120 to 09558a9 Compare January 7, 2026 08:02
@OskarStark
Copy link
Contributor

OskarStark commented Jan 7, 2026

Thank you @Guikingone.

@OskarStark OskarStark merged commit 2ef7f15 into symfony:main Jan 7, 2026
342 checks passed
@Guikingone Guikingone deleted the platform/fallback branch January 8, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chr-hertel chr-hertel Awaiting requested review from chr-hertel chr-hertel is a code owner

@OskarStark OskarStark Awaiting requested review from OskarStark OskarStark is a code owner

Assignees

No one assigned

Labels

Feature New feature Platform Issues & PRs about the AI Platform component Status: Needs Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Guikingone @OskarStark @chr-hertel @carsonbot