fix: GreetingModule sometimes occurs a timeout error

[24sama]

What type of PR is this?

/kind bug

What this PR does / why we need it:

• Fix: GreetingModule sometimes occurs a timeout error.
  For now, the timeout value of the GreetingModule will be calculated by adding up the timeout values of all hosts
• Print human-readable task timeout error message.

Which issue(s) this PR fixes:

Fixes #1288

Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

[ks-ci-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 24sama

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [24sama]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Nello-Angelo]

    hello := &task.RemoteTask{
            Name:     "Greetings",
            Desc:     "Greetings",
            Hosts:    h.Runtime.GetAllHosts(),
            Action:   new(GreetingsTask),
            Parallel: true,
            Timeout:  800 * time.Second,
    }

I just changed the option from Timeout: 30 * time.Second, to Timeout: 800 * time.Second, in kubekey/pkg/bootstrap/precheck/module.go (but you may set any time)

[24sama]

I just changed the option from Timeout: 30 * time.Second, to Timeout: 800 * time.Second, in kubekey/pkg/bootstrap/precheck/module.go (but you may set any time)

This is also a hardcode and maybe it doesn't work when the number of hosts is very large (100 hosts?).

This PR will automatically increase the imeout value of this task according to the number of hosts.
For example:

hosts:
  - {name: node1, address: 172.16.0.2, internalAddress: 172.16.0.2, port: 8022, user: ubuntu, password: "Qcloud@123"} 
  - {name: node2, address: 172.16.0.3, internalAddress: 172.16.0.3, password: "Qcloud@123"}
  - {name: node3, address: 172.16.0.4, internalAddress: 172.16.0.4, password: "Qcloud@123"}

The task timeout will be 30s cause the default host's timeout value is 10s.

- {name: node1, address: 172.16.0.2, internalAddress: 172.16.0.2, port: 8022, user: ubuntu, password: "Qcloud@123", timeout: 30} 
  - {name: node2, address: 172.16.0.3, internalAddress: 172.16.0.3, password: "Qcloud@123", timeout: 30}
  - {name: node3, address: 172.16.0.4, internalAddress: 172.16.0.4, password: "Qcloud@123", timeout: 30}

The task timeout will be 90s.

- {name: node1, address: 172.16.0.2, internalAddress: 172.16.0.2, port: 8022, user: ubuntu, password: "Qcloud@123", timeout: 30} 
  - {name: node2, address: 172.16.0.3, internalAddress: 172.16.0.3, password: "Qcloud@123", timeout: 60}
  - {name: node3, address: 172.16.0.4, internalAddress: 172.16.0.4, password: "Qcloud@123", timeout: 30}
  - {name: node3, address: 172.16.0.4, internalAddress: 172.16.0.4, password: "Qcloud@123", timeout: 10}

The task timeout will be 130s (30+60+30+10).

[pixiake]

/lgtm

[ks-ci-bot]

LGTM label has been added.

Git tree hash: 849d5f213b0aa7e8d5e9c552a020d43da4022ab5