Skip to content

Conversation

JenniferWang
Copy link
Contributor

@JenniferWang JenniferWang commented Oct 15, 2025

Open to discussion. GPU based integration tests may be more expensive to run:

  1. Job may wait for a while in the queue. I've seen no wait to 30min-wait...
  2. E2E job may run for a while

I think it might be better to run them continuously on the trunk (hourly or every 2 hours) or manually on a specific PR instead of on every PR.

Of course, we could add conditions on when to run a specific tests based on paths but I feel it's probably overkill for now.

@ebsmothers

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 15, 2025
@JenniferWang JenniferWang changed the title [WIP] Add test_vllm_policy_correctness to CI Create continuous run only workflow Oct 15, 2025
@JenniferWang JenniferWang marked this pull request as ready for review October 16, 2025 13:13
@JenniferWang JenniferWang changed the title Create continuous run only workflow [CI] Create continuous run only workflow Oct 16, 2025
@JenniferWang JenniferWang marked this pull request as draft October 16, 2025 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant