Skip to content

Commit

Permalink
Backport PR elastic#8391
Browse files Browse the repository at this point in the history
---------

**Commit 1:**
Don't try to delete index template along with pattern

This commit reverts a change to use the ingest API when deleting index
patterns via the Kibana UI. Back when I was building the Filebeat wizard
it made sense to try to delete any index templates or pipelines that may
have been created along with an index pattern. But now that we're only
shipping with CSV Upload, and we don't delete the actual indices that
CSV upload creates, it doesn't make much sense to delete the template.
Now we'll treate the indices and templates consistently.

This also fixes an issue where users would get a fatal error if they
were using Security and they didn't have permissions to delete
templates. Every index pattern deletion would also attempt to delete an
associated template, so if the user didn't have the correct permissions
they would get a 403.

Related: elastic#6457

* Original sha: daa5b3b
* Authored by Matthew Bargar <[email protected]> on 2016-09-20T22:43:03Z
  • Loading branch information
elastic-jasper committed Sep 22, 2016
1 parent 4752dbb commit bc7b8d3
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,7 @@ uiModules.get('apps/management')
}
}

ingest.delete($scope.indexPattern.id)
.then($scope.indexPattern.destroy.bind($scope.indexPattern))
courier.indexPatterns.delete($scope.indexPattern)
.then(refreshKibanaIndex)
.then(function () {
$location.url('/management/data/index');
Expand Down

0 comments on commit bc7b8d3

Please sign in to comment.