Skip to content

[wip] Fix CI for publishing plugin images#515

Open
cosmicBboy wants to merge 13 commits intomainfrom
nielsb/publish-plugin-images
Open

[wip] Fix CI for publishing plugin images#515
cosmicBboy wants to merge 13 commits intomainfrom
nielsb/publish-plugin-images

Conversation

@cosmicBboy
Copy link
Contributor

No description provided.

Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
@cosmicBboy cosmicBboy requested a review from kumare3 January 7, 2026 20:05
kumare3
kumare3 previously approved these changes Jan 7, 2026
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>
@cosmicBboy cosmicBboy changed the title Fix CI for publishing plugin images [DO NOT MERGE] Fix CI for publishing plugin images Jan 7, 2026
@cosmicBboy cosmicBboy changed the title [DO NOT MERGE] Fix CI for publishing plugin images [wip] Fix CI for publishing plugin images Jan 7, 2026
Signed-off-by: Niels Bantilan <niels.bantilan@gmail.com>

DEFAULT_SGLANG_IMAGE = (
flyte.Image.from_debian_base(name="sglang-app-image")
flyte.Image.from_debian_base(name="sglang-app-image", python_version=(3, 13))
Copy link
Contributor

Choose a reason for hiding this comment

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

does this mean we'll only build one version of the sglang-app-image and it'll only be 3.13? that's fine but will users accidentally somehow use another one? what happens if they create an sglang app while on 3.11?

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.

3 participants