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

.stdout usage in task files breaks execution #19

Open
atothRedHat opened this issue Jul 11, 2019 · 2 comments
Open

.stdout usage in task files breaks execution #19

atothRedHat opened this issue Jul 11, 2019 · 2 comments

Comments

@atothRedHat
Copy link

I've run across issues with the main.yml and setup.yml files where tasks fail because the conditional or attribute value source uses the ".stdout" function call. removing the ".stdout" fixes the issues and the task executes complete as intended.

Expected Outcome: Tasks complete normally
Actual Outcome: Tasks fail

@atothRedHat atothRedHat changed the title .stdout usage in task files breaks execurtion .stdout usage in task files breaks execution Jul 11, 2019
fubarhouse added a commit that referenced this issue Jul 14, 2019
@fubarhouse
Copy link
Owner

fubarhouse commented Jul 14, 2019

@atothRedHat,

Thanks for reporting, I've made some small changes (in branch issue/19) to simplify some of the logic and removed a conditional which sets the variable causing the problem - so that it is (now) always set instead of causing an error. Honestly I'm unsure how that wasn't set but this should mitigate the edge-case.

If you're able to re-test this on your environment I would be interested in the feedback, and we can merge the changes or revisit as necessary. Please let me know! The changes can be found in branch issue/19.

@atothRedHat
Copy link
Author

nope, issue is still there. If you look at the files, the .stdout call is still made in 2 spots which breaks the run.
setup.yml:

  • name: "Rust | Set $HOME"
    set_fact:
    fubarhouse_user_dir: "{{ shell_home_dir.stdout }}"
    when: fubarhouse_user_dir is not defined

main.yml:

  • name: "Install from installer"
    include_tasks: installer.yml
    when:
    • ((build_rust_from_source|bool == false) and
      (rust_version_requirement not in rust_preinstall_version_r.stdout)) or
      (rust_update|bool == true)

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

No branches or pull requests

2 participants