[Sisuite-devel] RE: [systemimager-commits] r3547 - trunk/sbin

[Bernard Li]

Title: Re: [systemimager-commits] r3547 - trunk/sbin

Hi Andrea:

 

Basically I was trying to fix the following "bug":

 

After imaging has completed, the last message in /var/lib/systemimager/clients.xml usually is a message "Imaging Complete".  So when you invoke the virtual console after you lose connection, it would say "Unable to retrieve previous messages!" but then after that it will say "Imaging Complete".

 

If you can fix this behaviour with the functionality you want intact, that's fine... :-)

 

BTW, I have tested on x86_64 and for some reason it always say it could not gather previous messages (eg. no dmesg output at all) - I wonder if there are some network port issues...  do you know how I can debug that?  Alternatively, have you tested virtual console under x86_64?  It works fine for me under x86.

 

Thanks,

 

Bernard

---

From: Andrea Righi [mailto:[EMAIL PROTECTED]
Sent: Sat 06/05/2006 03:10
To: Bernard Li
Cc: sisuite-devel@lists.sourceforge.net
Subject: Re: [systemimager-commits] r3547 - trunk/sbin

Bernard,

sometimes when I reinstall a client I usually open the virtual console
and then I launch the reinstallation. In this case the previous messages
cannot be retrieved (because there is no previous message), but when the
network goes up on the client I can see the messages on the virtual
console without closing and reopening the window... with your change
this behaviour is not supported... what do you think, could we need that?

Cheers,
-Andrea

Bernard Li wrote:
> Author: bli
> Date: 2006-05-03 00:59:16 -0500 (Wed, 03 May 2006)
> New Revision: 3547
>
> Modified:
>    trunk/sbin/si_monitortk
> Log:
> Only refresh virtual console message if there were no errors retrieving previous messages
> - in this case, if we cannot reach the host for previous messages, it will not print the
>   last message from /var/lib/systemimager/clients.xml
>
>
> Modified: trunk/sbin/si_monitortk
> ===================================================================
> --- trunk/sbin/si_monitortk   2006-05-01 17:43:52 UTC (rev 3546)
> +++ trunk/sbin/si_monitortk   2006-05-03 05:59:16 UTC (rev 3547)
> @@ -498,15 +498,14 @@
>                  'title'
>              );
>          }
> +        # Refresh console view (default 100ms).
> +        my $cons_timer = Tk::After->new(
> +                $sub_win,
> +                100,
> +                'repeat',
> +                \&refresh_console
> +        );
>      }
> -
> -    # Refresh console view (default 100ms).
> -    my $cons_timer = Tk::After->new(
> -            $sub_win,
> -            100,
> -            'repeat',
> -            \&refresh_console
> -    );
>  }
> 
>  # Usage:
>