Add change remote ssh port in alluxio-common.sh

[zhaoyim]

… all

What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.

Why are the changes needed?

Please clarify why the changes are needed. For instance,

1. If you propose a new API, clarify the use case for a new API.
2. If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?

Please list the user-facing changes introduced by your change, including

1. change in user-facing APIs
2. addition or removal of property keys
3. webui

[alluxio-bot]

Hi @zhaoyim, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

[alluxio-bot]

Automated checks report:

• Commits associated with Github account: PASS
• PR title follows the conventions: FAIL
  • The title of the PR does not pass all the checks. Please fix the following issues:
    • First word must be capitalized
    • First word of title ("15520") is not an imperative verb. Please use one of the valid words

Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.

[alluxio-bot]

You did it @zhaoyim!

Thank you for signing the Contribution License Agreement.

[zhaoyim]

Add a global var to define the remote ssh port in common.sh. In this way, we just only to change the port once.

[maobaolong]

@zhaoyim Thanks for this contribution! Can you give an example to show how to set the ssh PORT?

Something like PORT=XXX in conf/alluxio-env.sh?

[zhaoyim]

Can you give an example to show how to set the ssh PORT?
Something like PORT=XXX in conf/alluxio-env.sh?

Will you mean add the example in the alluxio-common.sh ? I add the example comments on it, could you have a look? Thanks!

[maobaolong]

@zhaoyim Sorry, I mean that alluxio-common.sh is a common tools for alluxio shell framework, but alluxio-env.sh is a config file as well as alluxio-properties, so we should tell user add an env var SSH_PORT in alluxio-env.sh can change the default SSH_PORT.

[zhaoyim]

@zhaoyim Sorry, I mean that alluxio-common.sh is a common tools for alluxio shell framework, but alluxio-env.sh is a config file as well as alluxio-properties, so we should tell user add an env var SSH_PORT in alluxio-env.sh can change the default SSH_PORT.

Thanks for your comments! I will move the SSH_PORT into the alluxio-env.sh and changed all the sh source from alluxio-env.sh.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.