Error on Ubuntu 12.04 and Puppet 3.0.1

[ilyavaiser]

root@host2:/var/log# puppet agent --test
Info: Retrieving plugin
Info: Loading facts in /var/lib/puppet/lib/facter/puppet_vardir.rb
Info: Loading facts in /var/lib/puppet/lib/facter/concat_basedir.rb
Info: Loading facts in /var/lib/puppet/lib/facter/facter_dot_d.rb
Info: Loading facts in /var/lib/puppet/lib/facter/root_home.rb
Info: Loading facts in /var/lib/puppet/lib/facter/pe_version.rb
Info: Caching catalog for host2.globalit.zr.ru
Info: Applying configuration version '1354656181'
Error: Could not apply complete catalog: Found 1 dependency cycle:
(Exec[concat_/etc/exports] => Concat[/etc/exports] => Service[nfs-kernel-server] => Class[Nfs::Server::Debian] => Concat[/etc/exports] => Exec[concat_/etc/exports])
Try the '--graph' option and opening the resulting '.dot' file in OmniGraffle or GraphViz
Finished catalog run in 0.12 seconds

[haraldsk]

Hey!

Could you add the puppet code from the manifest so i can see how you're calling the module?

[masteinhauser]

I'm running into this exact same issue. Below is some debug info I hope you find useful:

Puppet Manifest:

  include nfs::server
  nfs::server::export{ '/mnt/x1/cache':
    ensure  => 'mounted',
    clients => '172.25.20.0/21(rw,insecure,async,no_root_squash) localhost(rw)',
    mount   => '/mnt/cache',
    atboot  => true,
  }

No clients are currently configured, I do not have storeconfigs enabled on my Puppet Master.

Puppet Client: 2.7.11
Puppet Master: 3.0.2

[jigfox]

any news on this? I'm running into the same issue

[adepretis]

same here

[pkasprzak]

I also have the same problem (puppet client / master = 3.2.3) on Ubuntu 13.04.

A quick hack to solve it (will probably break any non debian platforms):

From dec0782e1407c47cf4d4b33817f8dba8ea258a9e Mon Sep 17 00:00:00 2001
From: Piotr Kasprzak <piotr.kasprzak@gwdg.de>
Date: Sun, 18 Aug 2013 11:51:25 +0200
Subject: [PATCH] Fix dependency loop

---
 manifests/server/configure.pp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/manifests/server/configure.pp b/manifests/server/configure.pp
index dbef395..28a40a4 100644
--- a/manifests/server/configure.pp
+++ b/manifests/server/configure.pp
@@ -1,10 +1,10 @@
 class nfs::server::configure {

   concat {'/etc/exports': 
-    require => Class["nfs::server::${nfs::server::osfamily}"]
+#    require => Class["nfs::server::${nfs::server::osfamily}"]
+    require => Package["nfs-kernel-server"]
   }

-
   concat::fragment{
     'nfs_exports_header':
       target  => '/etc/exports',
-- 
1.7.11.1

[skurylo]

Looking at configure.pp, why does concat have to require "Class["nfs::server::${nfs::server::osfamily}"]" ?

Looking in the debian and redhat classes, I don't notice anything. So I made the change:

class nfs::server::configure {

• include "nfs::server::${osfamily}"
  concat {'/etc/exports':
• require => Class["nfs::server::${nfs::server::osfamily}"]
  }

This seems to work for me.

[amette]

Yup, just as pkasprzak said, his patch works for Debian. Just nowadays you have to do it in the file manifests/server.pp

[cscd98]

Issue also exists in 13.10, trying the module for the first time

[haraldsk]

I doubt that will work on RHEL.