kemra102 / puppet-cloudwatchlogs

Puppet module for configuring AWS Cloudwatch Logs on Amazon Linux, Ubuntu, Red Hat & CentOS EC2 instances.
BSD 2-Clause "Simplified" License
11 stars 36 forks source link

Updating module versions causes various failures #21

Closed flyinbutrs closed 7 years ago

flyinbutrs commented 8 years ago

These lines:

      file { '/var/awslogs/etc/config':
        ensure => 'link',
        target => '/etc/awslogs/config',
      }

result in this error when updating from a previous version of this module:

Notice: /Stage[main]/Cloudwatchlogs/File[/var/awslogs/etc/config]: Not removing directory; use 'force' to override
Notice: /Stage[main]/Cloudwatchlogs/File[/var/awslogs/etc/config]: Not removing directory; use 'force' to override
Error: Could not remove existing file
Error: /Stage[main]/Cloudwatchlogs/File[/var/awslogs/etc/config]/ensure: change from directory to link failed: Could not remove existing file

After fixing that, I then had another error:

This require and this one are only valid on amazon linux, as on other OS's, there is no Package['awslogs']. I've replaced it with a variable that's set based on OS.

20

c22 commented 8 years ago

Looks like PR #20 was merged in, @kemra102 any chance we can get an updated release on Puppet Forge?

kemra102 commented 7 years ago

Done, apologies for the significant delay.