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

No release is made on merge with master #314

Open
connoratrug opened this issue Jun 17, 2020 · 3 comments
Open

No release is made on merge with master #314

connoratrug opened this issue Jun 17, 2020 · 3 comments
Labels
bug Something isn't working chart-jenkins Jenkins helm chart High impact issue 🚒

Comments

@connoratrug
Copy link
Contributor

Which chart are you referring?

molgenis-jenkins

How to reproduce

Merge commit containing Conventional Commit Message containing feat: into master

Expected behavior

A release is made is build is successful

Observed behavior

No release is made (sometimes)

for example: https://jenkins.dev.molgenis.org/blue/organizations/jenkins/molgenis%2Fmolgenis-node-lifelines-edge/detail/master/66/pipeline

@connoratrug connoratrug added bug Something isn't working chart-jenkins Jenkins helm chart High impact issue 🚒 labels Jun 17, 2020
@jvanveen
Copy link
Contributor

jvanveen commented Jun 17, 2020

De vorige build werd getriggered door commit 39a0318. Semantic release maakt vervolgens tijdens het bouwen een release commit(0d6dc2c). De volgende build gaat daarna over commit 39a03185, maar ook over 0d6dc2c(de chore commit). 0d6dc2c had al bij Jenkins bekend moeten zijn.

@connoratrug
Copy link
Contributor Author

connoratrug commented Jun 17, 2020

When i look at the webhook settings in de github admin panel i can see lots of hooks where failing , updated the hook url from htttp to https , after this the hooks where successful, but do not know it this is really connected to this issue

@fdlk
Copy link
Contributor

fdlk commented Jul 17, 2020

In the build log, it says "Stage "Release: [ master ]" skipped due to when conditional" so I'd think this is an issue in the Jenkins Pipeline changelog condition, which checks all commits instead of the last.
This makes "build this when not [skip ci]" skip the build as soon as one of the merged commits is "[skip ci]"
That's useful when you want to trigger on a changelog [release this] type of tag, you want to trigger then when one of the commits contains the tag. But we negate it and then it means the steps are skipped as soon as one of the commits contains [skip ci]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working chart-jenkins Jenkins helm chart High impact issue 🚒
Projects
None yet
Development

No branches or pull requests

3 participants