Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

You cannot collect exported resources without storeconfigs being set #76

Open
schoekek opened this issue Jul 17, 2015 · 10 comments
Open

Comments

@schoekek
Copy link

Hi,

i get the follow issue on puppet master:

Jul 17 11:28:03 sev34 puppet-master[5815]: You cannot collect exported resources without storeconfigs being set; the collection will be ignored on line 159 in file /etc/puppet/environments/production/modules/keepalived/manifests/lvs/virtual_server.pp

The collect export is disable in manifest:

keepalived::lvs::virtual_server {
   collect_exported    => false,
}

I dont use storeconfig in puppet.conf

Regards Karsten

@arioch
Copy link
Contributor

arioch commented Aug 4, 2015

Could you past the full (redacted if needed) manifest?
If you've set collect_exported to false it won't collect anything at all.

@arioch
Copy link
Contributor

arioch commented Aug 4, 2015

Could you also paste what happens if you change these lines from:

if $collect_exported {
  Keepalived::Lvs::Real_server <<| virtual_server == $name |>>
}

to:

if $collect_exported {
  fail "DEBUG _${collect_exported}_"
  Keepalived::Lvs::Real_server <<| virtual_server == $name |>>
}

@schoekek
Copy link
Author

schoekek commented Aug 6, 2015

i change the line and no change of error message?
In with file write the fail DEBUG message from your code change?
Here is the manifest:

            class { '::keepalived' : }
            keepalived::vrrp::instance { 'lb6':
                interface           => 'eth0',
                state               => 'MASTER',
                virtual_router_id   => '61',
                priority            => '101',
                virtual_ipaddress   => '10.133.13.108/28',
            }
            keepalived::lvs::virtual_server { 'fvlb20_ldap':
                ip_address          => '10.133.13.108',
                port                => '389',
                lb_algo             => 'wlc',
                lb_kind             => 'DR',
                collect_exported    => false,
            }
            keepalived::lvs::real_server { 'fvs81_ldap':
                virtual_server  => 'fvlb20_ldap',
                ip_address      => '10.133.13.107',
                port            => '389',
                options         => {
                    'MISC_CHECK' => {
                        misc_path    => '"/usr/local/bin/check_ldap 10.133.13.107"',
                        misc_timeout => '5',
                        misc_dynamic => '',
                    },
                },
            }
            keepalived::lvs::real_server { 'fvs82_ldap':
                virtual_server  => 'fvlb20_ldap',
                ip_address      => '10.133.13.109',
                port            => '389',
                options         => {
                    'MISC_CHECK' => {
                        misc_path    => '"/usr/local/bin/check_ldap 10.133.13.109"',
                        misc_timeout => '5',
                        misc_dynamic => '',
                    },
                },
            }

@arioch
Copy link
Contributor

arioch commented Aug 6, 2015

These lines.

Your manifest looks ok to me.

@schoekek
Copy link
Author

schoekek commented Aug 6, 2015

i change this lines, but no effect.

@arioch
Copy link
Contributor

arioch commented Aug 6, 2015

Which version of puppet are you running? 4.x perhaps?

@schoekek
Copy link
Author

schoekek commented Aug 6, 2015

No, on master 3.7.2 (Denian jessie version) on Client 3.6.2 (puppet CE version on wheezy).

@arioch
Copy link
Contributor

arioch commented Aug 6, 2015

It's weird because if $collect_exported {} prevents the only stored config related snippet in the entire module from being loaded. If it doesn't error out it doesn't reach the inside of said if $collect_exported {}. Which is a good thing.

Any chance you have a vagrant box to replicate this?

@schoekek
Copy link
Author

schoekek commented Aug 6, 2015

sorry we don't use vagrant, i can test it on other maschine.
pls have a little bit time, i go tomorro on vacation ;-)

@kdole
Copy link

kdole commented Dec 10, 2016

@arioch Puppet unfortunately complains about exported/collected resources during compilation even if they're not invoked. One solution is to put them in a separate module that is only included if storeconfigs is available. For example: saz/puppet-ssh#25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants