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

Always use remote branch for updates #575

Merged

Conversation

danielmitterdorfer
Copy link
Member

With this commit we always attempt to checkout the remote branch first.
In a scenario where a user has already cloned the repository but uses
the --offline flag the correct branches might not be proper local
tracking branches but are already available on the target machine as
remote branches.

Closes #573

With this commit we always attempt to checkout the remote branch first.
In a scenario where a user has already cloned the repository but uses
the `--offline` flag the correct branches might not be proper local
tracking branches but are already available on the target machine as
remote branches.

Closes elastic#573
@danielmitterdorfer danielmitterdorfer added bug Something's wrong :Track Management New operations, changes in the track format, track download changes and the like :Benchmark Candidate Management Anything affecting how Rally sets up Elasticsearch labels Sep 19, 2018
@danielmitterdorfer danielmitterdorfer added this to the 1.0.2 milestone Sep 19, 2018
Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

LGTM, sorry this fell through the cracks.

@danielmitterdorfer danielmitterdorfer merged commit 0d84a46 into elastic:master Oct 12, 2018
@danielmitterdorfer
Copy link
Member Author

Thanks for the review and no worries. :)

@danielmitterdorfer danielmitterdorfer deleted the offline-update-branch branch October 12, 2018 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Benchmark Candidate Management Anything affecting how Rally sets up Elasticsearch bug Something's wrong :Track Management New operations, changes in the track format, track download changes and the like
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants