Merge lp:~psivaa/ue-qa-kernel-tests-infrastructure/sudoers-fix-for-trusty into lp:ue-qa-kernel-tests-infrastructure

[Para Siva]

This is to fix the "sudo: no tty present and no askpass program specified" errors in trusty tests. It's not clear how the setup could be working for raring and saucy without NOPASSWD being allowed for sudo in sudoers, but somehow this is needed for trusty and fixes the issue..

[Paul Larson (pwlars)]

Strange... they must be getting it from somewhere, or coming in as root already. In fact, I thought that most of these were set up so that the script sshes in as root. In any case, maybe we should just go ahead and add raring and saucy to the list as well?
Approving, assuming you've tried this already.

review: Approve

[Para Siva (psivaa)]

Thanks. I've tried this with trusty and saucy. I'll add the the same change to include raring and saucy and then try one to confirm before merging.

[lp:~psivaa/ue-qa-kernel-tests-infrastructure/sudoers-fix-for-trusty]

344. By Para Siva on 2013-11-08

Adding NOPASSWD for sudo for raring and saucy

345. By Para Siva on 2013-11-08

Merging prev changes

[C de-Avillez (hggdh2)]

@pwlars: no, the code does not run as root. This is needed so that we will not need to use askpass or another hack to do root things.

And yes, if the sudo setup did not change, all releases from Precise onwards need this. Alternatively, you could assume that all releases nowadays need it.

review: Approve

=== modified file 'kernel-sru/jenkins-script-bare-metal-kteam.sh'
--- kernel-sru/jenkins-script-bare-metal-kteam.sh	2013-02-26 20:38:58 +0000
+++ kernel-sru/jenkins-script-bare-metal-kteam.sh	2013-11-08 15:42:31 +0000
@@ -114,9 +114,10 @@
 [ $RC -ne 0 ] && sys_exit_KO "$SYS_HOST did not answer in time, bailing out"
 #
 # make sure we can get root (the tests will need it)
+# Having to add NOPASSWD for sudo for raring, saucy and trusty tests
 #
 case $RELEASE in
-  oneiric|precise|quantal)
+  oneiric|precise|quantal|raring|saucy|trusty)
     sys_ssh ubuntu@$SYS_HOST "echo ubuntu | sudo -S sed -i 's/^%sudo.*$/%sudo ALL=(ALL:ALL) NOPASSWD: ALL/' /etc/sudoers"
     RC=$?
     ;;