[MCOL-3494] S3 When postConfigure failed to access online storage, better error msg is needed Created: 2019-09-10 Updated: 2020-09-21 Resolved: 2020-08-31 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | installation |
| Affects Version/s: | 1.4.0 |
| Fix Version/s: | 1.4.5, 5.4.1 |
| Type: | Bug | Priority: | Major |
| Reporter: | Daniel Lee (Inactive) | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Sprint: | 2020-1, 2020-2, 2020-3, 2020-4, 2020-5, 2020-6, 2020-7 | ||||||||
| Description |
|
Build tested: 1.4.0-1 server commit: I configured the storagemanager.cnf to use AWS S3, but have the region setup incorrectly, used us-west-2b instead of us-west-2. postConfigure would return the following messages: We should output better messages to indicate that ColumnStore failed to connect to online S3 storage. postConfigure terminal output s1pm1: DBRM::send_recv caught: InetStreamSocket::connect: connect() error: Connection refused to: InetStreamSocket: sd: 10 inet: 127.0.0.1 port: 8616 crit.log file: [root@localhost columnstore]# cat crit.log err.log [root@localhost columnstore]# cat err.log |
| Comments |
| Comment by Ben Thompson (Inactive) [ 2019-10-03 ] |
|
There should be an message in the err.log, it should read: |
| Comment by Daniel Lee (Inactive) [ 2019-10-11 ] |
|
postConfigure is a interactive process. Instead of having inetstreamsocket internal messages, which don't have anything to the user, output to the terminal, we should output error intended for the user so the user knows what the issue is. |
| Comment by Ben Thompson (Inactive) [ 2019-12-03 ] |
|
Some of the log messages were going to warning log file instead of error log file. This should be fixed in |
| Comment by Patrick LeBlanc (Inactive) [ 2020-03-17 ] |
|
I thought of an easy way to do this. It would be hard for postconfig to see if SM is complaining when ProcMgr/Mon is trying to start the system, but it would be easy for postConfig to run an access checker directly and get an error code. So, what I'd do is add a cmdline option to SM, or write a new prog for this purpose (only a few lines I think), which runs the existing access check (and nothing else), and returns 0 on success, 1 on failure. In postConfig, run it when the user selects the SM storage option. |
| Comment by Daniel Lee (Inactive) [ 2020-05-06 ] |
|
Build tested: 1.4.4-1, 1.5.0-1 sourcee 1.4.4-1 /root/ColumnStore/buildColumnstoreFromGithubSource/server /root/ColumnStore/buildColumnstoreFromGithubSource/server/engine Merge pull request #1179 from pleblanc1976/update-libs3-ref Updated the s3 lib. 1.5.0-1 /root/ColumnStore/buildColumnstoreFromGithubSource/server /root/ColumnStore/buildColumnstoreFromGithubSource/server/engine Date: Tue May 5 12:35:14 2020 -0500 Merge pull request #1178 from pleblanc1976/update-libs3-ref-1.5 Updated s3 lib ref I purposely enter invalid S3 related parameters for cpimport. I hung for over 10 minutes after I selected S3 storage. I also did another test, having only the region being invalid. Same behavior. I also did another test by entering all correct parameter values, cpimport advanced to the next prompt immediately. [centos7:root~]# ./postConfigure -sm-bucket dleeqadbroot1 -sm-region hello -sm-id ok -sm-secret secret This is the MariaDB ColumnStore System Configuration and Installation tool. IMPORTANT: This tool requires to run on the Performance Module #1 Prompting instructions: Press 'enter' to accept a value in (), if available or ===== Setup System Server Type Configuration ===== There are 2 options when configuring the System Server Type: single and multi 'single' - Single-Server install is used when there will only be 1 server configured 'multi' - Multi-Server install is used when you want to configure multiple servers now or Select the type of System Server install [1=single, 2=multi] (2) > 1 Performing the Single Server Install. Enter System Name (columnstore-1) > ===== Setup Storage Configuration ===== ----- Setup Performance Module DBRoot Data Storage Mount Configuration ----- Columnstore supports the following storage options...
These options are available on this system: [1, 2, 4] |
| Comment by Daniel Lee (Inactive) [ 2020-08-25 ] |
|
reopen per my last test result |
| Comment by Ben Thompson (Inactive) [ 2020-08-26 ] |
|
This change to SM did not get merged from 1.4 to 1.5 and needed to update the columnstore-post-install script to actually test env variables set by user on install. |
| Comment by Daniel Lee (Inactive) [ 2020-08-31 ] |
|
Builds verified: 1.4.5-1 (drone #483), 1.5.4-1 (drone #496) [centos7:root~]# cat err.log |