Skip to content

[kbn-es] make custom snapshot url a feature#45750

Merged
spalger merged 2 commits intoelastic:masterfrom
spalger:fix/es-custom-archive
Sep 15, 2019
Merged

[kbn-es] make custom snapshot url a feature#45750
spalger merged 2 commits intoelastic:masterfrom
spalger:fix/es-custom-archive

Conversation

@spalger
Copy link
Copy Markdown
Contributor

@spalger spalger commented Sep 15, 2019

Rather than simply overriding the URL, lay the groundwork for a fallback URL that's a standard part of kbn-es

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-operations

@spalger
Copy link
Copy Markdown
Contributor Author

spalger commented Sep 15, 2019

Verified that it works

oss:
image

x-pack:
image

@spalger spalger marked this pull request as ready for review September 15, 2019 15:10
@spalger spalger merged commit 510566d into elastic:master Sep 15, 2019
spalger pushed a commit that referenced this pull request Sep 15, 2019
* [kbn-es] make custom snapshot url a feature

* download es snapshot first

(cherry picked from commit 510566d)
spalger pushed a commit that referenced this pull request Sep 15, 2019
* [kbn-es] make custom snapshot url a feature

* download es snapshot first

(cherry picked from commit 510566d)
(cherry picked from commit f85c740)
spalger pushed a commit that referenced this pull request Sep 15, 2019
* [kbn-es] make custom snapshot url a feature

* download es snapshot first

(cherry picked from commit 510566d)
(cherry picked from commit f85c740)
spalger pushed a commit that referenced this pull request Sep 15, 2019
* [kbn-es] make custom snapshot url a feature

* download es snapshot first

(cherry picked from commit 510566d)
(cherry picked from commit f85c740)
@spalger
Copy link
Copy Markdown
Contributor Author

spalger commented Sep 15, 2019

7.x/7.5: f85c740
7.4: 14c9c72
7.3: ceefccd
6.8: c2fbda6

@spalger spalger deleted the fix/es-custom-archive branch September 15, 2019 15:18
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants