Skip to content
This repository was archived by the owner on Oct 23, 2024. It is now read-only.

Upgrade to latest mesos-go #496

Closed
wants to merge 3 commits into from
Closed

Upgrade to latest mesos-go #496

wants to merge 3 commits into from

Conversation

jdef
Copy link
Contributor

@jdef jdef commented Feb 25, 2017

  • minor ZK detector interface changes (should have no impact on mesos-dns)
  • mesos-go v0 API has been repackaged, update existing references to new package naming
  • ignore .bak files

@jdef jdef added the WIP label Feb 25, 2017
@jdef jdef requested review from gpaul and nlsun February 25, 2017 17:52
drewkerrigan
drewkerrigan approved these changes Oct 1, 2017
View reviewed changes
Copy link
Contributor

@drewkerrigan drewkerrigan left a comment

Choose a reason for hiding this comment

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

@jdef I ended up doing this in #508 as well - I'd be fine with merging this unless you had more to add.

@drewkerrigan drewkerrigan mentioned this pull request Oct 5, 2017
Merged
@jdef
Copy link
Contributor Author

jdef commented Jan 24, 2019

mesos-dns was migrated to dep. If we want to update the mesos-go deps, we should do it in another PR.

@jdef jdef closed this Jan 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@gpaul gpaul Awaiting requested review from gpaul

@nlsun nlsun Awaiting requested review from nlsun

1 more reviewer

@drewkerrigan drewkerrigan drewkerrigan approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
WIP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jdef @drewkerrigan