On Mon, Feb 1, 2021 at 12:46 PM Marcin Sobczyk <msobczyk(a)redhat.com> wrote:
On 2/1/21 11:41 AM, Marcin Sobczyk wrote:
> Hi,
>
> On 1/31/21 4:22 PM, Yedidyah Bar David wrote:
>> On Sun, Jan 17, 2021 at 2:42 PM Yedidyah Bar David <didi(a)redhat.com
>> <mailto:didi@redhat.com>> wrote:
>>> Hi all,
>>>
>>> Ran [1] for verifying [2]. Passed [3] as CUSTOM_REPOS.
>>>
>>> lago.log [4] shows that it did add it to internal_repo, but it was not
>>> actually installed later. Perhaps something changed in local repos
>>> handling or something like that?
>> Now happened again, so I spent some more time looking. I tried this
>> locally (on buri03) and it did work - passing '-s URL' to run_suite.sh
>> did install/update packages from there. In CI it does not, e.g. [1].
>>
>> I think that the problem is around here, inside global_setup.sh:
>>
>> *11:04:29* + sudo -nl firewall-cmd
>>
>> *11:04:29* ++ sudo -n firewall-cmd --query-service=ovirtlago
>>
>> *11:04:30* + [[ ! -n yes ]]
>>
>> *11:04:30* + known_hosts
>>
>> Meaning, there were no sudo permissions to run firewall-cmd,
>> so it didn't add 8585, so later (in lago.log):
>>
>> + ADDR=192.168.200.1
>> + curl_res=0
>> + curl -f
http://192.168.200.1:8585/default/el8/repodata/repomd.xml
>> <
http://192.168.200.1:8585/default/el8/repodata/repomd.xml>
>> % Total % Received % Xferd Average Speed Time Time Time
>> Current
>> Dload Upload Total Spent Left
>> Speed
>> \r 0 0 0 0 0 0 0 0 --:--:-- --:--:--
>> --:--:-- 0curl: (7) Failed to connect to 192.168.200.1 port 8585:
>> Connection refused
>> + curl_res=7
Are you sure this failed because the proxy wasn't running, and not because
it was blocked? How did this work for me on buri03, then?
>>
>> Since this seems like an infra issue, moving to infra list.
>>
>> Can someone have a look there? Thanks!
> The topic of adding and using extra repos in OST is complicated...
BTW we have an RFE for this that Harel is working on:
https://issues.redhat.com/browse/RHV-40844
Thanks :-). Commented there.
> Since HE suite is half-migrated to pytest and still depends on lago OST
> plugin, we need
> to add some legacy compatibility layer to 'conftest.py' like the one
> that still exists
> in basic suite:
>
>
https://github.com/oVirt/ovirt-system-tests/blob/ebab2fc2c1c10a36bd835f4a...
>
> Please try using custom repos with this added ^^^.
I already verified manually and merged the patch I wanted to test (to
ovirt-hosted-engine-ha).
I'll try to keep that in mind for the next time.
Best regards,
--
Didi