Created by: telmich
I agree rephrasing it to "Every change to the target system SHOULD result in a message". Even though consistency is a nice thing, I would not recommend to implement it, when…
Created by: tom-ee
The rephrased version combined with a recommendation against implemention it if it's not used is the current state.
For debugging inspecting code-* is the best way for…
Created by: darko-poljak
@tom-ee Thanks for pointing this.
1f93e7291e
~/.cdist/cache/<md5 hash>
Created by: darko-poljak
@tom-ee You can change host cache directory name, see CACHE PATH PATTERN FORMAT in cdist man page. This is also mentioned in local cache overview docs chapter.
It was…
~/.cdist/cache/<md5 hash>
Created by: tom-ee
Ah, that makes perfect sense. Thank you for the explanation.
Created by: tom-ee
Might help with narrowing this down: Does the stderr make it (to stdout sic) if the output-stream-capture is disabled with -S?
When running cdist config -b -j4 -S localhost, the stderr indeed is on stdout:
This is my stderr
ERROR: [7241]: localhost: Command failed: '/bin/sh -e…
Created by: darko-poljak
@AnotherKamila Can you test? cf36aece7d
Created by: darko-poljak
My cdist clone at branch https://github.com/darko-poljak/cdist/tree/bugfix/error_info_lost_with_jobs_%23632
Created by: darko-poljak
@telmich Does this happen when using default non-direct, staged file method, too?
Created by: darko-poljak
Ah, unzip should be rewritten to unzip to temp then rm orig then mv temp to orig. It is the way unix/linux kernel and inodes work.
Created by: darko-poljak
@telmich Can you test b870b6e43a ?
Thanks!
Created by: darko-poljak
Tested with consul agent running.
Created by: matthijskooijman
It seems that this also breaks the CDIST_BETA env var. It is parsed properly, but then overwritten with the default False from the commandline here:…
Created by: darko-poljak
@matthijskooijman Hi! Thanks for analysis. I am not sure for above, if argparse supports this, I have to check. And for CDIST_BETA and beta default part, I will check and…
Created by: darko-poljak
@matthijskooijman I think that the right way would be:
$ ./bin/cdist inventory -h
usage: cdist inventory [-h] {add-host,add-tag,del-host,del-tag,list}…
Created by: darko-poljak
Fixed argparse parsers' parent references so options are now fixed/more consistent.
Created by: matthijskooijman
Looks good for the --beta option, sounds like the right way to fix this. CDIST_BETA env doesn't work yet, though.