Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support dynamic node role #3436

Merged
merged 10 commits into from
Jun 14, 2022
Merged

Support dynamic node role #3436

merged 10 commits into from
Jun 14, 2022

Commits on May 26, 2022

  1. Support unknown node role 

    Currently OpenSearch only supports several built-in nodes like data node
role. If specify unknown node role, OpenSearch node will fail to start.
This limit how to extend OpenSearch to support some extension function.
For example, user may prefer to run ML tasks on some dedicated node
which doesn't serve as any built-in node roles. So the ML tasks won't
impact OpenSearch core function. This PR removed the limitation and user
can specify any node role and OpenSearch will start node correctly with
that unknown role. This opens the door for plugin developer to run
specific tasks on dedicated nodes.

Issue: opensearch-project#2877

Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed May 26, 2022
    Configuration menu
    Copy the full SHA
    433a196 View commit details
    Browse the repository at this point in the history

  2. fix cat nodes rest API spec 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed May 26, 2022
    Configuration menu
    Copy the full SHA
    4cb8dac View commit details
    Browse the repository at this point in the history

  3. fix mixed cluster IT failure 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed May 26, 2022
    Configuration menu
    Copy the full SHA
    2f2df16 View commit details
    Browse the repository at this point in the history

Commits on Jun 1, 2022

  1. add DynamicRole 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed Jun 1, 2022
    Configuration menu
    Copy the full SHA
    4245543 View commit details
    Browse the repository at this point in the history

  2. change generator method name 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed Jun 1, 2022
    Configuration menu
    Copy the full SHA
    a2fea20 View commit details
    Browse the repository at this point in the history

  3. fix failed docker test 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed Jun 1, 2022
    Configuration menu
    Copy the full SHA
    836be1f View commit details
    Browse the repository at this point in the history

Commits on Jun 14, 2022

  1. transform role name to lower case to avoid confusion 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed Jun 14, 2022
    Configuration menu
    Copy the full SHA
    5101c9a View commit details
    Browse the repository at this point in the history

  2. transform the node role abbreviation to lower case 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed Jun 14, 2022
    Configuration menu
    Copy the full SHA
    6e286c1 View commit details
    Browse the repository at this point in the history

  3. fix checkstyle 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed Jun 14, 2022
    Configuration menu
    Copy the full SHA
    8e2f2d8 View commit details
    Browse the repository at this point in the history

  4. add test for case-insensitive role name change 

    Signed-off-by: Yaliang Wu <ylwu@amazon.com>
    @ylwu-amzn
    ylwu-amzn committed Jun 14, 2022
    Configuration menu
    Copy the full SHA
    81d0d60 View commit details
    Browse the repository at this point in the history