From ad0b6a773aee770b6f596fec8a629a9ccd5214e3 Mon Sep 17 00:00:00 2001
From: Takashi Kajinami <tkajinam@redhat.com>
Date: Tue, 26 Jul 2022 11:15:47 +0900
Subject: [PATCH] Get rid of noop service provider

We no longer need as we use noop_resources generally to disable service
resources.

Depends-on: https://review.opendev.org/838964
Change-Id: I757450bd24433364dca7879da75074ca4f816212
---
 lib/puppet/provider/service/noop.rb | 17 -----------------
 manifests/profile/base/apache.pp    |  1 -
 2 files changed, 18 deletions(-)
 delete mode 100644 lib/puppet/provider/service/noop.rb

diff --git a/lib/puppet/provider/service/noop.rb b/lib/puppet/provider/service/noop.rb
deleted file mode 100644
index a1e9807b4..000000000
--- a/lib/puppet/provider/service/noop.rb
+++ /dev/null
@@ -1,17 +0,0 @@
-Puppet::Type.type(:service).provide :noop, :parent => :systemd do
-  def startcmd
-    [ "/bin/true" ]
-  end
-
-  def stopcmd
-    [ "/bin/true" ]
-  end
-
-  def restartcmd
-    [ "/bin/true" ]
-  end
-
-  def statuscmd
-    [ "/bin/true" ]
-  end
-end
diff --git a/manifests/profile/base/apache.pp b/manifests/profile/base/apache.pp
index f07d1a05e..30e10355a 100644
--- a/manifests/profile/base/apache.pp
+++ b/manifests/profile/base/apache.pp
@@ -40,7 +40,6 @@ class tripleo::profile::base::apache(
   class { 'apache':
     mpm_module => $mpm_module,
   }
-  Service <| title == 'httpd' |> { provider => 'noop' }
 
   include apache::mod::status
   include apache::mod::ssl