Ferenc Erki

Results 143 comments of Ferenc Erki
trafficstars

@alip: thanks for the patience here! I think I'll rebase this on top of current default branch here to trigger a fresh round of CI tests. ~~If all's well, I...

It seems the issue templates were ignored while reporting this one, but I believe it is safe to assume this is a feature request :) > What Ansible and Puppet...

It took me a while to figure out what you might mean by that exactly, but I believe it's this: "if there's already a cronjob that is an exact match...

@krimdomu: I've done some initial review, and I'd like to provide you some feedback _before_ it gets merged :) 1. It currently fails `t/resources.t` and `t/param_lookup_resource.t` tests. I didn't take...

It looks like the provided issue templates were ignored when posting this, so it's hard to tell the intent, and determine the proper course of action. If this was intended...

Thanks for the report, @tabulon! Kudos for the amazing amount of details :+1: > [...] introduced initially by PR #1157 (the fix for #1066 ) with [48c737b](https://github.com/RexOps/Rex/commit/48c737b78f69be92d5f31c2c79fe992f83a4e75c) (ultimately merged into...

> [...] maintainer approval is required to launch CI workflow (for first time contributors to the project, as is the case here). I've modified the repo settings to be more...

Dumping some thoughts for implementation details: - What to do with conflicting macro names, e.g. when both the Rexfile and the CMDB has an entry for `environment`? Is it an...

Thanks for the report, I can reproduce it on Linux too: ```perl rex 'nonwritable_file', sub { file '/tmp/a', content => gmtime(), mode => 444; }; ``` At this point, I...

As discussed on IRC, the minimum approach for this would be to: - test for file writability in all the `Rex::Interface:Fs::*` modules _before_ having a chance to hang on an...