Re: [PATCH lorax] Add fpaste to the installation environment.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2011-05-24 at 06:23 -0400, Martin Sivak wrote:
> Hi,
> 
> I do not think this is such a great idea. While this utility might be small, I do not think we should be adding unnecessary stuff like this into the basic initrd while at the same time trying to reduce memory consumption.
> 
> I see two use cases which might happen:
> 
> 1) Bug reporter will save the bug using our GUI/pyreport/scp and all the logs will go directly into bugzilla. He has no need for fpaste at that moment. And when no error emerges, he would benefit from having smaller image to download and unpack into memory.
> 
> 2) Developer and the people debugging would usually have no problem with attaching second initrd (using initrd overlay) containing all necessary debugging tools. It might be even included as separate choice in the CD boot menu and we have had it in our Brno PXE targets for years now (it is automatically added, no tedious configuration necessary). The same applies to all debug only utilities like strace, gdbserver, ...
> 
> In my point of view, we keep adding stuff like this and then complain about dependencies and the image's size. That is just hypocrisy.
> 
> PS:
> 
> If you do not require pasting the stuff on a website, we already have bash and it's netcat implementation included. Usage bellow:
> 
> cat > /dev/tcp/_hostname_/_port_ < _logfile_
> 
> Do not be deceived by the fact that no /dev/tcp exists, bash will understand it. You can then receive it by using:
> 
> nc -l _port_

I agree with Martin here. We talked about including fpaste few weeks
ago, and we decided not to include it. The problem that may come up in
the future is, that maybe someone will add some crazy dependencies to
fpaste, and we will end up with it in the initrd at the end, making the
size bigger again, and we would have to find out what packages are
getting installed as deps and are not needed for our purposes, and
adding more "remove" clauses in the template.

Adding utilities to initrd is nice and easy, but when we're fighting the
resulting size, we should really include only stuff that is needed for
functionality.

It started with strace, now fpaste and who knows where will it end...

> 
> --
> Martin SivÃk
> msivak@xxxxxxxxxx
> Red Hat Czech
> Anaconda team / Brno, CZ
> 
> ----- Original Message -----
> > On Mon, May 23, 2011 at 02:25:45PM -0400, Chris Lumens wrote:
> > > This can be handy when debugging.
> > > ---
> > >  share/ramdisk.ltmpl | 1 +
> > >  1 files changed, 1 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/share/ramdisk.ltmpl b/share/ramdisk.ltmpl
> > > index a8da015..3fc95db 100644
> > > --- a/share/ramdisk.ltmpl
> > > +++ b/share/ramdisk.ltmpl
> > > @@ -72,6 +72,7 @@ install "fedora-gnome-theme"
> > >  install "fedora-icon-theme"
> > >  install "firstaidkit-engine"
> > >  install "firstboot"
> > > +install "fpaste"
> > >  install "ftp"
> > >  install "gdb-gdbserver"
> > >  install "gdk-pixbuf"
> > > --
> > > 1.7.4.1
> > 
> > great idea, ack.
> > 
> > --
> > Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA
> > (PST8PDT)
> > 
> > _______________________________________________
> > Anaconda-devel-list mailing list
> > Anaconda-devel-list@xxxxxxxxxx
> > https://www.redhat.com/mailman/listinfo/anaconda-devel-list
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

-- 
Martin Gracik <mgracik@xxxxxxxxxx>

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list



[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux