+1 from me for granting microcephauto-connect to microcepth-support interface as it expected to be required.
kernel-module-load looks also needed to support more types of storage as you explained. It looks absolutely reasonable to me, so It is a +1 unless other @reviewers are aware of any better solution to handle this case
+2 votes for, 0 votes against, granting auto-connect of interfaces microceph-support and kernel-module-load to snap microceph. I have vetted the publisher for the use of the super-privileged interface.
@petersabaini - I noticed the latest revision on the snap on the store doesn’t include the kernel-module-load. Please let me know once this is uploaded and I’ll go ahead and grant access, making it live. Thanks!
while the snap version 19.2.0~git+snap36f71d7700 (rev 1148) got the microceph-support interface it seems the automatic review for the recent upload got rejected
Store upload failed:
interface 'microceph-support' not found in base declaration
interface 'microceph-support' not found in base declaration
interface 'microceph-support' not found in base declaration
unknown plugs interface name reference 'microceph-support'
unknown plugs interface name reference 'microceph-support'
unknown plugs interface name reference 'microceph-support'
hi @petersabaini - sorry for the delay in getting back to you.
I have gone ahead and manually approved the latest versions of this snap as the version of review-tools in the store does not include the support for the microchip-support interface. This interface has been merged upstream and should be released soon.
For now, let me know when you have a new release blocked by this error and I can go ahead and approve.