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

Issues with default data-dir post-#292 #307

Closed
jamtur01 opened this issue Jan 1, 2017 · 1 comment
Closed

Issues with default data-dir post-#292 #307

jamtur01 opened this issue Jan 1, 2017 · 1 comment

Comments

@jamtur01
Copy link

jamtur01 commented Jan 1, 2017

The #292 merge breaks the module if you don't use /opt/consul as the data directory.

   amazon-ebs: Notice: /Stage[main]/Consul::Install/File[/opt/consul/archives/consul-0.7.2]: Dependency File[/opt/consul/archives] has failures: true
    amazon-ebs: Error: Cannot create /opt/consul/archives; parent directory /opt/consul does not exist
    amazon-ebs: Error: /Stage[main]/Consul::Install/File[/opt/consul/archives]/ensure: change from absent to directory failed: Cannot create /opt/consul/archives; parent directory /opt/consul does not exist
    amazon-ebs: Notice: /Stage[main]/Consul::Install/Archive[/opt/consul/archives/consul-0.7.2.zip]: Dependency File[/opt/consul/archives] has failures: true
    amazon-ebs: Notice: /Stage[main]/Consul::Install/File[/opt/consul/archives/consul-0.7.2/consul]: Dependency File[/opt/consul/archives] has failures: true
    amazon-ebs: Notice: /Stage[main]/Consul::Install/File[/usr/local/bin/consul]: Dependency File[/opt/consul/archives] has failures: true
    amazon-ebs: Warning: /Stage[main]/Consul::Install/File[/opt/consul/archives/consul-0.7.2]: Skipping because of failed dependencies
    amazon-ebs: Warning: /Stage[main]/Consul::Install/Archive[/opt/consul/archives/consul-0.7.2.zip]: Skipping because of failed dependencies
    amazon-ebs: Warning: /Stage[main]/Consul::Install/File[/opt/consul/archives/consul-0.7.2/consul]: Skipping because of failed dependencies
    amazon-ebs: Warning: /Stage[main]/Consul::Install/File[/usr/local/bin/consul]: Skipping because of failed dependencies

My config hash:

consul::config_hash:
  data_dir: '/var/consul'
  datacenter: 'redacted'
  encrypt: 'redacted'
  enable_syslog: true
  rejoin_after_leave: true
  log_level: INFO
  retry_join_ec2:
    tag_key: Flag
    tag_value: consul
@solarkennedy
Copy link
Contributor

Sorry about this, I guess I merged it prematurely.
I'll make a PR to do that last part and actually use data_dir as the root.

solarkennedy added a commit that referenced this issue Jan 2, 2017
Use data_dir as a the root of the archive path. Fixes #307
spuder pushed a commit to spuder/puppet-consul that referenced this issue Feb 25, 2020
spuder pushed a commit to spuder/puppet-consul that referenced this issue Feb 25, 2020
Use data_dir as a the root of the archive path. Fixes voxpupuli#307
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

2 participants