Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add extra parameters variable #530

Merged
merged 1 commit into from
Feb 24, 2020
Merged

Conversation

rshad
Copy link
Contributor

@rshad rshad commented Feb 24, 2020

Hi all!

We added a new variable script_extra_params as an empty string by default, which will be used to include the extra parameters or optional ones in Ansible tasks responsible for running the action scripts for FIM testing.

script_extra_params: ""

script: "modify_files.py -i {{ files_generated_output_path }} -o {{ files_modified_output_path }} {{ script_extra_params }}"

script: "generate_files.py -c {{ files_config_linux_destination_path }} -o {{ files_generated_output_path }} {{ script_extra_params }}"

Same changes for Windows related tasks.

Kr,

Rshad

@rshad rshad requested review from xr09 and jm404 and removed request for xr09 February 24, 2020 16:20
@rshad rshad self-assigned this Feb 24, 2020
@rshad rshad force-pushed the changes-add-extra-vars branch from 18032fd to 9f04c08 Compare February 24, 2020 16:28
Copy link
Contributor

@jm404 jm404 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jm404 jm404 merged commit 5ceed36 into fim-system-tests Feb 24, 2020
@manuasir manuasir deleted the changes-add-extra-vars branch February 24, 2020 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants