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:
> 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:email@example.com>> wrote:
>>> Hi all,
>>> Ran  for verifying . Passed  as CUSTOM_REPOS.
>>> lago.log  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. .
>> 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
>> % Total % Received % Xferd Average Speed Time Time Time
>> Dload Upload Total Spent Left
>> \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:
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:
> Please try using custom repos with this added ^^^.
I already verified manually and merged the patch I wanted to test (to
I'll try to keep that in mind for the next time.