[qe-core] Unable to clone Git repository for wheels
[2022-10-06T02:01:46.761898+02:00] [info] ::: OpenQA::Isotovideo::Utils::checkout_git_repo_and_branch: Cloning git URL 'https://github.com/Zaoliang/functional_wheel' [2022-10-06T02:01:52.973889+02:00] [debug] Cloning into 'functional_wheel'... fatal: unable to access 'https://github.com/Zaoliang/functional_wheel/': OpenSSL SSL_connect: Connection reset by peer in connection to github.com:443
Regardless of the cause of the issue, I am against using personal repos for test. What happens if owner changes team or company or simply on vacation?
It's great for trying things out but when using on production, it should go into company owned github accounts, and squad owned repos.
- Category set to Infrastructure
Wheels should not cause failures of tests which do not use them, such as the kernel jobs I've linked above. External repository handling needs to be redesigned, preferably to only load wheels when the job explicitly asks for it.
- Project changed from openQA Tests to openQA Project
- Description updated (diff)
- Category deleted (
- Status changed from New to Workable
- Priority changed from High to Urgent
I think it's better to move the content from the wheels to the test repository, the experiment works and is something we can rely on, but only once #117673 is solved
So having the
save_ulog moved back into
Utils::Logging is the correct fix for the time being, and eventually we'll look again for other solutions.
We also need to watch the AC, when comming up with things like this, specially
It's great for trying things out but when using on production, it should go into company owned github accounts
I would leave squad owned repos out of the conversation, as these can also change in the future.
External repository handling needs to be redesigned
Indeed, also our approach to using these too.
#10 Updated by szarate about 2 months ago
#12 Updated by zluo about 2 months ago
So this wheel PR got reverted.
#14 Updated by zluo about 2 months ago
after discussion with szarate, we don't use external wheel repository from now. I can set it as resolved, however I fixed a minor issue in Logging.pm: