Update README.adoc

[aesteve-rh]

• Fix links syntax from markdown to asciidoc and remove outdated links.
• Remove some deprecated Fedora 30 parts and update prerequisites section.

Signed-off-by: Albert Esteve aesteve@redhat.com

[aesteve-rh]

@barpavel @nirs fyi, please review.

[arso]

no, it is not, mod_auth_openidc is only for RPM installation.

On Tuesday, April 12, 2022, Benny Zlotnik ***@***.***> wrote: @bennyz commented on this pull request. ________________________________ In README.adoc: > @@ -494,6 +492,9 @@ During engine setup, a certificate has to be issued

and you will be asked for a

hostname. If you want to upload and download images from administration

portal,

it has to be the name by which your machine is accessible from the

outside.

+NOTE: engine-setup might require mod_auth_openidc for keycloak, if so

you can enable

@arso is this needed for a dev env? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.<

https://ci4.googleusercontent.com/proxy/_pAhNV_ds9wbg_fzBPvbJfhIjK_xyzpb0_KP0P4NYNdzgasPGEMCngFfDL7THCW59bFoo8bwZhnhqEcr4MD4Xh09WkzbSiIgzHU_zYc7QC8D65UPLP5lwkpME2PK6nKdI8hzgVzbJ_Bt_ZXXvjLV4YwuDdXa_EsINOk9W17Zm-VzxKWk6iUVK6be_DryzuRqZXcYCoks7T_9vrauMeQlmadPxknQzqyy4Qh1YNsKOlt0jw2RYFd2=s0-d-e1-ft#https://github.com/notifications/beacon/AACMF2FQEH2CYR2QR2MUCADVEWREVA5CNFSM5THDQWEKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOHACOUJI.gif>Message ID: ***@***.***>

…

-- Artur Socha Senior Software Engineer, RHV Red Hat

[sleviim]

• line 340 - commiting > committing
• in the Install the following system components: part, Artiom got a new laptop (RHEL CSB) and had to install also the httpd package, maybe worth adding it to the list

[barpavel]

• mod_auth_openidc

PR description: no mod_auth_openidc change, can remove this comment.
On the other hand please add a sentence, that some deprecated Fedora 30 parts were removed.

[sleviim]

maybe worth adding also (optional) ovirt-provider-ovn in case of setting ovirt-ovn on setup

[barpavel]

• Install the following system components:

IMHO no need to add httpd to Prerequisites section.
I also checked my old engine VM, no httpd there.
I think httpd was just related to some package Artiom installed on his laptop that is not related to engine.

[barpavel]

LGTM.
IMHO ready to be merged :)