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

Give TensorRT-LLMa proper CI/CD 😍 #2886

Merged

Narsil merged 134 commits into main from trtllm/ci

Jan 21, 2025

Member

mfuntowicz commented Jan 7, 2025

No description provided.

mfuntowicz requested review from Narsil and Hugoch

January 7, 2025 10:54

mfuntowicz force-pushed the trtllm/ci branch from 7950d43 to ccafee0 Compare

January 7, 2025 10:57

paulinebm closed this

mfuntowicz reopened this

Contributor

paulinebm commented Jan 8, 2025

test with Morgan

paulinebm closed this

paulinebm reopened this

paulinebm closed this

paulinebm reopened this

paulinebm closed this

paulinebm reopened this

paulinebm closed this

paulinebm reopened this

paulinebm closed this

paulinebm reopened this

paulinebm closed this

paulinebm reopened this

paulinebm closed this

paulinebm reopened this

paulinebm closed this

paulinebm reopened this

paulinebm closed this

paulinebm reopened this

Collaborator

Narsil commented Jan 15, 2025

Can we please keep the same CI for every backend. It's much better to keep it that way so we don't maintain 3 separate CIs.

Member Author

mfuntowicz commented Jan 15, 2025 •

edited

Loading

@Narsil it's the case here no? We still need some test on the C++ side we wont escape.

The goal is really to have the same integration tests in the end + backend specific tests

Narsil reviewed

View reviewed changes

.github/workflows/ci_build.yaml Outdated Show resolved Hide resolved

Narsil reviewed

View reviewed changes

.github/workflows/build_trtllm.yaml Outdated Show resolved Hide resolved

mfuntowicz force-pushed the trtllm/ci branch from 4cca211 to f657af4 Compare

January 20, 2025 10:57


          test(ctest) enable address sanitizer

f729f2c

mfuntowicz and others added 10 commits

January 20, 2025 12:03


          misc(ci): ok debug

a791291


          misc(ci): wtf

af6428c


          misc(ci): wtf2

c3f3035


          misc(ci): wtf3

8f0da40


          misc(ci): use commit HEAD instead of merge commit for image id

3c7710c


          misc(ci): wtfinfini

79469be


          misc(ci): wtfinfini

0cf0732


          misc(ci): KAMEHAMEHA


          Merge TRTLLM in standard CI

d969dad


          misc(ci): remove input machine

ffb60ff

mfuntowicz force-pushed the trtllm/ci branch from f657af4 to ffb60ff Compare

January 20, 2025 11:04

mfuntowicz added 15 commits

January 20, 2025 12:07


          misc(ci): missing id-token for AWS auth

b267df5


          misc(ci): missing id-token for AWS auth

e083a92


          misc(ci): missing id-token for AWS auth

87039cf


          misc(ci): again...

549d7e3


          misc(ci): again...

c33eeb2


          misc(ci): again...

0a33615


          misc(ci): again...

08af269


          misc(ci): missing benchmark

0ed76ba


          misc(ci): missing backends

8c6d972


          misc(ci): missing launcher

debaea4


          misc(ci): give everything aws needs

7c9ee56


          misc(ci): give everything aws needs

d0b8e2e


          misc(ci): fix warnings

edfafeb


          misc(ci): attempt to fix sccache not building trtllm

a4d069f


          misc(ci): attempt to fix sccache not building trtllm again

a0e75b1

mfuntowicz requested a review from Narsil

January 21, 2025 08:49

Narsil approved these changes

View reviewed changes

Narsil merged commit 1736743 into main

16 of 17 checks passed

Narsil deleted the trtllm/ci branch

January 21, 2025 09:19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet