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.

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

Remote cluster info returns fatally for invalid hostnames #35658

Closed
jen-huang opened this issue Nov 16, 2018 · 2 comments · Fixed by #36671
Closed

Remote cluster info returns fatally for invalid hostnames #35658

jen-huang opened this issue Nov 16, 2018 · 2 comments · Fixed by #36671
Assignees
Labels
blocker :Distributed/Network Http and internode communication implementations

Comments

@jen-huang
Copy link

I added a cluster with an invalid hostname:

PUT _cluster/settings
{
  "persistent": {
    "cluster": {
      "remote": {
        "jentest": {
          "seeds": [
            "testname:1234"
          ]
        }
      }
    }
  }
}

this is causing GET _remote/info to return fatally

{
  "error": {
    "root_cause": [
      {
        "type": "illegal_argument_exception",
        "reason": "unknown host [testname]"
      }
    ],
    "type": "illegal_argument_exception",
    "reason": "unknown host [testname]",
    "caused_by": {
      "type": "unknown_host_exception",
      "reason": "testname: nodename nor servname provided, or not known"
    }
  },
  "status": 400
}

For our Remote Clusters management UI feature, we're using _remote/info to fetch the list and info of clusters for display. Ideally it would still return the list of all clusters, maybe with the error attached to the corresponding cluster.

@jasontedor jasontedor added the :Distributed/Network Http and internode communication implementations label Nov 16, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@cjcenizal
Copy link
Contributor

A temporary UI-based solution would be to give the user a button that says "Remove remote clusters that have errors". Clicking this button will show a confirmation modal which lists the remote clusters containing seed nodes that match the offending host name. Confirming their removal will send a request to _cluster/settings that removes them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker :Distributed/Network Http and internode communication implementations
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants