search

vexxhost / magnum-cluster-api

Cluster API driver for OpenStack Magnum
Apache License 2.0
41 stars 17 forks source link

bug:ci pr body is mandatory now #342

Closed okozachenko1203 closed 3 months ago

okozachenko1203 commented 3 months ago

This line fails CIs instead of skipping for PRs having no body with this error 

Error in Start of Pipeline step.
Cannot invoke method contains() on null object

https://github.com/vexxhost/magnum-cluster-api/blob/2d3f3519d5e920d3a900bb6863c2643f42097370/Jenkinsfile#L4

okozachenko1203 commented 3 months ago

for instance, https://github.com/vexxhost/magnum-cluster-api/pull/341/checks?check_run_id=22980579762

mnaser commented 3 months ago

Not valid anymore since we're moved to Zuul.