Skip to content

Comments

[ci] use es snapshots on non pull requests#22834

Merged
jbudz merged 3 commits intoelastic:masterfrom
jbudz:ci/snapshot-master
Sep 10, 2018
Merged

[ci] use es snapshots on non pull requests#22834
jbudz merged 3 commits intoelastic:masterfrom
jbudz:ci/snapshot-master

Conversation

@jbudz
Copy link
Contributor

@jbudz jbudz commented Sep 7, 2018

No description provided.

@jbudz jbudz added WIP Work in progress Team:Operations Kibana-Operations Team labels Sep 7, 2018
@jbudz jbudz self-assigned this Sep 7, 2018
@elasticmachine
Copy link
Contributor

💔 Build Failed

function checkout_clone_target {
pick_clone_target
if [[ $cloneBranch = "master" && $cloneAuthor = "elastic" ]]; then
export TEST_ES_FROM=snapshot
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to checkout the es branch if we are running from a snapshot? Might be good time savings.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah i was hoping to do that, we reference a dotfile in the repo for setting up the correct java version. i'll see if there's someone else we can find it

Copy link
Member

Choose a reason for hiding this comment

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

Ah, that's right - that's fine for now.

Copy link
Member

Choose a reason for hiding this comment

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

This also is only for master - I am fine with that for now. We can always add 6.x to the list.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

jbudz added a commit that referenced this pull request Sep 10, 2018
* [ci] use es snapshots on non pull requests

* remove extra export

* always clone

* master -> 6.x
@jbudz jbudz added v7.0.0 v6.5.0 and removed WIP Work in progress labels Sep 10, 2018
@jbudz jbudz merged commit 47cd173 into elastic:master Sep 10, 2018
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 this pull request may close these issues.

3 participants