]> git.apps.os.sepia.ceph.com Git - teuthology.git/commitdiff
Revert "Install.py: Prevent prompts from breaking apt"
authorDan Mick <dan.mick@inktank.com>
Fri, 12 Apr 2013 17:56:14 +0000 (10:56 -0700)
committerDan Mick <dan.mick@inktank.com>
Fri, 12 Apr 2013 17:57:41 +0000 (10:57 -0700)
This reverts commit 5995ae7e78dd19f4036f891db9db9fec97d6eab5.

With the changes to ceph-qa-chef and the teuthology kernel task,
we're no longer touching packaged file /etc/grub.d/10_linux, which
was the reason for this apt forcing.  Remove so that we find other
package problems that might be masked by this; we can always
put it back if there are such problems until we can fix those as well.

Signed-off-by: Dan Mick <dan.mick@inktank.com>
teuthology/task/install.py

index 4255bb09ca92853cf7b5e02e886013168ef7e288..aa98fc2be77d5ec6af8c558fe14e0fcb81fc8118 100644 (file)
@@ -130,8 +130,7 @@ def _update_deb_package_list_and_install(ctx, remote, debs, config):
     remote.run(
         args=[
             'sudo', 'apt-get', 'update', run.Raw('&&'),
-            'sudo', 'DEBIAN_FRONTEND=noninteractive', 'apt-get', '-y', '--force-yes',
-            '-o', run.Raw('Dpkg::Options::="--force-confdef"'), '-o', run.Raw('Dpkg::Options::="--force-confold"'),
+            'sudo', 'apt-get', '-y', '--force-yes',
             'install',
             ] + ['%s=%s' % (d, version) for d in debs],
         stdout=StringIO(),
@@ -213,8 +212,7 @@ def _remove_deb(remote, debs):
             ] + debs + [
             run.Raw(';'),
             'do',
-            'sudo', 'DEBIAN_FRONTEND=noninteractive', 'apt-get', '-y', '--force-yes',
-            '-o', run.Raw('Dpkg::Options::="--force-confdef"'), '-o', run.Raw('Dpkg::Options::="--force-confold"'), 'purge',
+            'sudo', 'apt-get', '-y', '--force-yes', 'purge',
             run.Raw('$d'),
             run.Raw('||'),
             'true',
@@ -237,8 +235,7 @@ def _remove_deb(remote, debs):
     # then let apt clean up
     remote.run(
         args=[
-            'sudo', 'DEBIAN_FRONTEND=noninteractive', 'apt-get', '-y', '--force-yes',
-            '-o', run.Raw('Dpkg::Options::="--force-confdef"'), '-o', run.Raw('Dpkg::Options::="--force-confold"'),
+            'sudo', 'apt-get', '-y', '--force-yes',
             'autoremove',
             ],
         stdout=StringIO(),
@@ -471,8 +468,7 @@ def _upgrade_ceph_packages(ctx, remote, debs, ceph_branch):
     remote.run(
         args=[
             'sudo', 'apt-get', 'update', run.Raw('&&'),
-            'sudo', 'DEBIAN_FRONTEND=noninteractive', 'apt-get', '-y', '--force-yes', 
-            '-o', run.Raw('Dpkg::Options::="--force-confdef"'), '-o', run.Raw('Dpkg::Options::="--force-confold"'),
+            'sudo', 'apt-get', '-y', '--force-yes',
             'upgrade',
             ] + ['%s=%s' % (d, version) for d in debs],
         stdout=StringIO(),