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

Replace validate_* with data types #350

Merged
merged 16 commits into from
Apr 19, 2018

Conversation

baurmatt
Copy link
Contributor

@baurmatt baurmatt commented Aug 4, 2017

To drastically improve the performance of this module, all validate_* functions get replaced in this PR with data types. Therefor this PR also drops support for Puppet 3!

Fixes #345

@baurmatt baurmatt force-pushed the fix/performance-issues branch from 92e046c to 5b5a5df Compare August 4, 2017 09:59
@baurmatt baurmatt force-pushed the fix/performance-issues branch 2 times, most recently from a4effe1 to fb2c28c Compare August 15, 2017 12:11
@lazyfrosch
Copy link
Contributor

I'm friendly on removing Puppet 3 support for puppet-icinga2.

@lbetz @bobapple Can I get your opinion on that.

Strategy:

  • Creating a puppet3 branch, which won't be maintained, but kept as reference
  • Removing travis tests for Puppet 3
  • Updating metadata accordingly
  • Doing next release as 1.4.0 - upstream deprecation shouldn't be our breaking release...

@lazyfrosch
Copy link
Contributor

Let's discuss this in #362 - I would like to merge this PR here afterwards

@lbetz lbetz added this to the v2.0.0 milestone Sep 23, 2017
@baurmatt baurmatt force-pushed the fix/performance-issues branch 6 times, most recently from 27e320f to 398fc2e Compare November 27, 2017 16:01
Due to killed tests in Travis a lot of builds fail. This commit
lower the parallel tests to 4 PARALLEL_TEST_PROCESSORS
@baurmatt baurmatt force-pushed the fix/performance-issues branch from 398fc2e to e7542ca Compare April 18, 2018 08:12
@bobapple bobapple changed the base branch from master to v2.x April 19, 2018 12:31
@bobapple bobapple merged commit e7542ca into voxpupuli:v2.x Apr 19, 2018
@baurmatt baurmatt deleted the fix/performance-issues branch April 19, 2018 13:42
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

Successfully merging this pull request may close these issues.

5 participants