Node add fix

[rajan-mis]

Signed-off-by: Rajan Mishra [email protected]

[acch]

Why add a separate set_fact task? Can't we simply define the new var (scale_vars_update) within the existing assert task - or is it used anywhere else?

    - name: update | Check if any running node needs to be updated
      vars:
        msg: |-
          ...
        scale_vars_update: ...
      assert:
        that:
          - "{{ 'True' not in scale_vars_update }}"
        msg: "{{ msg.split('\n') }}"

[rajan-mis]

Why add a separate set_fact task? Can't we simply define the new var (scale_vars_update) within the existing assert task - or is it used anywhere else?

    - name: update | Check if any running node needs to be updated
      vars:
        msg: |-
          ...
        scale_vars_update: ...
      assert:
        that:
          - "{{ 'True' not in scale_vars_update }}"
        msg: "{{ msg.split('\n') }}"

@acch Thanks, agree i will change this in the same task.

[rajan-mis]

@acch Created Issue to incorporate this enhancement.
#665