Fabiano Fidêncio | 1 Jul 2012 21:36

Re: [PATCH 10/10] Add desktop profile for Windows

On Wed, Jun 13, 2012 at 4:47 PM, Zeeshan Ali (Khattak)
<zeeshanak@...> wrote:
> On Wed, Jun 13, 2012 at 6:49 PM, Daniel P. Berrange
<berrange@...> wrote:
>> On Wed, Jun 13, 2012 at 06:44:29PM +0300, Zeeshan Ali (Khattak) wrote:
>>>
>>> >  <install-script id='http://microsoft.com/windows/sif'>
>>> >    <profile>desktop</profile>
>>> >
>>> >    <assets>
>>> >       <disk media='floppy'/>
>>> ..
>>> >  <install-script id='http://fedoraproject.org/kickstart'>
>>> >    <profile>desktop</profile>
>>> >
>>> >    <assets>
>>> >       <disk media='harddisk'/>
>>>
>>> How do we make sure that we assume the correct device name/number/path
>>> in the scripts? Why not have these as configuration parameters as well
>>> so that app tells us where to find these? Some parameters could be
>>> mandatory: app gets error if it doesn't set these params.
>>
>> I'd say the rule is that the disk external "data" disk should be the
>> first available unit number, after the primary VM disks. As written
>> the install scripts assume only a single primary VM disk is used.
>> If we wanted to allow more, we'd have to add a new parameter to the
>> install script API to allow the app to specify how many disks it is
>> attaching.
>
(Continue reading)

Fabiano Fidêncio | 1 Jul 2012 21:35

Re: [PATCH 10/10] Add desktop profile for Windows

On Wed, Jun 13, 2012 at 7:25 AM, Daniel P. Berrange
<berrange@...> wrote:
> On Mon, Jun 11, 2012 at 11:23:11PM -0300, Fabiano Fidêncio wrote:
>> +     <xsl:template match="/install-script-config">
>> +sc config TlntSvr start= auto
>> +net user <xsl:value-of select="config/user-realname"/> <xsl:value-of
select="config/admin-password"/> /add /passwordreq:no
>> +net localgroup administrators <xsl:value-of select="config/user-realname"/> /add
>> +net accounts /maxpwage:unlimited
>> +copy a:\<xsl:value-of select="config/user-realname"/>.bmp "c:\Documents and Settings\All
Users\Application Data\Microsoft\User Account Pictures"
>> +REGEDIT /S a:\windows.reg
>> +EXIT
>
>
> ...<continued my reply from the fedora patch>
>
> where as to use the Windows desktop profile, an application needs to know
> that they must provide a floppy disk containing an icon file in the BMP
> format.
>
> I'm wondering how we can avoid the need for applications to know about
> these OS specific requirements.
>
>
>
>> diff --git a/data/oses/windows.xml b/data/oses/windows.xml
>> index 8769511..2fb18be 100644
>> --- a/data/oses/windows.xml
>> +++ b/data/oses/windows.xml
(Continue reading)

Fabiano Fidêncio | 1 Jul 2012 21:36

Re: [PATCH 08/10] Add desktop profile for Fedora 15 and newer

On Wed, Jun 13, 2012 at 7:16 AM, Daniel P. Berrange
<berrange@...> wrote:
> On Mon, Jun 11, 2012 at 11:23:09PM -0300, Fabiano Fidêncio wrote:
>> +# Set user avatar
>> +mkdir /mnt/unattended-media
>> +mount /dev/sda /mnt/unattended-media
>> +cp /mnt/unattended-media/<xsl:value-of select="config/user-login"/> /var/lib/AccountsService/icons/
>> +umount /mnt/unattended-media
>
> Interesting. So to use the Fedora desktop profile, an application needs
> to already know that they must provide a 'sda' hard disk, containing
> a filesystem containing a specially named icon file.....

To be honest, it is part of set_direct_boot_params() function, that
will be moved from gnome-boxes to libosinfo, IIUC.
I think we can leave it as is, at least for now. But, of course, I
really want to read your opinion.

>
> ...<continued in my reply to the windows patch>
>
> Daniel
> --
> |: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
> |: http://libvirt.org              -o-             http://virt-manager.org :|
> |: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
> |: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--

-- 
Fabiano Fidêncio
(Continue reading)

Daniel P. Berrange | 2 Jul 2012 12:02
Picon
Favicon

Re: [PATCH 10/10] Add desktop profile for Windows

On Sun, Jul 01, 2012 at 04:36:20PM -0300, Fabiano Fidêncio wrote:
> On Wed, Jun 13, 2012 at 4:47 PM, Zeeshan Ali (Khattak)
> <zeeshanak <at> gnome.org> wrote:
> > On Wed, Jun 13, 2012 at 6:49 PM, Daniel P. Berrange <berrange <at> redhat.com> wrote:
> >> On Wed, Jun 13, 2012 at 06:44:29PM +0300, Zeeshan Ali (Khattak) wrote:
> >>>
> >>> >  <install-script id='http://microsoft.com/windows/sif'>
> >>> >    <profile>desktop</profile>
> >>> >
> >>> >    <assets>
> >>> >       <disk media='floppy'/>
> >>> ..
> >>> >  <install-script id='http://fedoraproject.org/kickstart'>
> >>> >    <profile>desktop</profile>
> >>> >
> >>> >    <assets>
> >>> >       <disk media='harddisk'/>
> >>>
> >>> How do we make sure that we assume the correct device name/number/path
> >>> in the scripts? Why not have these as configuration parameters as well
> >>> so that app tells us where to find these? Some parameters could be
> >>> mandatory: app gets error if it doesn't set these params.
> >>
> >> I'd say the rule is that the disk external "data" disk should be the
> >> first available unit number, after the primary VM disks. As written
> >> the install scripts assume only a single primary VM disk is used.
> >> If we wanted to allow more, we'd have to add a new parameter to the
> >> install script API to allow the app to specify how many disks it is
> >> attaching.
> >
(Continue reading)

Zeeshan Ali (Khattak | 3 Jul 2012 01:37
Picon

Re: [PATCH 08/10] Add desktop profile for Fedora 15 and newer

On Sun, Jul 1, 2012 at 10:36 PM, Fabiano Fidêncio <fabiano <at> fidencio.org> wrote:
> On Wed, Jun 13, 2012 at 7:16 AM, Daniel P. Berrange <berrange <at> redhat.com> wrote:
>> On Mon, Jun 11, 2012 at 11:23:09PM -0300, Fabiano Fidêncio wrote:
>>> +# Set user avatar
>>> +mkdir /mnt/unattended-media
>>> +mount /dev/sda /mnt/unattended-media
>>> +cp /mnt/unattended-media/<xsl:value-of select="config/user-login"/> /var/lib/AccountsService/icons/
>>> +umount /mnt/unattended-media
>>
>> Interesting. So to use the Fedora desktop profile, an application needs
>> to already know that they must provide a 'sda' hard disk, containing
>> a filesystem containing a specially named icon file.....
>
> To be honest, it is part of set_direct_boot_params() function, that
> will be moved from gnome-boxes to libosinfo, IIUC.
> I think we can leave it as is, at least for now. But, of course, I
> really want to read your opinion.

He already gave you his opinion: We can't make those assumptions in
libosinfo. While Boxes may or may not be the primary user of
libosinfo, we need to keep the API generic enough for other apps too.
We can't leave this as is since this will be broken for every other
app but Boxes. Also things don't remain the same in Boxes so the next
time the VM configuration code changes, nobody would remember to
update this code in libosinfo and we'll have mysterious issues at our
hand.

--

-- 
Regards,

(Continue reading)

Fabiano Fidêncio | 3 Jul 2012 05:22

Fwd: [PATCH 09/10] Change osinfo_install_script_generate to write the output in a file

---------- Forwarded message ----------
From: Fabiano Fidêncio <fabiano@...>
Date: Sun, Jul 1, 2012 at 4:36 PM
Subject: Re: [virt-tools-list] [PATCH 09/10] Change
osinfo_install_script_generate to write the output in a file
To: "Daniel P. Berrange" <berrange@...>

On Wed, Jun 13, 2012 at 7:35 AM, Daniel P. Berrange
<berrange@...> wrote:
> On Mon, Jun 11, 2012 at 11:23:10PM -0300, Fabiano Fidêncio wrote:
>>     For libosinfo API:
>>         Now, instead of return a string, writes that string in a file,
>>         passed as argument
>>
>>     For osinfo-install-script tool:
>>         The output will be wrote in:
>>             - Linuxes: <short_id>,ks
>>             - Windows 2k3r2 and older: windows.sif
>>             - Windows 2k8 and newer: windows.xml
>>
>>     It will be used to create, easily, multiple scripts, as used in:
>>     http://bugzilla-attachments.gnome.org/attachment.cgi?id=214681
>
> I was a bit dubious about this one at first, but seeing what you
> needed to do with the windows scripts, I understand now.
>
> If we follow my suggestion, of actually putting the required
> filename in the XML metadata, this API will need to change
> slightly too.

(Continue reading)

Zeeshan Ali (Khattak | 3 Jul 2012 19:18
Picon

Re: Fwd: [PATCH 09/10] Change osinfo_install_script_generate to write the output in a file

On Tue, Jul 3, 2012 at 6:22 AM, Fabiano Fidêncio <fabiano <at> fidencio.org> wrote:
> ---------- Forwarded message ----------
> From: Fabiano Fidêncio <fabiano <at> fidencio.org>
> Date: Sun, Jul 1, 2012 at 4:36 PM
> Subject: Re: [virt-tools-list] [PATCH 09/10] Change
> osinfo_install_script_generate to write the output in a file
> To: "Daniel P. Berrange" <berrange <at> redhat.com>
>
>
> On Wed, Jun 13, 2012 at 7:35 AM, Daniel P. Berrange <berrange <at> redhat.com> wrote:
>> On Mon, Jun 11, 2012 at 11:23:10PM -0300, Fabiano Fidêncio wrote:
>>>     For libosinfo API:
>>>         Now, instead of return a string, writes that string in a file,
>>>         passed as argument
>>>
>>>     For osinfo-install-script tool:
>>>         The output will be wrote in:
>>>             - Linuxes: <short_id>,ks
>>>             - Windows 2k3r2 and older: windows.sif
>>>             - Windows 2k8 and newer: windows.xml
>>>
>>>     It will be used to create, easily, multiple scripts, as used in:
>>>     http://bugzilla-attachments.gnome.org/attachment.cgi?id=214681
>>
>> I was a bit dubious about this one at first, but seeing what you
>> needed to do with the windows scripts, I understand now.
>>
>> If we follow my suggestion, of actually putting the required
>> filename in the XML metadata, this API will need to change
>> slightly too.
(Continue reading)

Zeeshan Ali (Khattak | 4 Jul 2012 21:45
Picon

Tracking VM installations

Hi,

We have a requirement in Boxes for showing progress bars on VMs under
installation:

https://github.com/gnome-design-team/gnome-mockups/raw/master/boxes/boxes-install6.png
https://bugzilla.gnome.org/show_bug.cgi?id=679107

However, we are bit clueless on how that could be achieved. The only
idea I have is to know the number of passes (reboots) and typical disk
usage in each pass for "every" (we only do this for express installs
so we don't need this for all) OS and estimate the progress based on
that knowledge. If that idea is worth looking into, we'll need this
information in libosinfo and hence me cross-posting this to
virt-tools-list.

Other ideas welcome! Keep in mind that we don't need to be very
accurate, just accurate enough to not annoy the user.

--

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124

Richard W.M. Jones | 4 Jul 2012 23:09
Picon
Favicon
Gravatar

Re: Tracking VM installations

On Wed, Jul 04, 2012 at 10:45:29PM +0300, Zeeshan Ali (Khattak) wrote:
> Hi,
> 
> We have a requirement in Boxes for showing progress bars on VMs under
> installation:
> 
> https://github.com/gnome-design-team/gnome-mockups/raw/master/boxes/boxes-install6.png
> https://bugzilla.gnome.org/show_bug.cgi?id=679107
> 
> However, we are bit clueless on how that could be achieved. The only
> idea I have is to know the number of passes (reboots) and typical disk
> usage in each pass for "every" (we only do this for express installs
> so we don't need this for all) OS and estimate the progress based on
> that knowledge. If that idea is worth looking into, we'll need this
> information in libosinfo and hence me cross-posting this to
> virt-tools-list.
> 
> Other ideas welcome! Keep in mind that we don't need to be very
> accurate, just accurate enough to not annoy the user.

It's a tricky one :-)

You can take a screenshot of the guest (assuming a visual
installation).  Oz does this.  Displaying an icon-sized screenshot
might be a surrogate for displaying a progress bar, since many
installers will display a progress bar themselves.
https://github.com/clalancette/oz/blob/74d99497840e3bade4d679ed94cf9a4628827bcc/oz/Guest.py#L624

You can use libguestfs (specifically, virt-df) to display the amount
of disk space used in the guest.  For many installers, disk space will
(Continue reading)

Daniel P. Berrange | 5 Jul 2012 11:21
Picon
Favicon

Re: Tracking VM installations

On Wed, Jul 04, 2012 at 10:45:29PM +0300, Zeeshan Ali (Khattak) wrote:
> Hi,
> 
> We have a requirement in Boxes for showing progress bars on VMs under
> installation:
> 
> https://github.com/gnome-design-team/gnome-mockups/raw/master/boxes/boxes-install6.png
> https://bugzilla.gnome.org/show_bug.cgi?id=679107
> 
> However, we are bit clueless on how that could be achieved. The only
> idea I have is to know the number of passes (reboots) and typical disk
> usage in each pass for "every" (we only do this for express installs
> so we don't need this for all) OS and estimate the progress based on
> that knowledge. If that idea is worth looking into, we'll need this
> information in libosinfo and hence me cross-posting this to
> virt-tools-list.
> 
> Other ideas welcome! Keep in mind that we don't need to be very
> accurate, just accurate enough to not annoy the user.

I don't see any general purpose way todo a progress feedback. The best
you can do is to show whether /something/ is happening (by looking at
disk & net I/O stats).  If you want actual progress feedback then you're
looking at modifying the guest OS init process / installer to send feedback
over a virtio-serial device directly, or indirectly via a guest agent.

Daniel
--

-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
(Continue reading)


Gmane