ungleich-gitea

ungleich-gitea commented on issue ungleich-public/cdist#145

2021-11-20 15:43:00 +00:00

Add support for dnf

mentioned in issue #799

ungleich-gitea commented on issue ungleich-public/cdist#144

2021-11-20 15:42:59 +00:00

Documentation of (global) explorers

closed

ungleich-gitea commented on issue ungleich-public/cdist#143

2021-11-20 15:42:57 +00:00

Onchange command in __config_file is not executed after state change

Created by: darko-poljak

@lubo Can you perhaps fix this bug? Do you have non-working case (init manifest sample) with expected result?

ungleich-gitea commented on issue ungleich-public/cdist#143

2021-11-20 15:42:56 +00:00

Onchange command in __config_file is not executed after state change

Created by: lubo

@darko-poljak I might do that, I've opened this issue mainly as a reminder for myself. This is my use case:

I have a nginx site configuration file in…

ungleich-gitea commented on issue ungleich-public/cdist#143

2021-11-20 15:42:55 +00:00

Onchange command in __config_file is not executed after state change

Created by: darko-poljak

@lubo Did you try to write full path to nginx in "--onchange" argument? Just in case.

ungleich-gitea commented on issue ungleich-public/cdist#143

2021-11-20 15:42:53 +00:00

Onchange command in __config_file is not executed after state change

Created by: darko-poljak

@lubo Ah, I see in type's source that in absent case onchange is not executed. This needs to be fixed.

ungleich-gitea commented on issue ungleich-public/cdist#141

2021-11-20 15:42:52 +00:00

Option argument cannot begin with an hyphen

Created by: darko-poljak

@uqam-fob Try this instead:

__line /tmp/xx --regex='-x' --line 'x'

The above that fails is a known failure/bug with argparse module which is used by…

ungleich-gitea commented on issue ungleich-public/cdist#141

2021-11-20 15:42:50 +00:00

Option argument cannot begin with an hyphen

Created by: antifob

Thanks, @darko-poljak, for the pointer. it still fails so I PR'ed a fix.

For those interested: upstream argparse issue at https://bugs.python.org/issue9334

edit: typo

ungleich-gitea commented on issue ungleich-public/cdist#140

2021-11-20 15:42:49 +00:00

No output for manifest or remotes

Created by: darko-poljak

@uqam-fob The reason for implementing output-streams: important information was lost during a config run, hidden in all the other output. We now store all that, including…

ungleich-gitea commented on issue ungleich-public/cdist#140

2021-11-20 15:42:48 +00:00

No output for manifest or remotes

Created by: antifob

Thanks for the information.

When I wrote "remote scripts", I was mentioning "remote-copy" and "remote-exec" scripts; as this patch affects more than the scripts you…

ungleich-gitea commented on issue ungleich-public/cdist#140

2021-11-20 15:42:46 +00:00

No output for manifest or remotes

Created by: asteven

I understand your problem: it used to work, now it doesn't. Without you changing anything yourself. That sucks from a users point of view.

I implemented the output stream…

ungleich-gitea commented on issue ungleich-public/cdist#140

2021-11-20 15:42:45 +00:00

No output for manifest or remotes

Created by: darko-poljak

@uqam-fob @asteven Adding option to turn this off shouldn't be hard. I will take this task and this new command line/config option. If you agree, by default, saving output…

ungleich-gitea commented on issue ungleich-public/cdist#140

2021-11-20 15:42:44 +00:00

No output for manifest or remotes

Created by: antifob

@asteven Yes, I understand the problem and why it might be desired. Overall, I think it is a good solution. I just don't see why it should be forced on users.

@darko-poljak…

ungleich-gitea commented on issue ungleich-public/cdist#140

2021-11-20 15:42:42 +00:00

No output for manifest or remotes

Created by: tom-ee

I'd second adding a switch to disable the "save output streams". In particular the stderr/*- and stdout/*-files created by the new features are only available after the…

ungleich-gitea commented on issue ungleich-public/cdist#140

2021-11-20 15:42:41 +00:00

No output for manifest or remotes

Created by: darko-poljak

@uqam-fob @asteven @tom-ee https://github.com/darko-poljak/cdist/tree/output_streams_switch Can you clone my repo and test this branch? If you run cdist config as usual…

ungleich-gitea commented on issue ungleich-public/cdist#140

2021-11-20 15:42:40 +00:00

No output for manifest or remotes

Created by: antifob

$ ./cdist/bin/cdist config -S --remote-copy ./remote/copy --remote-exec ./remote/exec localhost
+ ssh localhost sh -c "rm -rf /var/lib/cdist"

👍

ungleich-gitea commented on issue ungleich-public/cdist#139

2021-11-20 15:42:38 +00:00

__package_apt ignores --version parameter

Created by: darko-poljak

@thriqon In __package_apt version parameter is not yet supported. I will implement this support and you can test it.

ungleich-gitea commented on issue ungleich-public/cdist#139

2021-11-20 15:42:37 +00:00

__package_apt ignores --version parameter

Created by: darko-poljak

@thriqon I added support for --version to __package_apt. Can you clone my repo and checkout https://github.com/darko-poljak/cdist/tree/package_apt_version and test…

ungleich-gitea commented on issue ungleich-public/cdist#139

2021-11-20 15:42:35 +00:00

__package_apt ignores --version parameter

Created by: thriqon

Looks good. Thanks for the swift implementation!

ungleich-gitea commented on issue ungleich-public/cdist#138

2021-11-20 15:42:34 +00:00

Write new __apt_source with index update and deprecate old type

Created by: tom-ee

PR #623 resolved this.