From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leon Romanovsky Subject: Re: srp_daemon : =?utf-8?B?4oCLRGlzYWxs?= =?utf-8?B?b3cg4oCLYWxsIHRhcmdldHMg4oCLaWYg4oCLbm90IOKAi2V4cGxpY2l0bHkg?= =?utf-8?Q?=E2=80=8Ballowed?= by default Date: Wed, 10 May 2017 19:05:36 +0300 Message-ID: <20170510160536.GE1839@mtr-leonro.local> References: <1494349653.3752.4.camel@profitbricks.com> <021b0a17-0552-6ee4-ec19-0e86ead527ef@sandisk.com> <1050725096.6555338.1494351706650.JavaMail.zimbra@redhat.com> <20170509180159.GK10073@mtr-leonro.local> <1494423046.3739.8.camel@profitbricks.com> <22803.7541.113159.214164@gargle.gargle.HOWL> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="VUDLurXRWRKrGuMn" Return-path: Content-Disposition: inline In-Reply-To: <22803.7541.113159.214164-4mDQ13Tdud8Jw5R7aSpS0dP8p4LwMBBS@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Roland Fehrenbacher Cc: Benjamin Drung , Laurence Oberman , Bart Van Assche , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org --VUDLurXRWRKrGuMn Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, May 10, 2017 at 04:02:29PM +0200, Roland Fehrenbacher wrote: > >>>>> "BD" =3D=3D Benjamin Drung writes: > > Hi all, > > BD> Am Dienstag, den 09.05.2017, 21:01 +0300 schrieb Leon > BD> Romanovsky: > >> On Tue, May 09, 2017 at 01:41:46PM -0400, Laurence Oberman wrote: > >> > > >> > > >> > ----- Original Message ----- > >> > > From: "Bart Van Assche" To: > >> > > "Benjamin Drung" , linux-rdm > >> > > a@vger.kernel.org Sent: Tuesday, May 9, 2017 1:11:06 PM > >> > > Subject: Re: srp_daemon : Disallow all targets if not > >> > > explicitly allowed by default > >> > > > >> > > On 05/09/17 10:07, Benjamin Drung wrote: > >> > > > srptools 1.0.3-2 in Debian disallows all targets if not > >> > > > explicitly allowed by default. Motivation (taken from > >> > > > debian/changelog): > >> > > > > >> > > > * Don't activate any targets per default. (Closes: #740945) > >> > > > =A0 This is more sensible than the previous default of > >> > > > bringing =A0 up all targets in the IB fabric upon boot. In a > >> > > > larger fabric =A0 with many storage targets available, most > >> > > > of the times only =A0 one or a few targets are wanted on a > >> > > > particular machine. > >> > > > > >> > > > Do you agree and could you change the default upstream? I > >> > > > prefer not to carry a different behavior in Debian alone. > >> > > > >> > > Hello Benjamin, > >> > > > >> > > What I expect is that users will hate this change. They will > >> > > notice that after they have installed and enabled srp_daemon > >> > > that no targets are discovered without having any clue why no > >> > > automatic login to SRP targets occurs. > >> > > > >> > > Bart. > >> > > > >> > > -- To unsubscribe from this list: send the line "unsubscribe > >> > > linux- rdma" in the body of a message to > >> > > majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info > >> > > at=A0=A0http://vger.kernel.org/majordomo-info.htm l > >> > > > >> > > >> > Hello > >> > > >> > Indeed, I agree with Bart here, this change will lead to a lot > >> > of confusion about why users no longer see device discovery. > >> > For me I would prefer its not changed. > >> > >> According to the Benjamin's description, this is already behavior > >> of Debian and their derivatives. So this change won't change for > >> these users anything. I think that the best solution will be to > >> keep in sync distros and upstream. Or enable by default on all > >> systems or disable by default on all systems. > > BD> This change was uploaded to Debian unstable two days ago. So it > BD> is a recent change done by Roland Fehrenbacher (CCed) which > BD> still can be reverted. I'm fine with any option as long as > BD> Debian and upstream are in sync. > > BD> Bart's objection (that users of large setups know how to edit > BD> the srp_daemon configuration file but most users of small setups > BD> don't know how to edit that configuration file) seem valid to > BD> me. Roland, your input please. > > it's hard to judge what is the most wanted default setup. The reason why > this was changed was the cited Debian bug report. I'm also a friend of > well-defined configs, so I agree with that opinion. I believe, > adding a '#' in front of a line of a config file shouldn't overburden an > admin who deals with IB storage under Linux, if she/he prefers the most > simple config. The change in Debian is also clearly communicated (NEWS > file), so I don't really see a problem. Finally: There are many packages > in Debian that have a different default config as compared to upstream. There are number of reasons to keep upstream and distros in sync. Main reasons are simplify testing (test once for all distros), packaging (part of our CI checks) and developing (new features are added to all distros automatically). Thanks > > Cheers, > > Roland --VUDLurXRWRKrGuMn Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEkhr/r4Op1/04yqaB5GN7iDZyWKcFAlkTOlAACgkQ5GN7iDZy WKfCXw//bcERqIl2p9fJ0MzOwgoqhn0gZeDoqo05ljn/dv+DMbL11CAIo6aOshv/ 3/FiCOKGyq0l7ZmVvXHyNFU2eAp6D+JaBayLgIIetmZkP48KXJ6778BUj1T0gRmq fkQdeO2uXZZPtwmoXo2KWhPT4M4WwhV1+L7NbqPt16L2M6Y2Kd++VUnx/Ppsfoj7 Jmji6ahKDTz/WfdSUMjkMAUMsaO79Hyie325p8dd0KljTMp01ERJYjdPB1cBGB0E 93mJg/WlbS+EAq0HUtCMiUBNSUAI2jw2YtlX3Al1h+Iclvs9r5z9MmGDuUyL0wy4 W0bb85y4Q+zoE5EjwNfzBNAZUPLFVxofFupu/T9XfNDiboJKTGht/RAAOWI1oHvy esPFV/B/vPlQQASD0p1H8DkcHGvTyI/AdB6+SUxDZ5EzZrTCeHlS/KJlOtVfAxAS US6Uc/58rBGFEvzsSzpr10tMvYUo4xTJjzlCSS0lrxkkCLkdOt1AOEAW7ihJ2ql2 U/YreLSRQiiT2i8wByE9WakqIvO553n7a88l40DPm8FuVka2/Rc6nYm5VccBkBVf Ig3X7WBRSIiXOC1aYsAAgupH/HPVicvLmieNbBgXnKSLVLc5kwbSV7hYj2qcfGRY E5c1Vm38HMWhZl9ANiB5bDePyBi+bom6DhoOL+nt1gZgkoVOy6w= =Psa/ -----END PGP SIGNATURE----- --VUDLurXRWRKrGuMn-- -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html