• Joined on 2021-11-20
ungleich-gitea commented on issue ungleich-public/cdist#317 2021-11-20 15:53:16 +00:00
Invalid remote code

Created by: telmich

Well, one could introduce a "--sed" parameter, but that would have to be repeated in other types as well. Not sure what's the cleanest solution here. Using a tmpfile is a…

ungleich-gitea commented on issue ungleich-public/cdist#317 2021-11-20 15:53:15 +00:00
Invalid remote code

Created by: jdguffey

After looking at GNU sed's documentation (http://www.gnu.org/software/sed/manual/sed.html), I see a minor but very important difference in how the "-i" flag is…

ungleich-gitea commented on issue ungleich-public/cdist#317 2021-11-20 15:53:13 +00:00
Invalid remote code

Created by: telmich

Hey Jake,

thanks for clearifying!

I guess we can change to create the backup file and rm -f it afterwards.

We could even add a parameter named "keepbackup" to the types…

ungleich-gitea commented on issue ungleich-public/cdist#317 2021-11-20 15:53:12 +00:00
Invalid remote code

Created by: pestaa

Thanks Jake for the additional info!

I agree that the backup extension should be provided. However, I don't think backing up should slow the process down. The current behavior…

ungleich-gitea commented on issue ungleich-public/cdist#316 2021-11-20 15:53:11 +00:00
csh is not compatible

Created by: jdguffey

Indeed. This is an issue that I have run into myself. To solve it, I made /bin/sh the default shell for root on all new FreeBSD deployments. I didn't really think anything of…

ungleich-gitea commented on issue ungleich-public/cdist#316 2021-11-20 15:53:09 +00:00
csh is not compatible

Created by: telmich

Hey Istvan,

Istvan Beregszaszi [Sat, Apr 14, 2012 at 07:59:36AM -0700]:

csh is the default shell on FreeBSD 9.0. Although it is supposed to be POSIX-compatible, it does…

ungleich-gitea commented on issue ungleich-public/cdist#316 2021-11-20 15:53:08 +00:00
csh is not compatible

Created by: telmich

Ha!

I've found a solution!

We can setup the variables locally (within a specific context/call) and then send it via the ssh option "-o SendEnv name"

Do you want to give it a…

ungleich-gitea commented on issue ungleich-public/cdist#316 2021-11-20 15:53:07 +00:00
csh is not compatible

Created by: telmich

cat *

ungleich-gitea commented on issue ungleich-public/cdist#316 2021-11-20 15:53:05 +00:00
csh is not compatible

Created by: telmich

Added a permanent entry in the log directory: doc/dev/logs/2012-05-31.csh-compatibilty

ungleich-gitea commented on issue ungleich-public/cdist#315 2021-11-20 15:53:04 +00:00
__rvm does not work, if rvm is installed on the system

Created by: telmich

Unsetting rvm* does the trick. Have a working version in my private branch, will push it upstream in the next days.

ungleich-gitea commented on issue ungleich-public/cdist#313 2021-11-20 15:53:02 +00:00
Replace all --param yes/no with boolean param!

Created by: jdguffey

What if, rather than forcing everything to always be true/false (which is, of course, a legitimate option), we refactor types to allow all yes/no/true/false parameters to be…

ungleich-gitea commented on issue ungleich-public/cdist#313 2021-11-20 15:53:01 +00:00
Replace all --param yes/no with boolean param!

Created by: telmich

Good morning Jake,

Jake Guffey [Wed, Sep 12, 2012 at 06:32:55PM -0700]:

What if, rather than forcing everything to always be true/false (which is, of course, a legitimate…

ungleich-gitea commented on issue ungleich-public/cdist#313 2021-11-20 15:53:00 +00:00
Replace all --param yes/no with boolean param!

Created by: jdguffey

Aha. I like this. Perhaps it could be solved in one of a couple ways (that would work alongside my previous suggestion).

  1. Implement in cdist core

Since cdist already makes…

ungleich-gitea commented on issue ungleich-public/cdist#313 2021-11-20 15:52:58 +00:00
Replace all --param yes/no with boolean param!

Created by: jdguffey

Err...Implement in types should be 2. Apparently I can't count. :/

ungleich-gitea commented on issue ungleich-public/cdist#313 2021-11-20 15:52:57 +00:00
Replace all --param yes/no with boolean param!

Created by: telmich

Last Missing: __jail (blocker for 2.1)

ungleich-gitea commented on issue ungleich-public/cdist#313 2021-11-20 15:52:56 +00:00
Replace all --param yes/no with boolean param!

Created by: telmich

Already done! yeah!

ungleich-gitea commented on issue ungleich-public/cdist#311 2021-11-20 15:52:54 +00:00
Order of execution

Created by: telmich

Hihi, there's a bug in your description - the correct order is

Currently, cdist's order of execution is: global explorers the initial manifest type explorers << these ones…

ungleich-gitea commented on issue ungleich-public/cdist#311 2021-11-20 15:52:53 +00:00
Order of execution

Created by: jdguffey

What do you mean by overwrite vs create new with regard to second explorer runs? I'm not sure I understand the distinction.

ungleich-gitea commented on issue ungleich-public/cdist#311 2021-11-20 15:52:51 +00:00
Order of execution

Created by: telmich

Assuming there are 2 explorer runs, we also have two (probably different) outputs of explorers.

We could either overwrite the output of the first run ("updating") or create…

ungleich-gitea commented on issue ungleich-public/cdist#311 2021-11-20 15:52:50 +00:00
Order of execution

Created by: telmich

Postponing this, as the 2.1 release is coming up soon and we probably need to agree on the right here.