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

Support Puppet 5 and test it in travis #330

Merged
merged 4 commits into from
Aug 28, 2017
Merged

Support Puppet 5 and test it in travis #330

merged 4 commits into from
Aug 28, 2017

Conversation

lazyfrosch
Copy link
Contributor

@lazyfrosch lazyfrosch commented Jun 29, 2017

  • Update dependencies in metadata
  • Fix travis issues
  • Fix minor spec issues appearing with Puppet 5 (dependency tests)

fixes #327
resolves #233
fixes #351

@lazyfrosch
Copy link
Contributor Author

Not that easy as it seems, not sure what the problem is with the metadata...

@lazyfrosch lazyfrosch self-assigned this Jun 29, 2017
@lazyfrosch lazyfrosch mentioned this pull request Jun 29, 2017
@lazyfrosch
Copy link
Contributor Author

Puppet 5 doesn't work with forge_modules in fixtures currently: https://tickets.puppetlabs.com/browse/PUP-7738

Travis seems to be killing some procs when we start too much.

Also see theforeman/puppet-foreman_proxy#359
@lazyfrosch
Copy link
Contributor Author

forge_modules have been fixed in Puppet 5.0.1

There seems to be a problem with too much parallel spec processes, also see theforeman/puppet-foreman_proxy#359

@lazyfrosch
Copy link
Contributor Author

There still seems to be problems with concat on Puppet 5:

      # Puppet::Parser::Compiler::CatalogValidationError:
      #   Could not find resource 'Concat[/etc/icinga2/features-available/foo.conf]' in parameter 'require' at /home/travis/build/Icinga/puppet-icinga2/spec/fixtures/modules/icinga2/manifests/feature.pp:37
      #   ./vendor/bundle/ruby/2.4.0/gems/puppet-5.0.1/lib/puppet/parser/compiler/catalog_validator/relationship_validator.rb:37:in `block in validate_relationship'
  19) icinga2::feature ubuntu-16.04-x86_64 with ensure => present, feature => foo should contain File[/etc/icinga2/features-enabled/foo.conf] that notifies Class[icinga2::service]
      Failure/Error:
        should contain_file('/etc/icinga2/features-enabled/foo.conf').with({
          'ensure' => 'link',
        }).that_notifies('Class[icinga2::service]')
      
      Puppet::Error:
        Could not find resource 'Concat[/etc/icinga2/features-available/foo.conf]' in parameter 'require' at /home/travis/build/Icinga/puppet-icinga2/spec/fixtures/modules/icinga2/manifests/feature.pp:37 on node testing-docker-4213a8fa-53b7-4e40-b7ea-72851bc829a5.ec2.internal

@lazyfrosch
Copy link
Contributor Author

@lazyfrosch
Copy link
Contributor Author

Working on that issue again...

@lazyfrosch lazyfrosch added this to the v1.3.1 milestone Aug 28, 2017
@lazyfrosch lazyfrosch changed the title WIP: Support Puppet 5 and test it in travis Support Puppet 5 and test it in travis Aug 28, 2017
@lazyfrosch lazyfrosch merged commit ab86950 into master Aug 28, 2017
@lazyfrosch lazyfrosch deleted the testing/puppet-5 branch August 28, 2017 13:49
@cristianbalan-mf
Copy link

Are you working on supporting Puppet 5 but not 4? (#363 + #351)

@lbetz
Copy link
Contributor

lbetz commented Sep 29, 2017

v2.0.0 will support puppet 4 and 5. The difference between 4 and 5 are very less.

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

Successfully merging this pull request may close these issues.

Review and update puppetlabs-concat dependency puppet 5
3 participants