Code review comment for ~bladernr/plainbox-provider-checkbox:add-sosreport-job

Revision history for this message
Jonathan Cave (jocave) wrote :

I think miscellanea/sosreport needs resource requirements added to make sure sosreport is available.

Given we are moving to provide support for the server plans in checkbox delivered both by deb and snap the pattern we have been using is:

requires:
 package.name == 'sosreport' or executable.name == 'sosreport'

(Excutable may not always be same as package name!)

review: Needs Fixing

« Back to merge proposal