Add dry run option #309

Closed
opened 2021-11-20 15:24:02 +00:00 by ungleich-gitea · 6 comments

Created by: telmich

Similar to patch, use -n, --dry-run.

*Created by: telmich* Similar to patch, use -n, --dry-run.
ungleich-gitea added this to the future milestone 2021-11-20 15:24:02 +00:00
ungleich-gitea added the
feature
label 2021-11-20 15:24:02 +00:00
Author
Owner

Created by: dheule

I think now it's ok to close this bug ...

*Created by: dheule* I think now it's ok to close this bug ...
Author
Owner

Created by: dheule

I have tested it (manual), seems to work.

Ok, i will try my best for unittests next week ...

Daniel

2014-02-09 18:57 GMT+01:00 Nico Schottelius notifications@github.com:

I think it's only the command line option that has been implemented - at
least I remember there are no unit tests for it (=it does not exist ;-)
and I am pretty sure I have never tested it. :-)

Just my 2 rappen from Lenzerheide,

Nico

dheule [Sat, Feb 08, 2014 at 11:45:20AM -0800]:

I think this issue is implemented via

-n, --dry-run Do not execute code

is this correct ? can we close this issue ?


Reply to this email directly or view it on GitHub:
https://github.com/telmich/cdist/issues/102#issuecomment-34553762

PGP key: 7ED9 F7D3 6B10 81D7 0EC5 5C09 D7DC C8E4 3187 7DF0


Reply to this email directly or view it on GitHubhttps://github.com/telmich/cdist/issues/102#issuecomment-34580606
.

*Created by: dheule* I have tested it (manual), seems to work. Ok, i will try my best for unittests next week ... Daniel 2014-02-09 18:57 GMT+01:00 Nico Schottelius notifications@github.com: > I think it's only the command line option that has been implemented - at > least I remember there are no unit tests for it (=it does not exist ;-) > and I am pretty sure I have never tested it. :-) > > Just my 2 rappen from Lenzerheide, > > Nico > > dheule [Sat, Feb 08, 2014 at 11:45:20AM -0800]: > > > I think this issue is implemented via > > > > -n, --dry-run Do not execute code > > > > is this correct ? can we close this issue ? > > > > --- > > > > Reply to this email directly or view it on GitHub: > > https://github.com/telmich/cdist/issues/102#issuecomment-34553762 > > ## > > PGP key: 7ED9 F7D3 6B10 81D7 0EC5 5C09 D7DC C8E4 3187 7DF0 > > — > Reply to this email directly or view it on GitHubhttps://github.com/telmich/cdist/issues/102#issuecomment-34580606 > .
Author
Owner

Created by: telmich

I think it's only the command line option that has been implemented - at
least I remember there are no unit tests for it (=it does not exist ;-)
and I am pretty sure I have never tested it. :-)

Just my 2 rappen from Lenzerheide,

Nico

dheule [Sat, Feb 08, 2014 at 11:45:20AM -0800]:

I think this issue is implemented via

-n, --dry-run Do not execute code

is this correct ? can we close this issue ?


Reply to this email directly or view it on GitHub:
https://github.com/telmich/cdist/issues/102#issuecomment-34553762

PGP key: 7ED9 F7D3 6B10 81D7 0EC5 5C09 D7DC C8E4 3187 7DF0

*Created by: telmich* I think it's only the command line option that has been implemented - at least I remember there are no unit tests for it (=it does not exist ;-) and I am pretty sure I have never tested it. :-) Just my 2 rappen from Lenzerheide, Nico dheule [Sat, Feb 08, 2014 at 11:45:20AM -0800]: > I think this issue is implemented via > > -n, --dry-run Do not execute code > > is this correct ? can we close this issue ? > > --- > > Reply to this email directly or view it on GitHub: > https://github.com/telmich/cdist/issues/102#issuecomment-34553762 ## PGP key: 7ED9 F7D3 6B10 81D7 0EC5 5C09 D7DC C8E4 3187 7DF0
Author
Owner

Created by: dheule

I think this issue is implemented via

-n, --dry-run Do not execute code

is this correct ? can we close this issue ?

*Created by: dheule* I think this issue is implemented via -n, --dry-run Do not execute code is this correct ? can we close this issue ?
Author
Owner

Created by: telmich

Began code change in resolver branch in config_install.py

*Created by: telmich* Began code change in resolver branch in config_install.py
Author
Owner

Created by: telmich

Implementation: Execute everything, but skip the code execution phase. Maybe a simple

if not dry_run:
self.code.run_code_remote() ... (and local)

*Created by: telmich* Implementation: Execute everything, but skip the code execution phase. Maybe a simple if not dry_run: self.code.run_code_remote() ... (and local)
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: ungleich-public/cdist#309
No description provided.