Bug 1474413

also this message should be change:

[ INFO  ] Default OVN provider added to database

To 

[ INFO  ] Adding default OVN provider to database

Stage should inform about what is happening not about what happend as others.

Moved to engine for now.

'Configure Image I/O Proxy on this host?' is in ovirt-imageio.

You might ask doc people for a thorough review.

Please note that for proper tracking you probably need to clone this bug to ovirt-imageio instead of pushing patches to both projects with this single bug. Perhaps it's less important for patches to master (next version, currently 4.2) compared to stable branches.

Removed ovirt-imageio patch tracker.

Bug has been cloned and the patch can be tracked here:
https://bugzilla.redhat.com/show_bug.cgi?id=1479704

space was fixed for OVN, however no question mark in the end for consistency with other options.

Actual result:
Configure ovirt-provider-ovn (Yes, No) [Yes]: 

Expected result:
Configure ovirt-provider-ovn? (Yes, No) [Yes]:

tested in:
ovirt-engine-setup-plugin-vmconsole-proxy-helper-4.2.0-0.0.master.20171006152719.gitbc9a33d.el7.centos.noarch
ovirt-engine-setup-plugin-ovirt-engine-common-4.2.0-0.0.master.20171006152719.gitbc9a33d.el7.centos.noarch
ovirt-engine-setup-plugin-websocket-proxy-4.2.0-0.0.master.20171006152719.gitbc9a33d.el7.centos.noarch
ovirt-engine-setup-plugin-ovirt-engine-4.2.0-0.0.master.20171006152719.gitbc9a33d.el7.centos.noarch

However found few more of 2 types:
With ?:
          Configure Image I/O Proxy on this host? (Yes, No) [Yes]: 
          Do you want Setup to configure the firewall? (Yes, No) [Yes]: 
          Where is the DWH database located? (Local, Remote) [Local]: 
          Would you like Setup to automatically configure postgresql and create DWH database, or prefer to perform that manually? (Automatic, Manual) [Automatic]: 
          Where is the Engine database located? (Local, Remote) [Local]: 
          Would you like Setup to automatically configure postgresql and create Engine database, or prefer to perform that manually? (Automatic, Manual) [Automatic]: 
          Use weak password? (Yes, No) [No]: yes
          Do you wish to set the application as the default page of the web server? (Yes, No) [Yes]: 
          Do you wish Setup to configure that, or prefer to perform that manually? (Automatic, Manual) [Automatic]: 


And without Without ?:
          Configure Data Warehouse on this host (Yes, No) [Yes]: 
          Configure VM Console Proxy on this host (Yes, No) [Yes]:
          Configure Engine on this host (Yes, No) [Yes]: 
          Configure WebSocket Proxy on this host (Yes, No) [Yes]:          
          Configure ovirt-provider-ovn (Yes, No) [Yes]:
          Use default credentials (admin@internal) for ovirt-provider-ovn (Yes, No) [Yes]: 

Thus if we wanna keep it without '?' the fix is alright, but we should align all the others as well, leaving the bug open for now.
Can we choose one of the format and use it consistently?

Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

In ovirt-engine-setup-4.2.2.1-0.1.el7.noarch with ovirt-imageio-common-1.2.1-0.el7ev.noarch is still '?'
Configure Image I/O Proxy on this host? (Yes, No) [Yes]:

But in the patch I can see the fix. So how is the status, should this be also fixed in this BZ?

Lucie,

I checked with :
ovirt-imageio-common-1.2.1-0.201802191420.git748bd64.el7.centos.noarch
ovirt-engine-setup-4.2.2.3-0.0.master.20180304080652.git60ddba2.el7.centos.noarch

and the question looks like this:

'Configure Image I/O Proxy on this host (Yes, No) [Yes]:'

QA has not 4.2.2.3 build yet, it is not fixed in ovirt-engine-4.2.2.2-0.1.el7.noarch with ovirt-imageio-common-1.2.1-0.el7ev.noarch. Please move it to ON_QA when we have the proper build.

