Skip to content

[Infra] Remove unused publish_proxy_extras and prisma_schema_sync jobs from CircleCI config#25821

Merged
yuneng-berri merged 1 commit intolitellm_internal_stagingfrom
litellm_/funny-meninsky
Apr 15, 2026
Merged

[Infra] Remove unused publish_proxy_extras and prisma_schema_sync jobs from CircleCI config#25821
yuneng-berri merged 1 commit intolitellm_internal_stagingfrom
litellm_/funny-meninsky

Conversation

@yuneng-berri
Copy link
Copy Markdown
Collaborator

Relevant issues

Summary

Cleaning up two CircleCI jobs that are no longer needed.

Changes

  • Remove publish_proxy_extras: We now publish litellm-proxy-extras to PyPI via PyPI trusted publishing (OIDC-based, no long-lived token), so the CircleCI-driven publish path using TWINE_USERNAME / PYPI_PUBLISH_USERNAME / PYPI_PUBLISH_PASSWORD is no longer used. Manually confirmed these env vars no longer exist in the CircleCI project settings, so the job cannot succeed on a delta anyway.
  • Remove prisma_schema_sync: This was a smoke test that booted the docker image against a fresh Postgres with --use_prisma_db_push. It is a leaf job with no downstream dependents, and e2e_ui_testing already exercises prisma db push against the current schema.prisma end-to-end. Dropping the duplicate saves a CI slot.

No other jobs reference either of these — confirmed by grep and by checking requires: lists.

Testing

  • Ran YAML parse validation on the trimmed config.
  • Verified via grep that publish_proxy_extras and prisma_schema_sync are not referenced elsewhere (no requires: dependents, no workflow entries other than their own).
  • Manually confirmed in CircleCI project settings that PYPI_PUBLISH_USERNAME / PYPI_PUBLISH_PASSWORD are no longer configured.

Type

🚄 Infrastructure

publish_proxy_extras is superseded by PyPI trusted publishing (OIDC);
the CircleCI project no longer has PYPI_PUBLISH_* credentials configured.
prisma_schema_sync is a leaf smoke test with no dependents, and db push
against the current schema is already exercised by e2e_ui_testing.
@yuneng-berri yuneng-berri requested a review from a team April 15, 2026 23:43
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview, Comment Apr 15, 2026 11:45pm

Request Review

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 15, 2026

Greptile Summary

This PR removes two CircleCI jobs that are no longer needed: publish_proxy_extras (replaced by PyPI OIDC-based trusted publishing) and prisma_schema_sync (a leaf smoke-test superseded by e2e_ui_testing). No other jobs reference either removed job, and neither appears anywhere else in the repository.

Confidence Score: 5/5

Safe to merge — pure deletion of two unused CI jobs with no downstream dependents.

Both removed jobs are confirmed to have no requires: dependents and no references anywhere else in the repository. The YAML is structurally sound after the deletion, and build_docker_database_image retains other dependents (build_and_test, test_bad_database_url). No functional code or test coverage is affected.

No files require special attention.

Important Files Changed

Filename Overview
.circleci/config.yml Removes publish_proxy_extras and prisma_schema_sync job definitions and their corresponding workflow entries; no remaining references to either job anywhere in the repo.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    BDI[build_docker_database_image] --> BAT[build_and_test]
    BDI --> TBDU[test_bad_database_url]
    BDI -.->|removed| PSS[~~prisma_schema_sync~~]
    E2E[e2e_ui_testing] -->|covers prisma db push| DB[(PostgreSQL)]
    PPE[~~publish_proxy_extras~~] -.->|removed - replaced by| OIDC[PyPI OIDC Trusted Publishing]

    style PSS fill:#ffcccc,stroke:#cc0000,color:#333
    style PPE fill:#ffcccc,stroke:#cc0000,color:#333
    style OIDC fill:#ccffcc,stroke:#009900,color:#333
Loading

Reviews (1): Last reviewed commit: "[Infra] Remove unused publish_proxy_extr..." | Re-trigger Greptile

@yuneng-berri yuneng-berri enabled auto-merge April 15, 2026 23:48
@shin-berri shin-berri self-requested a review April 15, 2026 23:48
@yuneng-berri yuneng-berri merged commit 2f72eb6 into litellm_internal_staging Apr 15, 2026
86 of 91 checks passed
@yuneng-berri yuneng-berri deleted the litellm_/funny-meninsky branch April 15, 2026 23:52
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