Skip to content

Add in container structure tests for the nonroot image - #7796

Merged
6 commits merged into
BerriAI:litellm_dev_01_15_2025_p1from
rajatvig:AddTests
Jan 16, 2025
Merged

Add in container structure tests for the nonroot image#7796
6 commits merged into
BerriAI:litellm_dev_01_15_2025_p1from
rajatvig:AddTests

Conversation

@rajatvig

Copy link
Copy Markdown
Contributor

No description provided.

Hexoplon and others added 6 commits January 15, 2025 12:51
…ob (BerriAI#7652)

* fix(helm): corrected indentation in migration-job.yaml

* feat(helm): add securityContext and pull policy values to migration job
* update lunary doc

* better title

* tweaks

* Update langchain.md

* Update lunary_integration.md
@vercel

vercel Bot commented Jan 16, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2025 0:12am

@ghost

ghost commented Jan 16, 2025

Copy link
Copy Markdown

Hey @rajatvig can you share a screenshot of these tests running and passing for you

@rajatvig

Copy link
Copy Markdown
Contributor Author

Hey @rajatvig can you share a screenshot of these tests running and passing for you

While I cannot verify the CircleCI setup (no access), running the tests work once #7795 would merge in main.

➜ container-structure-test test --image non_root_image:latest --config docker/tests/nonroot.yaml

=====================================
====== Test file: nonroot.yaml ======
=====================================
=== RUN: File Existence Test: Prisma Folder
--- PASS
duration: 0s
=== RUN: File Existence Test: Prisma Schema
--- PASS
duration: 0s
=== RUN: Metadata Test
--- PASS
duration: 0s

=====================================
============== RESULTS ==============
=====================================
Passes:      3
Failures:    0
Duration:    0s
Total tests: 3

PASS

@ghost
ghost changed the base branch from main to litellm_dev_01_15_2025_p1 January 16, 2025 00:58
@ghost
ghost merged commit 84f9849 into BerriAI:litellm_dev_01_15_2025_p1 Jan 16, 2025
ghost pushed a commit that referenced this pull request Jan 16, 2025
* Add back in non root image fixes (#7781)

* Add back in non root image fixes

* Fix dockerfile

* Fix perms

* Add in container structure tests for the nonroot image (#7796)

* feat(helm): add securityContext and pull policy values to migration job (#7652)

* fix(helm): corrected indentation in migration-job.yaml

* feat(helm): add securityContext and pull policy values to migration job

* fix confusing save button label (#7778)

* [integrations/lunary] Improve Lunary documentaiton (#7770)

* update lunary doc

* better title

* tweaks

* Update langchain.md

* Update lunary_integration.md

* Fix wrong URL for internal user invitation (#7762)

* format

* done

* Update instructor tutorial (#7784)

* Add in container structure tests for the nonroot image

---------

Co-authored-by: Zackeus Bengtsson <32719220+Hexoplon@users.noreply.github.com>
Co-authored-by: yujonglee <yujonglee.dev@gmail.com>
Co-authored-by: Hugues Chocart <chocart.hugues@icloud.com>
Co-authored-by: Nikolaiev Dmytro <dima.nikol.99@gmail.com>

---------

Co-authored-by: Rajat Vig <rajatvig@users.noreply.github.com>
Co-authored-by: Zackeus Bengtsson <32719220+Hexoplon@users.noreply.github.com>
Co-authored-by: yujonglee <yujonglee.dev@gmail.com>
Co-authored-by: Hugues Chocart <chocart.hugues@icloud.com>
Co-authored-by: Nikolaiev Dmytro <dima.nikol.99@gmail.com>
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026
* Add back in non root image fixes (BerriAI#7781)

* Add back in non root image fixes

* Fix dockerfile

* Fix perms

* Add in container structure tests for the nonroot image (BerriAI#7796)

* feat(helm): add securityContext and pull policy values to migration job (BerriAI#7652)

* fix(helm): corrected indentation in migration-job.yaml

* feat(helm): add securityContext and pull policy values to migration job

* fix confusing save button label (BerriAI#7778)

* [integrations/lunary] Improve Lunary documentaiton (BerriAI#7770)

* update lunary doc

* better title

* tweaks

* Update langchain.md

* Update lunary_integration.md

* Fix wrong URL for internal user invitation (BerriAI#7762)

* format

* done

* Update instructor tutorial (BerriAI#7784)

* Add in container structure tests for the nonroot image

---------

Co-authored-by: Zackeus Bengtsson <32719220+Hexoplon@users.noreply.github.com>
Co-authored-by: yujonglee <yujonglee.dev@gmail.com>
Co-authored-by: Hugues Chocart <chocart.hugues@icloud.com>
Co-authored-by: Nikolaiev Dmytro <dima.nikol.99@gmail.com>

---------

Co-authored-by: Rajat Vig <rajatvig@users.noreply.github.com>
Co-authored-by: Zackeus Bengtsson <32719220+Hexoplon@users.noreply.github.com>
Co-authored-by: yujonglee <yujonglee.dev@gmail.com>
Co-authored-by: Hugues Chocart <chocart.hugues@icloud.com>
Co-authored-by: Nikolaiev Dmytro <dima.nikol.99@gmail.com>
This pull request was closed.
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.

4 participants