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.

Sign up for GitHub

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

Jump to bottom

Bump dcos-mesos to 1.2.0-rc2 #1295

Merged
merged 2 commits into from Mar 2, 2017
Merged

Bump dcos-mesos to 1.2.0-rc2 #1295

merged 2 commits into from Mar 2, 2017

Conversation

adam-mesos
Copy link
Contributor

@adam-mesos adam-mesos commented Feb 24, 2017 •
edited

High Level Description

Bumps dcos-mesos to the official Apache Meso 1.2.0-rc2 tag to include such critical fixes as:

e1c11f0 Fixed fetcher to not pick up environment variables it should not see.
4683320 Fixed nested container agent flapping issue after reboot.
056978a Modified the executor driver to always relink on agent failover.
12e99f7 Fixed a bug around executor not able to use reserved resources.
d80af6c Fixed a crash on the default executor due to a failed invariant check.
7e5439d Fixed a crash on the agent when handling the SIGUSR1 signal.
e53fcaf Fixed a crash on the master upon receiving an invalid inverse offer.

Related Issues

  • CORE-864 Mesos masters crashing after receiving inverse offers
  • CORE-779 Bump dcos-mesos to Mesos 1.2 (when ready)

Checklist for all PR's

  • Included a test which will fail if code is reverted but test is not. If there is no test please explain here: Relying on Apache Mesos unit tests and existing (and continuously improving) dcos-integration-tests.
  • Read the DC/OS contributing guidelines
  • Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:

@d2iq-mergebot
Copy link
Collaborator

This repo has @mesosphere-mergebot integration. You can interact with the following commands.

@mesosphere-mergebot bump-ee

@adam-mesos adam-mesos added this to the 1.9 milestone Feb 24, 2017
@adam-mesos adam-mesos mentioned this pull request Feb 24, 2017
Closed
6 tasks
@adam-mesos
Copy link
Contributor Author

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Enterprise Bump PR: mesosphere/dcos-enterprise/pull/640

@greggomann
Copy link
Contributor

I checked out the branch that this PR bumps Mesos to - the cherry-picks included there look good to me. 🚢

greggomann
greggomann approved these changes Feb 28, 2017
View reviewed changes
Copy link
Contributor

@greggomann greggomann left a comment

Choose a reason for hiding this comment

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

Checked the commits included here; LGTM 🚢

@adam-mesos adam-mesos changed the base branch from 1.9.0-rc1 to master March 1, 2017 18:54
@adam-mesos
Copy link
Contributor Author

Rebased on latest dcos/dcos master.

spahl
spahl approved these changes Mar 1, 2017
View reviewed changes
Copy link
Contributor

@spahl spahl left a comment

Choose a reason for hiding this comment

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

Shas are pointing to the right things 🚢

@adam-mesos
Copy link
Contributor Author

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Enterprise Bump mesosphere/dcos-enterprise/pull/640 updated.

klueska
klueska reviewed Mar 1, 2017
View reviewed changes
packages/mesos/README.md
@@ -4,6 +4,4 @@ These commits can be found in the repository at <a href="https://github.com/meso
<li>[09975e642168a47c466e211f02a590b6e3e1e40c] Changed agent_host to expect a relative path.
<li>[d2cf9ac1dfa0d7d2b5ebcf8f9da86190706bb5cd] Mesos UI: Change paths, pailer, and logs to work with a reverse proxy.
<li>[3d042bae0431da237d2c109525ea81fe8eb9943c] Revert "Fixed the broken metrics information of master in WebUI."
<li>[7fdf195b5f3bb4d4a0bd155d1663a96f9eaf4da1] Fixed fetcher to not pick up environment variables it should not see.
Copy link
Contributor

Choose a reason for hiding this comment

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

Was this one actually cherry-picked before, but not listed here? Or is this a new one? If it's a new one, why is it before the GPU one and not after?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This patch was cherry-picked on top of rc1, but is now included (in improved form) in the official rc2, so it is being removed from our cherry-pick list.

@adam-mesos
Copy link
Contributor Author

@mesosphere-mergebot bump-ee

adam-mesos added a commit to mesosphere/dcos that referenced this pull request Mar 1, 2017
@adam-mesos
Merge dcos#1295
a8b40cd
@adam-mesos adam-mesos mentioned this pull request Mar 1, 2017
Merged
@adam-mesos adam-mesos merged commit 54f464e into dcos:master Mar 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spahl spahl spahl approved these changes

@klueska klueska klueska approved these changes

@greggomann greggomann greggomann approved these changes

@tillt tillt Awaiting requested review from tillt

@hatred hatred Awaiting requested review from hatred

@jieyu jieyu Awaiting requested review from jieyu

Assignees
No one assigned
Labels
Ship It
Projects
None yet
Milestone
1.9
5 participants
@adam-mesos @d2iq-mergebot @greggomann @spahl @klueska