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

apm-server setup --template installs incorrect template #1922

Closed
graphaelli opened this issue Feb 13, 2019 · 2 comments · Fixed by #1934
Closed

apm-server setup --template installs incorrect template #1922

graphaelli opened this issue Feb 13, 2019 · 2 comments · Fixed by #1934
Assignees
Labels
Milestone

Comments

@graphaelli
Copy link
Member

apm-server setup --template -E setup.template.overwrite=true -e

is currently (at 7127b9e) creating an index pattern including:

    "order" : 1,
    "index_patterns" : [
      "apm-server-7.0.0-*"
    ],
    "settings" : {
      "index" : {
        "lifecycle" : {
          "name" : "apm-server-7.0.0",
          "rollover_alias" : "apm-server-7.0.0"
        },
...

Logs also indicate settings are off:

2019-02-13T02:49:50.356Z	INFO	[index-management.ilm]	ilm/ilm.go:123	Policy name: apm-server-7.0.0-beta1
2019-02-13T02:49:50.360Z	INFO	[index-management]	idxmgmt/std.go:165	Set output.elasticsearch.index to 'apm-server-7.0.0-beta1' as ILM is enabled.
2019-02-13T02:49:50.394Z	INFO	[index-management]	idxmgmt/std.go:223	Auto ILM enable success.
2019-02-13T02:49:50.397Z	INFO	[index-management.ilm]	ilm/std.go:134	do not generate ilm policy: exists=true, overwrite=false
2019-02-13T02:49:50.397Z	INFO	[index-management]	idxmgmt/std.go:237	ILM policy successfully loaded.
2019-02-13T02:49:50.397Z	INFO	[index-management]	idxmgmt/std.go:355	Set setup.template.name to '{apm-server-7.0.0-beta1 {now/d}-000001}' as ILM is enabled.
2019-02-13T02:49:50.397Z	INFO	[index-management]	idxmgmt/std.go:360	Set setup.template.pattern to 'apm-server-7.0.0-beta1-*' as ILM is enabled.
2019-02-13T02:49:50.397Z	INFO	[index-management]	idxmgmt/std.go:394	Set settings.index.lifecycle.rollover_alias in template to {apm-server-7.0.0-beta1 {now/d}-000001} as ILM is enabled.
2019-02-13T02:49:50.398Z	INFO	[index-management]	idxmgmt/std.go:398	Set settings.index.lifecycle.name in template to {apm-server-7.0.0-beta1 map[policy:{"phases":{"hot":{"actions":{"rollover":{"max_age":"30d","max_size":"50gb"}}}}}]} as ILM is enabled.

The correct template is installed with apm-server run.

@graphaelli graphaelli added the bug label Feb 13, 2019
@graphaelli graphaelli added this to the 7.0 milestone Feb 13, 2019
@simitt
Copy link
Contributor

simitt commented Feb 13, 2019

Oh, great finding, I can look into that one.

@simitt
Copy link
Contributor

simitt commented Feb 13, 2019

created a PR in beats elastic/beats#10721

@simitt simitt self-assigned this Feb 13, 2019
simitt added a commit to simitt/apm-server that referenced this issue Feb 18, 2019
Add tests for template setup.
fixes elastic#1922
simitt added a commit to simitt/apm-server that referenced this issue Feb 18, 2019
Add tests for template setup.
fixes elastic#1922
simitt added a commit to simitt/apm-server that referenced this issue Feb 18, 2019
Add tests for template setup.
fixes elastic#1922
simitt added a commit to simitt/apm-server that referenced this issue Feb 18, 2019
Add tests for template setup.
fixes elastic#1922
simitt added a commit to simitt/apm-server that referenced this issue Feb 18, 2019
Add tests for template setup.
fixes elastic#1922
simitt added a commit to simitt/apm-server that referenced this issue Feb 19, 2019
Add tests for template setup.
fixes elastic#1922
simitt added a commit that referenced this issue Feb 20, 2019
Add tests for template setup.
fixes #1922
simitt added a commit that referenced this issue Feb 20, 2019
Add tests for template setup.
fixes #1922
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants