On 2/1/21 12:04 PM, Yedidyah Bar David wrote:
On Mon, Feb 1, 2021 at 12:46 PM Marcin Sobczyk <[email protected]
<mailto:[email protected]>> 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
<[email protected] <mailto:[email protected]>
>> <mailto:[email protected] <mailto:[email protected]>>> 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>
>> <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?
Not sure really, but I recall situations in the past, that the repo server
created by lago OST plugin was working for me without any suite running...
(like it was never really killed).
Let's focus on the porting so we can get rid of that part and use the
new way of working:
https://github.com/oVirt/ovirt-system-tests/blob/f31b6c59349d87c9605477e25af8c837b7e0817a/run_suite.sh#L640
>>
>> 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
<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/ebab2fc2c1c10a36bd835f4a8cdb18bd71903cf8/basic-suite-master/test-scenarios/conftest.py#L31
<https://github.com/oVirt/ovirt-system-tests/blob/ebab2fc2c1c10a36bd835f4a8cdb18bd71903cf8/basic-suite-master/test-scenarios/conftest.py#L31>
>
> 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,
>
> Regards, Marcin
>
>> Best regards,
>>
>> [1]
>>
https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7726/
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7726/>
>>
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7726/
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7726/>>
>>
>>
>> On Sun, Jan 17, 2021 at 2:42 PM Yedidyah Bar David
<[email protected] <mailto:[email protected]>
>> <mailto:[email protected] <mailto:[email protected]>>> 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?
>>
>> Thanks and best regards,
>>
>> [1]
>>
https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7681/
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7681/>
>>
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7681/
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7681/>>
>>
>> [2]
https://github.com/oVirt/ovirt-ansible-collection/pull/202
<https://github.com/oVirt/ovirt-ansible-collection/pull/202>
>>
<https://github.com/oVirt/ovirt-ansible-collection/pull/202
<https://github.com/oVirt/ovirt-ansible-collection/pull/202>>
>>
>> [3]
>>
https://jenkins.ovirt.org/job/oVirt_ovirt-ansible-collection_standard-check-pr/468/artifact/check-patch.el8.x86_64/
<https://jenkins.ovirt.org/job/oVirt_ovirt-ansible-collection_standard-check-pr/468/artifact/check-patch.el8.x86_64/>
>>
<https://jenkins.ovirt.org/job/oVirt_ovirt-ansible-collection_standard-check-pr/468/artifact/check-patch.el8.x86_64/
<https://jenkins.ovirt.org/job/oVirt_ovirt-ansible-collection_standard-check-pr/468/artifact/check-patch.el8.x86_64/>>
>>
>> [4]
>>
https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7681/artifact/exported-artifacts/test_logs/he-basic-suite-master/post-012_local_maintenance_sdk_pytest.py/lago_logs/lago.log
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7681/artifact/exported-artifacts/test_logs/he-basic-suite-master/post-012_local_maintenance_sdk_pytest.py/lago_logs/lago.log>
>>
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7681/artifact/exported-artifacts/test_logs/he-basic-suite-master/post-012_local_maintenance_sdk_pytest.py/lago_logs/lago.log
<https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/7681/artifact/exported-artifacts/test_logs/he-basic-suite-master/post-012_local_maintenance_sdk_pytest.py/lago_logs/lago.log>>
>> --
>> Didi
>>
>>
>>
>> --
>> Didi
--
Didi
_______________________________________________
Infra mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Privacy Statement: https://www.ovirt.org/privacy-policy.html
oVirt Code of Conduct:
https://www.ovirt.org/community/about/community-guidelines/
List Archives:
https://lists.ovirt.org/archives/list/[email protected]/message/OGCANSXV2Z5PCSF4O7SWX2TD7O3ISEER/
