-
-
Notifications
You must be signed in to change notification settings - Fork 54
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
Incompatibility with Puppet 8 (Puppet::Pops::Evaluator::DeferredValue) #167
Comments
Thanks for reporting this. Could you confirm the version of the cc @chelnak - thought you'd be interested in anything that looked like it could be a Puppet 8 regression? |
Output with trace (concat v8.0.0, which is not yet marked as compatible):
Output with trace (concat v7.4.0 - which is marked as compatible):
Seeing the trace now, I guess it might be more related to concat. |
Interesting. We will take a look. |
Uh oh, looks like I touched that line last, but I don't think that's caused the regression. To confirm, is it also broken with Puppet 8 and stdlib 7.3? |
I assume you mean concat 7.3. I downgraded stdlib and concat to:
The error seems to be the same:
|
Small reproducer (to make sure its not my env that is acting up):
|
@chelnak / @alexjfisher - I reproduced the incompatibility with only using Concat (and not nested Deferred function calls) and opened a issue for the concat module. |
Thanks. Someone will get on it asap. |
Hi @chelnak is there any update on that? Unfortunately I'm not sure how to solve it myself... |
There has been a new Concat release since this issue was opened. Are you seeing the Same issues with the latest version? |
Yes. The issue still persists with Concat 8.0.1 - I'll update my issue over there. Or should I rather go for opening a Jira-Ticket at puppetlabs (for concat) directly? Update: Filed a puppetlabs jira ticket: https://tickets.puppetlabs.com/browse/MODULES-11437 |
This is solved with concat v9.0.1 - Would it make sense to require at least this version? |
Affected Puppet, Ruby, OS and module versions/distributions
How to reproduce (e.g Puppet code you use)
What are you seeing
When executing the agent, it fails when using the Deferred function:
What behaviour did you expect instead
It should not abort. Pre-Puppet v8.0 this has been working fine
Output log
See above.
Any additional information you'd like to impart
I was able to pin the issue to the Deferred calls for register_to_file (and to_toml):
https://github.com/voxpupuli/puppet-gitlab_ci_runner/blob/master/manifests/runner.pp#L92
https://github.com/voxpupuli/puppet-gitlab_ci_runner/blob/master/manifests/runner.pp#L107
But I did not (yet) find a better way on how to handle this. At least to me, it seems that this should/must work.
I'm not sure whether it's an issue in the underlying Concat-Module or if its the way Deferred functions are used here.
The text was updated successfully, but these errors were encountered: