Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

livecdreboot - rename and tidy #9716 #937

Closed
wants to merge 0 commits into from
Closed

livecdreboot - rename and tidy #9716 #937

wants to merge 0 commits into from

Conversation

MalloZup
Copy link
Contributor

tested with opensuse && slle version, tested with flag Boot_to_snapshot = 1/0 -> OK

The commit is applicable also for main.pm branch SLE12, so the main.pm contains the same change.
" livecdreboot" -> reboot_after install

  • grub tests of livecdreboot are moved to first_boot
    here we have a enforced test for avoid grub timeout, since we had failure because needles matching taked to long (first_boot.pm)
  • only one question: on opensuse branch there was a test already reboot_after_install.pm. i removed this, because i saw no utilisation of this test. ( it tested only a needle)

@sysrich
Copy link
Member

sysrich commented Jan 14, 2016

Hi Dario! Thanks - I'm going to pop over to your desk in a second and we can work through this PR :)

@coolo
Copy link
Contributor

coolo commented Jan 14, 2016

I'd prefer if the waiting_for_reboot is seperated from the reboot_after_installation

@sysrich
Copy link
Member

sysrich commented Jan 14, 2016

Because I don't understand, are you suggesting we should go for 3 test modules?
waiting_for_reboot which would effectively be reboot_after_install lines 17-48? (or lines 17-52 to get the yast logs also?)
then reboot_after_installation which would be reboot_after_install lines 54-58?
then first_boot?

@coolo
Copy link
Contributor

coolo commented Jan 14, 2016

Yes, I think I'd prefer that. because 99% of reboot_after_install is the install and 1% is the reboot :)

@MalloZup MalloZup closed this Jan 14, 2016
@MalloZup
Copy link
Contributor Author

-> so we made 3 tests instead of 2 tests ?

reboot_after_install lines 17-52 -> waiting_for_reboot
reboot_after_installation 54-58 -> grub_test || reboot_after_installation. for me grub_test it's clearer
first_boot rest unchanged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants