Signed-off-by: Chris Lalancette <clalance(a)redhat.com>
---
bin/aeolus-check-services | 18 +-----------------
recipes/aeolus/manifests/conductor.pp | 21 ++-------------------
spec/spec_helper.rb | 2 +-
3 files changed, 4 insertions(+), 37 deletions(-)
diff --git a/bin/aeolus-check-services b/bin/aeolus-check-services
index 8cd51e8..1964487 100755
--- a/bin/aeolus-check-services
+++ b/bin/aeolus-check-services
@@ -1,5 +1,5 @@
#!/usr/bin/ruby
-init_scripts=%w(aeolus-conductor condor deltacloud-core deltacloud-ec2-us-east-1
deltacloud-ec2-us-west-1 deltacloud-mock httpd imagefactory iwhd libvirtd mongod ntpd
postgresql qpidd)
+init_scripts=%w(aeolus-conductor deltacloud-core deltacloud-ec2-us-east-1
deltacloud-ec2-us-west-1 deltacloud-mock httpd imagefactory iwhd libvirtd mongod ntpd
postgresql qpidd)
init_scripts.each do |script|
puts "\nChecking #{script} ..."
@@ -11,19 +11,3 @@ init_scripts.each do |script|
puts " \e[1;31mFAILURE:\e[0m #{out.strip}"
end
end
-
-# Other checks
-commands = [
- {:name => 'condor_q', :command => 'condor_q'},
- {:name => 'condor_status', :command => 'condor_status'}
-]
-commands.each do |cmd|
- puts "\nChecking #{cmd[:name]} ..."
- cmd = "#{cmd[:command]}"
- out = `#{cmd}`
- if $?.to_i == 0
- puts " \e[1;32mSuccess:\e[0m #{out.strip}"
- else
- puts " \e[1;31mFAILURE:\e[0m #{out.strip}"
- end
-end
diff --git a/recipes/aeolus/manifests/conductor.pp
b/recipes/aeolus/manifests/conductor.pp
index aded6b3..36b4a8a 100644
--- a/recipes/aeolus/manifests/conductor.pp
+++ b/recipes/aeolus/manifests/conductor.pp
@@ -5,7 +5,6 @@ class aeolus::conductor inherits aeolus {
# specific versions of these two packages are needed and we need to pull the third
in
package {['aeolus-conductor',
'aeolus-conductor-daemons',
- 'condor',
'aeolus-all']:
ensure => 'installed',
provider => $package_provider }
@@ -19,22 +18,6 @@ class aeolus::conductor inherits aeolus {
### Setup selinux for deltacloud
selinux::mode{"permissive":}
- ### Start the aeolus services
- file {"/etc/condor/config.d/10deltacloud.config":
- source => "puppet:///modules/aeolus/condor_config.local",
- require => Package['aeolus-conductor-daemons', 'condor'] }
- # condor requires an explicit non-localhost hostname
- # TODO we can also kill the configure sequence here instead
- exec{"/bin/echo 'hostname/domain should be explicitly set and should not be
localhost.localdomain'":
- logoutput => true,
- onlyif => "/usr/bin/test `/bin/hostname` =
'localhost.localdomain'"
- }
- service { ['condor']:
- ensure => 'running',
- enable => true,
- hasstatus => true,
- require => File['/etc/condor/config.d/10deltacloud.config'] }
-
### Setup apache for deltacloud
include apache
if $enable_https {
@@ -50,7 +33,7 @@ class aeolus::conductor inherits aeolus {
hasstatus => true,
require => [Package['aeolus-conductor-daemons'],
Rails::Migrate::Db[migrate_aeolus_database],
- Service['condor', 'httpd'],
+ Service['httpd'],
Apache::Site[aeolus-conductor], Exec[reload-apache]] }
### Initialize and start the aeolus database
@@ -130,7 +113,7 @@ class aeolus::conductor::disabled {
}
### Stop the aeolus services
- service { ['condor', 'httpd']:
+ service { ['httpd']:
ensure => 'stopped',
enable => false,
require => Service['aeolus-conductor',
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index 4332150..adf9aa8 100644
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -17,7 +17,7 @@ AEOLUS_DEPENDENCY_PACKAGES = ['rubygem-aws', 'curl',
'java-1.6.0-openjdk', 'open
'appliance-tools', 'livecd-tools',
'python-imgcreate']
# TODO want to include httpd, qpidd here as well but that requires elevated permissions
-AEOLUS_DEPENDENCY_SERVICES = ['mongod', 'condor', 'sshd',
'postgresql']
+AEOLUS_DEPENDENCY_SERVICES = ['mongod', 'sshd', 'postgresql']
IWHD_URI='http://localhost:9090/'
--
1.7.4.4