Bug 1706822
Summary: | [engine-setup] Confusing message in engine-setup about installing local DBs manually | ||
---|---|---|---|
Product: | [oVirt] ovirt-engine | Reporter: | Steve Goodman <sgoodman> |
Component: | Setup.Engine | Assignee: | Asaf Rachmani <arachman> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Guilherme Santos <gdeolive> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 4.3.0 | CC: | arachman, bugs, didi, lleistne, lveyde, stirabos |
Target Milestone: | ovirt-4.4.0 | Flags: | pm-rhel:
ovirt-4.4?
sbonazzo: planning_ack? sbonazzo: devel_ack+ lleistne: testing_ack+ |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2020-05-20 20:03:17 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | Integration | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Steve Goodman
2019-05-06 10:57:44 UTC
I agree that this text requires some refinement, I think it was never reviewed in itself. I guess the assumption is that if you say "manually", it means you should know what you do. Specific points: 1. The text "Please create database for ovirt-engine use" is hard-coded. Perhaps we should replace "ovirt-engine" here with "oVirt". The function that outputs this also gets a "name", which is 'Engine' for the engine (not localized, currently) and 'DWH' for dwh. Also, if you choose to configure cinderlib integration, you get a third set of questions for its database - there, name is 'Cinderlib'. So perhaps: Please create a database for use by oVirt or Please create a database for use by oVirt, for '{name}' (where {name} will be 'Engine', 'DWH', 'Cinderlib') or, if you want the text you suggested in comment 0, Please create the database for the {name} to use So: Please create the database for the Engine to use Please create the database for the DWH to use Please create the database for the Cinderlib to use Not sure this is very nice, but also dropping 'the' isn't that nice. Suggestions are welcome... 2. Personally I'd drop 'ATTENTION' and 'Manual action required'. Not sure why we added that. 3. Re "Make sure that database can be accessed remotely.": I don't mind that much about this, can be dropped, as long as we properly document _when_ it should be (accessed remotely) and how. The original reason, AFAIR, for adding this text, was for integration of DWH with ManageIQ (CloudForms). If you want that, you have to make the DWH database accessible for ManageIQ. Also, if there isn't anything specific to RHV about this bug, perhaps move it to oVirt. (In reply to Yedidyah Bar David from comment #1) > Specific points: > > 1. The text "Please create database for ovirt-engine use" is hard-coded. > Perhaps we should replace "ovirt-engine" here with "oVirt". That's better than what's there right now. > that outputs this also gets a "name", which is 'Engine' for the engine (not > localized, currently) and 'DWH' for dwh. Also, if you choose to configure > cinderlib integration, you get a third set of questions for its database - > there, name is 'Cinderlib'. So perhaps: > > Please create a database for use by oVirt > > or > > Please create a database for use by oVirt, for '{name}' (where {name} will > be 'Engine', 'DWH', 'Cinderlib') > > or, if you want the text you suggested in comment 0, > > Please create the database for the {name} to use > > So: > > Please create the database for the Engine to use > Please create the database for the DWH to use > Please create the database for the Cinderlib to use > > Not sure this is very nice, but also dropping 'the' isn't that nice. > Suggestions are welcome... Those are all helpful hints and would allay confusion. > 2. Personally I'd drop 'ATTENTION' and 'Manual action required'. Not sure > why we added that. I agree with you. No real value added. > 3. Re "Make sure that database can be accessed remotely.": I don't mind that > much about this, can be dropped, as long as we properly document _when_ it > should be (accessed remotely) and how. The original reason, AFAIR, for > adding this text, was for integration of DWH with ManageIQ (CloudForms). If > you want that, you have to make the DWH database accessible for ManageIQ. I'm not sure what to say here. I know what I was trying to do when I saw this message and found it confusing. I don't know in what other scenarios this might be viewed, and if it would be helpful in other scenarios. I'm in favor of providing helpful text directly in the UI (including terminal UIs like this). Maybe it would be helpful to add some context to what's there already, i.e. "If x, y, or z is true, then make sure that database can be accessed remotely." > Also, if there isn't anything specific to RHV about this bug, perhaps move > it to oVirt. I'll leave it to you guys to move this as you see fit. In response to comments in Gerrit: ------------------------------------------------------------------ Instead of "Use the following commands as an example" I suggest the following (consistent with the documentation): To create a default user: create role <user_name> with login encrypted password '<password>'; To create a database: create database <database_name> owner <user_name> template template0 encoding 'UTF8' lc_collate 'en_US.UTF-8' lc_ctype 'en_US.UTF-8'; --------------------------------------------------------------------- For me, the notation <user> is clear enough, I don't think adding "Replace <user> with the user name can hurt, but I don't think it's necessary. -------------------------------------------------------------------- Verified on: ovirt-engine-setup-4.4.0-0.13.master.el7.noarch Steps: 1. Run engine-setup 2. When prompted where DWH and Database should be located choose 'Local' 3. When prompted how DWH and Database should be configured choose 'Manual' Results: Following expected message prompted: Please create the database for the DWH to use. To create a user: postgres=# create role <user_name> with login encrypted password '<password>'; To create a database: postgres=# create database <database_name> owner <user_name> template template0 encoding 'UTF8' lc_collate 'en_US.UTF-8' lc_ctype 'en_US.UTF-8'; If you plan for a remote application to use this database, make sure it can be accessed remotely. This bugzilla is included in oVirt 4.4.0 release, published on May 20th 2020. Since the problem described in this bug report should be resolved in oVirt 4.4.0 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. |