Skip to content

[v10] Ensure operator tests are run when Go dependencies change #17032

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

Merged
zmb3 merged 4 commits into from
Oct 7, 2022
Merged

[v10] Ensure operator tests are run when Go dependencies change #17032

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9651eb8
Ensure operator tests are run when Go dependencies change
zmb3 Oct 4, 2022
cbba85b
Trigger operator tests on `api/types` and `lib/tbot` changes
hugoShaka Oct 4, 2022
3cb73aa
Fix formatting
zmb3 Oct 4, 2022
c9c4e52
Merge branch 'branch/v10' into bot/backport-17000-branch/v10
zmb3 Oct 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion .cloudbuild/scripts/internal/changes/changes.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,14 @@ func isCIChange(path string) bool {

func isOperatorChange(path string) bool {
path = strings.ToLower(path)
return strings.HasPrefix(path, "operator/")
// dependency updates can impact CRD generation,
// so ensure that operator tests are run when
// dependencies change
return path == "go.mod" ||
path == "go.sum" ||
strings.HasPrefix(path, "operator/") ||
strings.HasPrefix(path, "api/types") || // the operator uses directly Teleport types
strings.HasPrefix(path, "lib/tbot") // the operator embeds a tbot instance
}

func isDocChange(path string) bool {
Expand Down