(In reply to Lucie Leistnerova from comment #9)
> QA has not 4.2.2.3 build yet, it is not fixed in
> ovirt-engine-4.2.2.2-0.1.el7.noarch with
> ovirt-imageio-common-1.2.1-0.el7ev.noarch. Please move it to ON_QA when we
> have the proper build.

Proper build is available in http://resources.ovirt.org/pub/ovirt-4.2-pre; you can enable the repo with "yum install http://resources.ovirt.org/pub/yum-repo/ovirt-release42-pre.rpm".

(In reply to Lucie Leistnerova from comment #7)
> In ovirt-engine-setup-4.2.2.1-0.1.el7.noarch with
> ovirt-imageio-common-1.2.1-0.el7ev.noarch is still '?'
> Configure Image I/O Proxy on this host? (Yes, No) [Yes]:
> 
> But in the patch I can see the fix. So how is the status, should this be
> also fixed in this BZ?


Note that the above mentioned question doesn't come from ovirt-engine, it comes from ovirt-imageio which has not been released yet:

* 748bd64 - (HEAD -> master, origin/master, origin/HEAD) packaging: setup: Fix inconsistent options filling (4 settimane fa) <Ido Rosenzwig>
* b0a5ad0 - proxy: tests: add diff to isort tests (5 settimane fa) <Sandro Bonazzola>
* 7bb2894 - proxy: remove Content-Length from GET tests (7 settimane fa) <Daniel Erez>
* d6fd251 - daemon: catch PartialContent exception on upload (7 settimane fa) <Idan Shaby>
* 59277f8 - build: Fix make s/rpm on Fedora >= 27 (8 settimane fa) <Nir Soffer>
* e3d9c37 - (tag: v1.2.1) proxy: revert log commenting (9 settimane fa) <Daniel Erez>

Daniel, Nir, can you please issue a release ASAP?

Cloning this bug to ovirt-imageio for making it visible.

Moving to ON_QA for the engine part, the imageio part should be handled in bug #1557439

Questions in engine part are correct.

verified in ovirt-engine-setup-4.2.2.4-0.1.el7.noarch

(In reply to Sandro Bonazzola from comment #10)
> (In reply to Lucie Leistnerova from comment #9)
> > QA has not 4.2.2.3 build yet, it is not fixed in
> > ovirt-engine-4.2.2.2-0.1.el7.noarch with
> > ovirt-imageio-common-1.2.1-0.el7ev.noarch. Please move it to ON_QA when we
> > have the proper build.
> 
> Proper build is available in http://resources.ovirt.org/pub/ovirt-4.2-pre;
> you can enable the repo with "yum install
> http://resources.ovirt.org/pub/yum-repo/ovirt-release42-pre.rpm".
> 
> (In reply to Lucie Leistnerova from comment #7)
> > In ovirt-engine-setup-4.2.2.1-0.1.el7.noarch with
> > ovirt-imageio-common-1.2.1-0.el7ev.noarch is still '?'
> > Configure Image I/O Proxy on this host? (Yes, No) [Yes]:
> > 
> > But in the patch I can see the fix. So how is the status, should this be
> > also fixed in this BZ?
> 
> 
> Note that the above mentioned question doesn't come from ovirt-engine, it
> comes from ovirt-imageio which has not been released yet:
> 
> * 748bd64 - (HEAD -> master, origin/master, origin/HEAD) packaging: setup:
> Fix inconsistent options filling (4 settimane fa) <Ido Rosenzwig>
> * b0a5ad0 - proxy: tests: add diff to isort tests (5 settimane fa) <Sandro
> Bonazzola>
> * 7bb2894 - proxy: remove Content-Length from GET tests (7 settimane fa)
> <Daniel Erez>
> * d6fd251 - daemon: catch PartialContent exception on upload (7 settimane
> fa) <Idan Shaby>
> * 59277f8 - build: Fix make s/rpm on Fedora >= 27 (8 settimane fa) <Nir
> Soffer>
> * e3d9c37 - (tag: v1.2.1) proxy: revert log commenting (9 settimane fa)
> <Daniel Erez>
> 
> Daniel, Nir, can you please issue a release ASAP?

Sure, released imageio 1.2.2

This bugzilla is included in oVirt 4.2.2 release, published on March 28th 2018.

Since the problem described in this bug report should be
resolved in oVirt 4.2.2 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.