{"action":"created","issue":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/15466","labels_url":"https://api.github.com/repos/elastic/elasticsearch/issues/15466/labels{/name}","comments_url":"https://api.github.com/repos/elastic/elasticsearch/issues/15466/comments","events_url":"https://api.github.com/repos/elastic/elasticsearch/issues/15466/events","html_url":"https://github.com/elastic/elasticsearch/issues/15466","id":122410397,"number":15466,"title":"integration tests doc example out of date","user":{"login":"martin-a","id":1248717,"avatar_url":"https://avatars.githubusercontent.com/u/1248717?v=3","gravatar_id":"","url":"https://api.github.com/users/martin-a","html_url":"https://github.com/martin-a","followers_url":"https://api.github.com/users/martin-a/followers","following_url":"https://api.github.com/users/martin-a/following{/other_user}","gists_url":"https://api.github.com/users/martin-a/gists{/gist_id}","starred_url":"https://api.github.com/users/martin-a/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martin-a/subscriptions","organizations_url":"https://api.github.com/users/martin-a/orgs","repos_url":"https://api.github.com/users/martin-a/repos","events_url":"https://api.github.com/users/martin-a/events{/privacy}","received_events_url":"https://api.github.com/users/martin-a/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/discuss","name":"discuss","color":"fbca04"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/docs","name":"docs","color":"b3694f"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-12-16T02:00:09Z","updated_at":"2015-12-18T07:23:22Z","closed_at":"2015-12-18T07:23:22Z","body":"[Documentation about changing node settings](https://www.elastic.co/guide/en/elasticsearch/reference/2.1/integration-tests.html#changing-node-settings) in integration tests need some update.\r\nthe `settingsBuilder()` method is not accessible from ESIntegTestCase child classes.\r\n\r\nHere is a workaround to add some custom node settings that work with 2.1 sourcecode\r\n```\r\npublic class Mytests extends ESIntegTestCase {\r\n  @Override\r\n  protected Settings nodeSettings(int nodeOrdinal) {\r\n      return Settings.settingsBuilder().put(super.nodeSettings(nodeOrdinal))\r\n             .put(\"node.mode\", \"network\")\r\n             .build();\r\n  }\r\n\r\n}\r\n```"},"comment":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/comments/165700129","html_url":"https://github.com/elastic/elasticsearch/issues/15466#issuecomment-165700129","issue_url":"https://api.github.com/repos/elastic/elasticsearch/issues/15466","id":165700129,"user":{"login":"rjernst","id":289412,"avatar_url":"https://avatars.githubusercontent.com/u/289412?v=3","gravatar_id":"","url":"https://api.github.com/users/rjernst","html_url":"https://github.com/rjernst","followers_url":"https://api.github.com/users/rjernst/followers","following_url":"https://api.github.com/users/rjernst/following{/other_user}","gists_url":"https://api.github.com/users/rjernst/gists{/gist_id}","starred_url":"https://api.github.com/users/rjernst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rjernst/subscriptions","organizations_url":"https://api.github.com/users/rjernst/orgs","repos_url":"https://api.github.com/users/rjernst/repos","events_url":"https://api.github.com/users/rjernst/events{/privacy}","received_events_url":"https://api.github.com/users/rjernst/received_events","type":"User","site_admin":false},"created_at":"2015-12-18T07:23:22Z","updated_at":"2015-12-18T07:23:22Z","body":"I pushed a fix, the docs were assuming `Settings.setttingsBuilder` was statically imported. We should also really just get rid of `settingsBuilder()`, as it is the exact same as `builder()`...\r\n\r\nMaster: 2093ea5\r\n2.x: f545b6d\r\n2.1: c9d765f\r\n2.0: 1c094d3"}}