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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GitHub Target] Feature proposal: a templated list of tags to push #445

Open
tonyo opened this issue Jan 17, 2023 · 4 comments
Open

[GitHub Target] Feature proposal: a templated list of tags to push #445

tonyo opened this issue Jan 17, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@tonyo
Copy link
Member

tonyo commented Jan 17, 2023

We (the Web Backend SDK team) can implement on this, just want to get a 馃憤 before starting.

Context: Releasing Golang Submodules

A Go module is a collection of Go packages that are distributed as one unit.
Go also supports submodules (or "nested modules"): basically, they are Go modules that live in the same repository, but in different directories, each submodule having its own go.mod file in its root. The top-most project directory in this case becomes the root module.

Example: https://github.com/open-telemetry/opentelemetry-go is a Go module that has submodules trace, sdk, schema, and some others.

sentry-go will add a few submodules in the near future (e.g. our OpenTelemetry integration will live in a new submodule).

Now, when releasing a new version of a submodule (which can be released independently from the root module, but we're not going to do that for sentry-go for now), we have to tag the repository with "personalized" Git tag. So e.g. when we release v1.2.3 of submodule otel in sentry-go, we'll have to push the Git tag that would look like otel/v1.2.3. Each submodule will get its own tag in the form of {MODULE_PREFIX}/{VERSION}.

Problem

When making a new release for a Go project with a bunch of submodules, we need to push several Git tags to GitHub.
Craft (or specifically, the GitHub target) doesn't support pushing more than one tag at the moment

Proposed Solution

Add a new option to the github target: createTags (or just tags, name TBD) of type string[]. It's essentially a more powerful extension to the tagPrefix parameter that allows to specify a list of tags that have to be pushed by the target.
It will also support basic templating, so that e.g. the version to-be-released is a template variable that can be used.

For sentry-go the config can look something like this:

targets:
  - name: github
    createTags:
      - "v{VERSION}"      # Root module
      - "otel/v{VERSION}" # Nested module
@tonyo tonyo added the enhancement New feature or request label Jan 17, 2023
@tonyo tonyo self-assigned this Jan 17, 2023
@BYK
Copy link
Contributor

BYK commented Jan 17, 2023

I'm not on the team but I think this is a pretty good idea 鈽猴笍

Suggestions:

  1. Just name it tags
  2. Deprecate the tagPrefix config and roll it into this one
  3. Document the variables available to basic templating very well

@tonyo
Copy link
Member Author

tonyo commented Jan 23, 2023

Thanks @BYK 馃憤

@asottile-sentry asked whether we already can do the same thing just using multiple github targets with different tagPrefix.
I double-checked, and the problem right now is that every github target basically creates a full-fledged GitHub release that is visible on the Releases page, and we don't really need/want that for Go (just tags).

So the refined suggestion looks like this:

  1. No tags attribute, tagPrefix is empty or present => legacy behavior, print a warning that tagPrefix is deprecated.
  2. Both tags and tagPrefix are non-empty => raise an error, exit.
  3. tags is present, tagPrefix empty => new behavior, and the first tag from the list is used for creating the GitHub release.

@BYK
Copy link
Contributor

BYK commented Jan 23, 2023

Sounds good to me. An alternative might be to allow tagPrefix to be an array of strings (new behavior) or just a string. This is a bit "dirty" but allows for easier configuration and avoids any config migrations.

@tonyo
Copy link
Member Author

tonyo commented Jan 26, 2023

Gave it a bit more thought and decided not to implement this proposal right now: the use case is not strong enough to spend days/weeks working on the feature and then deprecating/migrating the existing apps.
Here's a smaller change that will cover our Go case: #447

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants