Skip to content

Add Verda validation, remove cron for other validations, move to specific validation run - #145

Merged
drewmalin merged 3 commits into
mainfrom
dm/validationtests
Aug 14, 2026
Merged

Add Verda validation, remove cron for other validations, move to specific validation run#145
drewmalin merged 3 commits into
mainfrom
dm/validationtests

Conversation

@drewmalin

Copy link
Copy Markdown
Contributor

These tests are vital, but also are at the whims of git pushes and overlapping cron/PRs, and thus result in lots of orphaned data.

Going forward, we should be very explicit about running these actions, as they ultimately consume both capacity and funds.

@drewmalin
drewmalin requested a review from a team as a code owner August 14, 2026 19:15
@patelspratik

Copy link
Copy Markdown
Contributor

does it make sense to only run these after merge like we do for the e2e on dev-plane? And enforce that new ones queue behind old ones instead of cancelling?

@drewmalin

Copy link
Copy Markdown
Contributor Author

does it make sense to only run these after merge like we do for the e2e on dev-plane? And enforce that new ones queue behind old ones instead of cancelling?

Yeah that's a good idea and points us to needing to formalize exactly which git shas are appropriate for pulling into devplane. One method could be that it's just the honor system. Another is that these runs occur on a non-main branch, and the merge to main is only done as a mark of validation/release candidacy.

@drewmalin
drewmalin merged commit 56bdea0 into main Aug 14, 2026
4 checks passed
@drewmalin
drewmalin deleted the dm/validationtests branch August 14, 2026 19:25
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.

2 participants