You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I agree this is confusing... I think we could standardize on 'slaveImage' but I would probably want to leave the old param names in place so we don't break anyone's existing pipelines.
https://github.com/RedHatInsights/insights-pipeline-lib/blob/master/vars/openShiftUtils.groovy
Examples:
So in 3 functions the same thing is called 3 different names: jenkinsSlaveImage, slaveImage and image.
I think it would be better to unify them to one name.
The text was updated successfully, but these errors were encountered: