infiniband-diags can't be installed in Fedora CoreOS due to perl dependency

infiniband-diags can't be installed in Fedora CoreOS due to perl dependency

[St Savage, Shane]

Hi all,

Just wanted to report that infiniband-diags cannot currently be installed in Fedora CoreOS because the perl dependency is explicitly forbidden.

https://github.com/coreos/fedora-coreos-config/blob/testing-devel/manifests/fedora-coreos.yaml#L170

This is a bit unfortunate because it also prevents usage of all the non-perl utilities (ibstat, etc) included in infiniband-diags.

Would it make sense to split the perl utilities to a separate package infiniband-diags-perl so that the C and shell utilities in infiniband-diags can be installed without the perl dependency?

Thank you,
Shane St Savage

infiniband-diags can't be installed in Fedora CoreOS due to perl dependency

[Shane St. Savage]

Hi all,

Just wanted to report that infiniband-diags cannot currently be
installed in Fedora CoreOS because the perl dependency is explicitly
forbidden.

https://github.com/coreos/fedora-coreos-config/blob/testing-devel/manifests/fedora-coreos.yaml#L170

This is a bit unfortunate because it also prevents usage of all the
non-perl utilities (ibstat, etc) included in infiniband-diags.

Would it make sense to split the perl utilities to a separate package
infiniband-diags-perl so that the C and shell utilities in
infiniband-diags can be installed without the perl dependency?

Thank you,
Shane St Savage

Re: infiniband-diags can't be installed in Fedora CoreOS due to perl dependency

[Leon Romanovsky]

On Sat, Aug 12, 2023 at 04:40:57PM +0000, St Savage, Shane wrote:
> Hi all,
> 
> Just wanted to report that infiniband-diags cannot currently be installed in Fedora CoreOS because the perl dependency is explicitly forbidden.
> 
> https://github.com/coreos/fedora-coreos-config/blob/testing-devel/manifests/fedora-coreos.yaml#L170
> 
> This is a bit unfortunate because it also prevents usage of all the non-perl utilities (ibstat, etc) included in infiniband-diags.
> 
> Would it make sense to split the perl utilities to a separate package infiniband-diags-perl so that the C and shell utilities in infiniband-diags can be installed without the perl dependency?

I suggest to remove perl dependency from rdma--core.spec and install
perl-dependant scripts only if perl is found on the system.

Thanks

> 
> Thank you,
> Shane St Savage

Re: infiniband-diags can't be installed in Fedora CoreOS due to perl dependency

[Jason Gunthorpe]

On Sun, Aug 13, 2023 at 01:33:05PM +0300, Leon Romanovsky wrote:
> On Sat, Aug 12, 2023 at 04:40:57PM +0000, St Savage, Shane wrote:
> > Hi all,
> > 
> > Just wanted to report that infiniband-diags cannot currently be installed in Fedora CoreOS because the perl dependency is explicitly forbidden.
> > 
> > https://github.com/coreos/fedora-coreos-config/blob/testing-devel/manifests/fedora-coreos.yaml#L170
> > 
> > This is a bit unfortunate because it also prevents usage of all the non-perl utilities (ibstat, etc) included in infiniband-diags.
> > 
> > Would it make sense to split the perl utilities to a separate package infiniband-diags-perl so that the C and shell utilities in infiniband-diags can be installed without the perl dependency?
> 
> I suggest to remove perl dependency from rdma--core.spec and install
> perl-dependant scripts only if perl is found on the system.

That is not how packaging is supposed to work

Everything should be installed always

This is really a Fedora question, we just follow what they
decide. Most likely the right answer is to put this kind of stuff in a
container and not run on the minimal coreos image.

Jason

Re: infiniband-diags can't be installed in Fedora CoreOS due to perl dependency

[Leon Romanovsky]

On Mon, Aug 14, 2023 at 12:32:24PM -0300, Jason Gunthorpe wrote:
> On Sun, Aug 13, 2023 at 01:33:05PM +0300, Leon Romanovsky wrote:
> > On Sat, Aug 12, 2023 at 04:40:57PM +0000, St Savage, Shane wrote:
> > > Hi all,
> > > 
> > > Just wanted to report that infiniband-diags cannot currently be installed in Fedora CoreOS because the perl dependency is explicitly forbidden.
> > > 
> > > https://github.com/coreos/fedora-coreos-config/blob/testing-devel/manifests/fedora-coreos.yaml#L170
> > > 
> > > This is a bit unfortunate because it also prevents usage of all the non-perl utilities (ibstat, etc) included in infiniband-diags.
> > > 
> > > Would it make sense to split the perl utilities to a separate package infiniband-diags-perl so that the C and shell utilities in infiniband-diags can be installed without the perl dependency?
> > 
> > I suggest to remove perl dependency from rdma--core.spec and install
> > perl-dependant scripts only if perl is found on the system.
> 
> That is not how packaging is supposed to work
> 
> Everything should be installed always

Isn't how we use some packages in pyverbs?

Thanks

> 
> This is really a Fedora question, we just follow what they
> decide. Most likely the right answer is to put this kind of stuff in a
> container and not run on the minimal coreos image.
> 
> Jason

Re: infiniband-diags can't be installed in Fedora CoreOS due to perl dependency

[Jason Gunthorpe]

On Mon, Aug 14, 2023 at 09:20:38PM +0300, Leon Romanovsky wrote:
> On Mon, Aug 14, 2023 at 12:32:24PM -0300, Jason Gunthorpe wrote:
> > On Sun, Aug 13, 2023 at 01:33:05PM +0300, Leon Romanovsky wrote:
> > > On Sat, Aug 12, 2023 at 04:40:57PM +0000, St Savage, Shane wrote:
> > > > Hi all,
> > > > 
> > > > Just wanted to report that infiniband-diags cannot currently be installed in Fedora CoreOS because the perl dependency is explicitly forbidden.
> > > > 
> > > > https://github.com/coreos/fedora-coreos-config/blob/testing-devel/manifests/fedora-coreos.yaml#L170
> > > > 
> > > > This is a bit unfortunate because it also prevents usage of all the non-perl utilities (ibstat, etc) included in infiniband-diags.
> > > > 
> > > > Would it make sense to split the perl utilities to a separate package infiniband-diags-perl so that the C and shell utilities in infiniband-diags can be installed without the perl dependency?
> > > 
> > > I suggest to remove perl dependency from rdma--core.spec and install
> > > perl-dependant scripts only if perl is found on the system.
> > 
> > That is not how packaging is supposed to work
> > 
> > Everything should be installed always
> 
> Isn't how we use some packages in pyverbs?

No, just the test suite which we don't really package as an actual
binary

Jason