-
Notifications
You must be signed in to change notification settings - Fork 82
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
Bump @dcos/mesos-client 0.1.9 #2571
Conversation
This repo has @mesosphere-mergebot integration. You can interact with the following commands.
|
8411017
to
6b0fbe4
Compare
Alfred test this |
Alfred test this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 feel free to merge once tests pass
Alfred test this |
7 similar comments
Alfred test this |
Alfred test this |
Alfred test this |
Alfred test this |
Alfred test this |
Alfred test this |
Alfred test this |
Fail consistently on Jenkins
But locally they pass
|
Alfred test this |
aec7b67
6b0fbe4
to
aec7b67
Compare
Alfred test this |
2 similar comments
Alfred test this |
Alfred test this |
Note: I made a commit to retrieve the expected data from the mesos master that fixed the tests. |
33e25c7
to
5afb283
Compare
5afb283
to
17c5cd9
Compare
17c5cd9
to
4d9ed29
Compare
Rebased & lifted the |
Alfred test this |
2 similar comments
Alfred test this |
Alfred test this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😱
🚢 |
✅ This PR depends on changes that are going to be introduced with #2593. For a list of changes please see: orlandohohmeier/fix/adjust-mesos-request-stubs...drozhkov/mesos-client-0.1.9
We thought we've resolved the issue, but that time bundle distribution messed up thus the old version persisted in the
dist
folder. With this bump I reassured that the updates went through.Closes DCOS-19946