Skip to content

[ui/tilemap] switch tile service based on env#10007

Merged
spalger merged 1 commit intoelastic:masterfrom
spalger:implement/tiles-stage-in-dev
Feb 21, 2017
Merged

[ui/tilemap] switch tile service based on env#10007
spalger merged 1 commit intoelastic:masterfrom
spalger:implement/tiles-stage-in-dev

Conversation

@spalger
Copy link
Copy Markdown
Contributor

@spalger spalger commented Jan 23, 2017

We currently have the "prod" tile service hard coded in the 5.2 branch in preparation for release, and plan to switch master over once we branch in preparation for release. This means we are at risk of forgetting to switch it, and since we don't really have a process for ensuring we don't forget these types of things I think we need a different solution.

This adjusts the config to use a different default in dev and prod, so when we build Kibana it will automatically use the prod version, regardless of the branch.

Copy link
Copy Markdown
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, it's a better approach.

@thomasneirynck thomasneirynck requested a review from epixa February 14, 2017 18:39
@epixa epixa self-assigned this Feb 14, 2017
@thomasneirynck thomasneirynck added the Feature:Visualizations Generic visualization features (in case no more specific feature label is available) label Feb 21, 2017
@spalger spalger merged commit a5f4ac2 into elastic:master Feb 21, 2017
@spalger spalger deleted the implement/tiles-stage-in-dev branch February 21, 2017 16:44
spalger added a commit that referenced this pull request Feb 21, 2017
Backports PR #10007

**Commit 1:**
[ui/tilemap] switch tile service based on env

* Original sha: 47701f3
* Authored by spalger <spalger@users.noreply.github.com> on 2017-01-23T16:36:13Z
thomasneirynck pushed a commit that referenced this pull request Feb 21, 2017
Backports PR #10007

**Commit 1:**
[ui/tilemap] switch tile service based on env

* Original sha: 47701f3
* Authored by spalger <spalger@users.noreply.github.com> on 2017-01-23T16:36:13Z
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v5.4.0 v6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants