Skip to content

Conversation

@mhucka
Copy link
Member

@mhucka mhucka commented Jan 20, 2025

This workflow does a plain TFQ build & test. It's scheduled to run every night. Its purpose is to guard against failures that might be missed when skipping the long-running tests or using caching to speed up the CI runs. Unlike the CI checks invoked on PRs and similar events, this workflow builds everything without caching and runs the full test suite, including "eternal" tests that take a long time to run.

This workflow does a plain TFQ build & test. It's scheduled to run
every night. Its purpose is to guard against failures that might be
missed when skipping the long-running tests or using caching to speed
up the CI runs. Unlike the CI checks invoked on PRs and similar
events, this workflow builds everything without caching and runs the
full test suite, including "eternal" tests that take a long time to
run.
@mhucka mhucka marked this pull request as ready for review January 20, 2025 17:04
@mhucka mhucka added area/tests Involves tests and testing of the TFQ codebase area/devops Involves build systems, Make files, Bazel files, continuous integration, and/or other DevOps topics labels Jan 20, 2025
@mhucka mhucka self-assigned this Jan 20, 2025
@mhucka mhucka enabled auto-merge January 20, 2025 17:05
Avoid wrapping the line in the GUI
This is just an effort to have some regularity in the file names.
If there have been no changes in the past 24 hours, then it's
pointless to run this full build and test. We can save resources by
skipping it.
@mhucka mhucka force-pushed the mhucka-add-nightly-full-test branch from 56f5ce7 to d8420a0 Compare January 21, 2025 02:02
Copy link
Collaborator

@MichaelBroughton MichaelBroughton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mhucka mhucka added this pull request to the merge queue Jan 21, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jan 21, 2025
@mhucka mhucka added this pull request to the merge queue Jan 21, 2025
@mhucka mhucka removed this pull request from the merge queue due to a manual request Jan 21, 2025
@mhucka mhucka merged commit 4467aba into tensorflow:master Jan 21, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/devops Involves build systems, Make files, Bazel files, continuous integration, and/or other DevOps topics area/tests Involves tests and testing of the TFQ codebase

Development

Successfully merging this pull request may close these issues.

2 participants