Stop using git.sr.ht/~spc/go-log and use copy on github.com

[adamruzicka]

TODO:

• bump the ygg submodule once [0.2.z] fix: Stop using git.sr.ht/~spc/go-log and use copy on github.com RedHatInsights/yggdrasil#332 gets merged and ideally tagged

[jirihnidek]

I have some suggestion. Otherwise, it looks OK.

[jirihnidek]

This does not seems as correct change. This "old" go module should not be required indirectly. If it was required indirectly, then it would cause problems anyway. I guess that you also have to update version of yggdrasil, where git.sr.ht/~spc/go-log v0.1.1 is not required.

[Odilhao]

@ehelms should we still keep EL7 here?

[Odilhao]

For RHEL7 we need to update

https://github.com/theforeman/foreman-packaging/blob/66016f546d8c324a0049b18604bd4253ffbdf7b4/packages/client/foreman_ygg_worker/foreman_ygg_worker.spec#L46

To

%if 0%{?rhel} >= 8 || 0%{?fedora}
Requires: (yggdrasil >= 0.2 with yggdrasil < 0.5)
%else
Requires: yggdrasil >= 0.2
Requires: yggdrasil < 0.5
%endif

[adamruzicka]

I totally forgot we dropped support for el7 clients in f-packaging, that's why the spec uses only 8+ features. Guess we should follow suit and drop it from here as well?

[Odilhao]

Since it's not present on f-clients anymore, feel free to drop the EL7 packit.

[adamruzicka]

🍏

[jirihnidek]

LGTM