From 0ef7bbcddcb5517f7e9448c3d50ed4086c845978 Mon Sep 17 00:00:00 2001 From: Nicolas De Loof Date: Tue, 25 Feb 2025 10:30:52 +0100 Subject: [PATCH] introduce watch.include Signed-off-by: Nicolas De Loof --- go.mod | 2 +- go.sum | 4 ++-- pkg/compose/watch.go | 21 +++++++++++++++++++++ pkg/watch/notify.go | 9 +++++++++ 4 files changed, 33 insertions(+), 3 deletions(-) diff --git a/go.mod b/go.mod index f28e626a1..28b14cb5c 100644 --- a/go.mod +++ b/go.mod @@ -7,7 +7,7 @@ require ( github.com/Microsoft/go-winio v0.6.2 github.com/acarl005/stripansi v0.0.0-20180116102854-5a71ef0e047d github.com/buger/goterm v1.0.4 - github.com/compose-spec/compose-go/v2 v2.4.9-0.20250225151507-331db8fefcb7 + github.com/compose-spec/compose-go/v2 v2.4.9-0.20250302154753-e508c724a35f github.com/containerd/containerd/v2 v2.0.2 github.com/containerd/platforms v1.0.0-rc.1 github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc diff --git a/go.sum b/go.sum index 8acb7edde..bde32edf7 100644 --- a/go.sum +++ b/go.sum @@ -81,8 +81,8 @@ github.com/cloudflare/cfssl v0.0.0-20180223231731-4e2dcbde5004 h1:lkAMpLVBDaj17e github.com/cloudflare/cfssl v0.0.0-20180223231731-4e2dcbde5004/go.mod h1:yMWuSON2oQp+43nFtAV/uvKQIFpSPerB57DCt9t8sSA= github.com/codahale/rfc6979 v0.0.0-20141003034818-6a90f24967eb h1:EDmT6Q9Zs+SbUoc7Ik9EfrFqcylYqgPZ9ANSbTAntnE= github.com/codahale/rfc6979 v0.0.0-20141003034818-6a90f24967eb/go.mod h1:ZjrT6AXHbDs86ZSdt/osfBi5qfexBrKUdONk989Wnk4= -github.com/compose-spec/compose-go/v2 v2.4.9-0.20250225151507-331db8fefcb7 h1:7NlxAsQcWvLpFlEHsBo80sJ1UMMs84kkf0yXGs6de2k= -github.com/compose-spec/compose-go/v2 v2.4.9-0.20250225151507-331db8fefcb7/go.mod h1:6k5l/0TxCg0/2uLEhRVEsoBWBprS2uvZi32J7xub3lo= +github.com/compose-spec/compose-go/v2 v2.4.9-0.20250302154753-e508c724a35f h1:kbmTPhf7d9kTnmH0ghAQTqxs1zenKnwKczJlC4z5WSc= +github.com/compose-spec/compose-go/v2 v2.4.9-0.20250302154753-e508c724a35f/go.mod h1:6k5l/0TxCg0/2uLEhRVEsoBWBprS2uvZi32J7xub3lo= github.com/containerd/cgroups/v3 v3.0.5 h1:44na7Ud+VwyE7LIoJ8JTNQOa549a8543BmzaJHo6Bzo= github.com/containerd/cgroups/v3 v3.0.5/go.mod h1:SA5DLYnXO8pTGYiAHXz94qvLQTKfVM5GEVisn4jpins= github.com/containerd/console v1.0.4 h1:F2g4+oChYvBTsASRTz8NP6iIAi97J3TtSAsLbIFn4ro= diff --git a/pkg/compose/watch.go b/pkg/compose/watch.go index cff1d4f0b..72f329a62 100644 --- a/pkg/compose/watch.go +++ b/pkg/compose/watch.go @@ -81,6 +81,7 @@ func (s *composeService) Watch(ctx context.Context, project *types.Project, serv type watchRule struct { types.Trigger + include watch.PathMatcher ignore watch.PathMatcher service string } @@ -90,6 +91,15 @@ func (r watchRule) Matches(event watch.FileEvent) *sync.PathMapping { if !pathutil.IsChild(r.Path, hostPath) { return nil } + included, err := r.include.Matches(hostPath) + if err != nil { + logrus.Warnf("error include matching %q: %v", hostPath, err) + return nil + } + if !included { + logrus.Debugf("%s is not matching include pattern", hostPath) + return nil + } isIgnored, err := r.ignore.Matches(hostPath) if err != nil { logrus.Warnf("error ignore matching %q: %v", hostPath, err) @@ -244,8 +254,19 @@ func getWatchRules(config *types.DevelopConfig, service types.ServiceConfig) ([] return nil, err } + var include watch.PathMatcher + if len(trigger.Include) == 0 { + include = watch.AnyMatcher{} + } else { + include, err = watch.NewDockerPatternMatcher(trigger.Path, trigger.Include) + if err != nil { + return nil, err + } + } + rules = append(rules, watchRule{ Trigger: trigger, + include: include, ignore: watch.NewCompositeMatcher( dockerIgnores, watch.EphemeralPathMatcher(), diff --git a/pkg/watch/notify.go b/pkg/watch/notify.go index b76292106..d09a6be15 100644 --- a/pkg/watch/notify.go +++ b/pkg/watch/notify.go @@ -68,6 +68,15 @@ type PathMatcher interface { MatchesEntireDir(file string) (bool, error) } +// AnyMatcher is a PathMatcher to match any path +type AnyMatcher struct{} + +func (AnyMatcher) Matches(f string) (bool, error) { return true, nil } +func (AnyMatcher) MatchesEntireDir(f string) (bool, error) { return true, nil } + +var _ PathMatcher = AnyMatcher{} + +// EmptyMatcher is a PathMatcher to match no path type EmptyMatcher struct{} func (EmptyMatcher) Matches(f string) (bool, error) { return false, nil }