Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Investigate why npm run release is not enough to trigger distro job on CI #226

Open
barmac opened this issue Apr 30, 2020 · 6 comments
Open

Comments

@barmac
Copy link
Contributor

barmac commented Apr 30, 2020

What should we do?

Investigate why npm run release is not enough to trigger distro job on CI. The possible reason is that we do not have a tag commit as creating this manually fixes the problem.

Why should we do it?

When I run npm run release for Zeebe Modeler 0.9.1 release, lerna created just an ordinary git tag instead of an annotated one. This caused Travis CI to skip the commit in the distro stage. I had to replace tag manually. @pinussilvestrus confirmed that he experienced the same issue.

@nikku
Copy link
Contributor

nikku commented Apr 30, 2020

What is an ordinary vs. an annotated tag? 🤔

@barmac
Copy link
Contributor Author

barmac commented Apr 30, 2020

git tag v0.9.1 (that's what probably lerna run with npm run release, outcome) v. git tag -a v0.9.1 (outcome). In the second case you get a commit which contains only the commit message plus reference to tag.

I guess we could just fix it on CI. 🤷

@nikku
Copy link
Contributor

nikku commented Apr 30, 2020

I do not see the relevant difference.

In both cases, CI will pick up the new tag and build it. Right?

@pinussilvestrus
Copy link
Contributor

Nope, that's the thing here. In the first scenario, CI does not trigger the distro builds. Maciej had to annotate the tag manually to properly work

@barmac
Copy link
Contributor Author

barmac commented Apr 30, 2020

OK let's clarify this issue. The problem was that I had to manually create a tag commit to have the release picked up by CI.

@barmac barmac changed the title Make sure to have an annotated tag on each release Investigate why npm run release is not enough to trigger distro job on CI Apr 30, 2020
@barmac
Copy link
Contributor Author

barmac commented Apr 30, 2020

I updated the issue description.

@nikku nikku added this to the Z11 milestone May 27, 2020
@nikku nikku added the backlog label Jun 10, 2020 — with bpmn-io-tasks
@barmac barmac removed this from the Z11 milestone Aug 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants