Kamil,
Thanks for all you have done. I saw you have moved the anaconda.py
and analog.py from
/lib/autoqa/ to /tests/rats_install/, I am afraid we have to move it
back in future,
since other tests, e.g. anaconda_install which installs guest from
ISOs will use
those functions, too.
Disclaimer first:
I am working in parallel on ticket 255 (centralized Config class) and I needed to decouple
a few things from rats_install and rats_sanity. Some of the changes I did may not be
obvious at first, sorry for that.
About anaconda.py and analog.py, yeah, I'm not happy about the current state either.
But I needed to make those module a little mode standalone and not as much intertwined
with the rest of our library, because this is rather a test-specific library. That's
why I decoupled all those logdir/scratchdir variables etc. I moved it to rats_install/
because currently nothing else needs it. When we need to re-use it for other tests in the
future, we can then move it back to lib/autoqa or devise a better location. I intend to
think hard about this issue, because it is clearly not optimal to move every test library
into lib/autoqa that two or more tests want to re-use.
About problem 5, those unreadable characters are control sequences,
it seems python
could interpret it, have you hit any problem?
Yep, I did. We use that log for grepping for important strings. But sometimes the strings
are mangled with the unreadable characters. Have a look at this stage1.log:
http://fpaste.org/40KB/raw/
There is this sentence:
"Runing anaconda 1.25, the Fera system instler - plea wait."
It was not recognized and that's why our test failed (even though the installation
continued just fine). Fortunately it doesn't seem to happen often.
We have to dig into this issue further. I believe it a bug somewhere (doesn't have to
be our code). But bear in mind I was testing F16 (F17 doesn't work yet), it
doesn't have to apply for F17 (we will know once we get usable compose).
Even though this issue is still unresolved, I would like to propose merging current
rats_install into master, because my other work depends on it. We can fix this problem
later. What do you think, Hongqing? Is there something else you would like to polish
before I merge it?