Skip to content
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.

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

x/build/maintner: make it easy for interested clients to subscribe/watch for new Go releases #37577

Open
dmitshur opened this issue Feb 29, 2020 · 1 comment
Labels
Builders x/build issues (builders, bots, dashboards) Documentation FeatureRequest NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone

Comments

@dmitshur
Copy link
Contributor

There are many clients that might be interested in knowing when a new Go release is tagged, in order to take some automated action. One recent example is pkg.go.dev (see issue #37568), but there are plenty of others.

Maintner already tracks the Go repository and knows when any tags are made. Interested clients can use a maintner client (either a normal one, or a tail-only client) to easily get notified and take appropriate action.

It should be helpful to make this easier to discover and use. /cc @julieqiu @toothrot @cagedmantis

Perhaps it can be done by adding an example to maintner package documentation. Here's a starting point for an example that uses maintner.TailNetworkMutationSource to watch for all new tags created in the main Go repository and printing them to stdout:

// Tail events from the maintner server in a loop.
for {
	err := maintner.TailNetworkMutationSource(context.Background(), godata.Server, func(e maintner.MutationStreamEvent) error {
		if e.Err != nil {
			log.Printf("# ignoring err: %v\n", e.Err)
			time.Sleep(5 * time.Second)
			return nil
		}
		m := e.Mutation
		if m.Gerrit == nil || m.Gerrit.Project != "go.googlesource.com/go" {
			// Not a Gerrit event, or not the main Go repo. Skip.
			return nil
		}
		for _, r := range m.Gerrit.Refs {
			if !strings.HasPrefix(r.Ref, "refs/tags/") {
				// Not a tag. Skip.
				continue
			}
			fmt.Printf("new tag in go repo: name=%q sha1=%q\n", r.Ref[len("refs/tags/"):], r.Sha1)
		}
		return nil
	})
	log.Printf("tail error: %v; restarting\n", err)
	time.Sleep(time.Second)
}

Using a full maintner client or maintner.NewNetworkMutationSource is a little more involved because it requires preserving state, which means writing to disk or another place. But this option can be investigated too.

Finally, we can consider exposing this functionality via a higher level API, but I think it should be easier to prototype this before committing to a public API that's hard to change.

@dmitshur dmitshur added Builders x/build issues (builders, bots, dashboards) NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. FeatureRequest labels Feb 29, 2020
@dmitshur dmitshur added this to the Backlog milestone Feb 29, 2020
@dmitshur
Copy link
Contributor Author

dmitshur commented Feb 29, 2020

Also see issues #34864 and #36898.

/cc @mvdan @jmhodges

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Builders x/build issues (builders, bots, dashboards) Documentation FeatureRequest NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Development

No branches or pull requests

1 participant