fix: changed set_fact to uri call

[HerbBoy]

Notes:
The set_fact was causing the following error:

objc[18913]: +[__NSCFConstantString initialize] may have been in progress in another thread when fork() was called.

this change corrects this whilst keeping the desired functionality.

[patrickjahns]

@HerbBoy

Unfortunately your change breaks compatibility with ansible 2.10 and 3.4 :

TASK [ansible-role-promtail : Get checksum for amd64 architecture] *************
fatal: [instance]: FAILED! => {"msg": "template error while templating string: No filter named 'split'.. String: {{ __promtail_checksums.content | split('\n') }}"}

Please see the failing tests

[HerbBoy]

@HerbBoy

Unfortunately your change breaks compatibility with ansible 2.10 and 3.4 :

TASK [ansible-role-promtail : Get checksum for amd64 architecture] *************
fatal: [instance]: FAILED! => {"msg": "template error while templating string: No filter named 'split'.. String: {{ __promtail_checksums.content | split('\n') }}"}

Please see the failing tests

Will look into alternatives. I am running the latest ver so forgot to check on whether it was compatible.

[dearlordylord]

how do we circumvent this error meanwhile?

[patrickjahns]

@Firfi

Please see upstream issue ansible/ansible#76322 regarding this. The role works as designed