Skip to content

Conversation

@risdenk
Copy link
Contributor

@risdenk risdenk commented Sep 9, 2017

Closes #26562

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@jasontedor
Copy link
Member

Thanks for the PR @risdenk. Do the packaging tests pass with this change?

@jasontedor
Copy link
Member

I ran the packaging tests locally after installing Vagrant 2.0.0 and everything worked as expected.

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@jasontedor jasontedor merged commit f476c10 into elastic:master Sep 9, 2017
jasontedor pushed a commit that referenced this pull request Sep 9, 2017
This commit adds support for Vagrant 2.x by allowing these versions to
pass the Vagrant version check.

Relates #26563
jasontedor pushed a commit that referenced this pull request Sep 9, 2017
This commit adds support for Vagrant 2.x by allowing these versions to
pass the Vagrant version check.

Relates #26563
jasontedor pushed a commit that referenced this pull request Sep 9, 2017
This commit adds support for Vagrant 2.x by allowing these versions to
pass the Vagrant version check.

Relates #26563
@jasontedor
Copy link
Member

Thanks again @risdenk.

@risdenk risdenk deleted the vagrant2support branch September 9, 2017 21:37
@risdenk
Copy link
Contributor Author

risdenk commented Sep 9, 2017

Thanks @jasontedor!

@lcawl lcawl removed the v6.1.0 label Dec 12, 2017
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team v5.6.1 v6.0.0-rc1 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vagrantVersionCheck should support Vagrant 2.x

6 participants