Bug 236915 - add dogtail command to kickstart
Summary: add dogtail command to kickstart
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: pykickstart
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Chris Lumens
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: F8Target
TreeView+ depends on / blocked
 
Reported: 2007-04-18 13:07 UTC by Alexander Todorov
Modified: 2007-11-30 22:12 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-06-05 14:56:49 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
New option in pykickstart/parser.py (964 bytes, patch)
2007-04-18 13:11 UTC, Alexander Todorov
no flags Details | Diff
patch to pykicstart/data.py (399 bytes, patch)
2007-04-18 14:34 UTC, Alexander Todorov
no flags Details | Diff
Classes to handle dogtail command in commands/*.py, pykickstart v1.0 (1.13 KB, text/x-python)
2007-05-03 11:55 UTC, Alexander Todorov
no flags Details
Import dogtail in commands/__init__.py, pykickstart v1.0 (639 bytes, patch)
2007-05-03 11:55 UTC, Alexander Todorov
no flags Details | Diff
Add dogtail to F7 & RHEL5 in handlers/control.py, pykickstart v1.0 (770 bytes, patch)
2007-05-03 11:56 UTC, Alexander Todorov
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 239024 0 medium CLOSED Dogtail support in Anaconda 2021-02-22 00:41:40 UTC

Internal Links: 239024

Description Alexander Todorov 2007-04-18 13:07:30 UTC
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Alexander Todorov 2007-04-18 13:11:04 UTC
Created attachment 152900 [details]
New option in pykickstart/parser.py

I hit enter accidently. Sorry for that.

New option "dogtail <url>" added to kickstart.
This is needed in order to support dogtail scripts in anaconda.

Comment 2 Alexander Todorov 2007-04-18 13:14:29 UTC
The rpm version is 0.43-1.el5.
The related Dogtail+Ananconda bug is: 
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=172891

Comment 3 Alexander Todorov 2007-04-18 14:34:40 UTC
Created attachment 152909 [details]
patch to pykicstart/data.py

added self.dogtail = "" in __init__

Comment 4 Chris Lumens 2007-04-20 17:30:36 UTC
We won't be able to put a change of this complexity into RHEL5 (since it needs
all the stuff in anaconda as well to really make sense) so I'm going to move
this to the Fedora product.  The pykickstart in Fedora is pretty different so
it'll take a bit of modification to make the patch work, but shouldn't be too
difficult.

I'll target this dogtail work for the first thing on F8.

Comment 5 Alexander Todorov 2007-04-30 15:18:57 UTC
It is better to use None instead of "" in
   self.dogtail = ""

Please consider this when modifying patch to work with Fedora.

Comment 6 Alexander Todorov 2007-05-03 11:55:19 UTC
Created attachment 154020 [details]
Classes to handle dogtail command in commands/*.py, pykickstart v1.0

Comment 7 Alexander Todorov 2007-05-03 11:55:57 UTC
Created attachment 154021 [details]
Import dogtail in commands/__init__.py, pykickstart v1.0

Comment 8 Alexander Todorov 2007-05-03 11:56:42 UTC
Created attachment 154022 [details]
Add dogtail to F7 & RHEL5 in handlers/control.py, pykickstart v1.0

Comment 9 Chris Lumens 2007-06-05 14:56:49 UTC
While we are interested in adding dogtail support to anaconda for this next
release, I don't know that it makes much sense to add it into kickstart as well.
 When you think about it, kickstart is an automated installation mechanism.  It
steps through things automatically, skipping steps that do not need to be
displayed because it already has the information those screens provide. 
Similarly, dogtail is an automated mechanism that will also steps through
anaconda screens automatically.  If kickstart is already stepping through them,
dogtail won't be able to do anything.

I think it makes more sense to provide this functionality with a command line
parameter.

Comment 10 Alexander Todorov 2007-06-05 15:22:47 UTC
(In reply to comment #9)
> Similarly, dogtail is an automated mechanism that will also steps through
> anaconda screens automatically.

Not always. Dogtail is automated GUI testing tool. Dogtail testcases may NOT
need to step over anaconda. It may perform tests independently of kickstart
(e.g. translation tests). Or trying to detect widget creation (exception message
dialog for example).

> I think it makes more sense to provide this functionality with a command line
> parameter.

There is a patch for a command line parameter:
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=154122

However kickstart is used for automated installation. Adding dogtail command may
be helpful to automate dogtail testing. AFAIK there is no way to specify command
line parameters with kickstart. Am I right?


Comment 11 Chris Lumens 2007-06-05 15:31:47 UTC
(In reply to comment #10)
> Not always. Dogtail is automated GUI testing tool. Dogtail testcases may NOT
> need to step over anaconda. It may perform tests independently of kickstart
> (e.g. translation tests). Or trying to detect widget creation (exception
message dialog for example).

Well the point here is that with kickstart, you're not going to see any of the
screens to perform tests on them.  The only way you'd get a screen would be to
remove information from the kickstart file, in which case you're now editing the
kickstart file for each run.

What I'm saying is that the automated nature of kickstart makes dogtail sort of
useless, since kickstart will skip over all the interested and complicated UIs
that we'll want to test with dogtail.

> There is a patch for a command line parameter:
> https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=154122

I'll be reviewing this in a week or two after we get through some common Fedora
bugs and RHEL5 stuff.

> However kickstart is used for automated installation. Adding dogtail command may
> be helpful to automate dogtail testing. AFAIK there is no way to specify command
> line parameters with kickstart. Am I right?

Correct, command line parameters are processed before any kickstart file is
downloaded.  I can sort of see a point to having a kickstart file containing
just the dogtail script location, but that could also just as easily be added to
the PXE boot parameters.

Comment 12 Alexander Todorov 2007-06-06 13:55:52 UTC
(In reply to comment #11)
> What I'm saying is that the automated nature of kickstart makes dogtail sort of
> useless, since kickstart will skip over all the interested and complicated UIs
> that we'll want to test with dogtail.
> 

One may use kickstart to proceed up to screen X where is a new gui widget or
something of interest. This can be tested with Dogtail and then let kickstart
finish the rest of the installation.
An example:
A ks.cfg file containing all but the root password. This will result in anaconda
jumping straigth to the root password entry screen. A dogtail testcase is
waiting for this screen and performs some check (e.g. is the translation
correct, are all strings translated, are buttons, textboxes enabled). Then it
fills the textboxes with root password and clicks next. From this step anaconda
continues with installing packages since it has all information needed.

Comment 16 Alexander Todorov 2007-06-08 10:11:59 UTC
Another thing:
AFAIK anaconda has limit of command line parameters length. It's 255 chars or
something like that. A longer dogtail url combined with other parameters can
easily go beyond that limit.


Note You need to log in before you can comment on or make changes to this bug.