Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

initial GHA workflows for "build test" and "remote push" #27

Merged
merged 23 commits into from
Feb 21, 2024
Merged

Conversation

andy-neuma
Copy link
Member

@andy-neuma andy-neuma commented Feb 19, 2024

SUMMARY:

  • initial set of "actions with a little a" that are the building blocks for eventual CI system
  • "build test" workflow
  • "remote push" workflow on a10g
  • update some requirement files to have packages listed in alphabetical order

NOTE: this PR is still somewhat nebulas as i'm still working through building and testing "neuralmagic-vllm" in our automation environment.

TEST:
currently, i'm working through various workflow components, i.e. "actions with a little a". the bits making up the actions in this PR have been constructed from my notes along the way.

we can do a "complete" run that includes: linting, building, installing, and running tests.

GHA link ... https://github.com/neuralmagic/neuralmagic-vllm/actions/runs/7975058564
testmo ... https://neuralmagic.testmo.net/automation/runs/view/8097

Latest GHA link ... https://github.com/neuralmagic/neuralmagic-vllm/actions/runs/7992489982

@varun-sundar-rabindranath

Hey Andy - perhaps it makes sense to rename all files to have a neuralmagic prefix ? (I think it'd be cumbersome - but will about merge conflicts with upstream)

@varun-sundar-rabindranath

Looks good in general - had a couple of comments, but nothing major and could be iterated on !

@andy-neuma andy-neuma changed the title initial GHA related items initial GHA workflows for "build test" and "remote push" Feb 20, 2024
.github/actions/nm-build-vllm/action.yml Outdated Show resolved Hide resolved
.github/scripts/determine-threading Outdated Show resolved Hide resolved
@andy-neuma andy-neuma merged commit 64bdde5 into main Feb 21, 2024
2 checks passed
@andy-neuma andy-neuma deleted the init-gha branch February 21, 2024 17:50
robertgshaw2-redhat pushed a commit that referenced this pull request Feb 22, 2024
SUMMARY:
* initial set of "actions with a little a" that are the building blocks
for eventual CI system
* "build test" workflow
* "remote push" workflow on `a10g`
* update some requirement files to have packages listed in alphabetical
order

NOTE: this PR is still somewhat nebulas as i'm still working through
building and testing "neuralmagic-vllm" in our automation environment.

TEST:
currently, i'm working through various workflow components, i.e.
"actions with a little a". the bits making up the actions in this PR
have been constructed from my notes along the way.

we can do a "complete" run that includes: linting, building, installing,
and running tests.

GHA link ...
https://github.com/neuralmagic/neuralmagic-vllm/actions/runs/7975058564
`testmo` ... https://neuralmagic.testmo.net/automation/runs/view/8097

Latest GHA link ...
https://github.com/neuralmagic/neuralmagic-vllm/actions/runs/7992489982

---------

Co-authored-by: andy-neuma <andy@neuralmagic.com>
robertgshaw2-redhat pushed a commit that referenced this pull request Feb 22, 2024
SUMMARY:
* initial set of "actions with a little a" that are the building blocks
for eventual CI system
* "build test" workflow
* "remote push" workflow on `a10g`
* update some requirement files to have packages listed in alphabetical
order

NOTE: this PR is still somewhat nebulas as i'm still working through
building and testing "neuralmagic-vllm" in our automation environment.

TEST:
currently, i'm working through various workflow components, i.e.
"actions with a little a". the bits making up the actions in this PR
have been constructed from my notes along the way.

we can do a "complete" run that includes: linting, building, installing,
and running tests.

GHA link ...
https://github.com/neuralmagic/neuralmagic-vllm/actions/runs/7975058564
`testmo` ... https://neuralmagic.testmo.net/automation/runs/view/8097

Latest GHA link ...
https://github.com/neuralmagic/neuralmagic-vllm/actions/runs/7992489982

---------

Co-authored-by: andy-neuma <andy@neuralmagic.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants