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

Updated mesos package to use cmake. #7190

Merged
merged 1 commit into from Jun 1, 2020

Conversation

akornatskyy
Copy link
Contributor

@akornatskyy akornatskyy commented Apr 13, 2020

High-level description

Build dcos linux mesos package with cmake.

Corresponding DC/OS tickets (required)

  • D2IQ-66807 build dcos linux mesos package with cmake.

@d2iq-mergebot
Copy link
Collaborator

This repo has @mesosphere-mergebot integration. You can perform the following commands by submitting a comment. Submit a comment with content "@mesosphere-mergebot help" to view more detailed help text and examples. Be sure the have a look at the mergebot documentation, too.

@mesosphere-mergebot changelog-not-required reason 
@mesosphere-mergebot bump-ee  
@mesosphere-mergebot request-review  
@mesosphere-mergebot sync  
@mesosphere-mergebot override-status pr-status-check jira-url 
@mesosphere-mergebot backport  
@mesosphere-mergebot label [merge-by-mergebot|Ship It|Work In Progress|Request For Comment|dependencies|Ready For Review|do not merge - test|Holding] 
@mesosphere-mergebot merge-it  
  • PR creators can apply one of [Ready For Review|Work In Progress]. Owners can apply any label.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Your pull request's branch is not based on the most recent version of master. Please rebase your changes against this repo's master branch.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Your pull request's branch is not based on the most recent version of master. Please rebase your changes against this repo's master branch.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Your pull request's branch is not based on the most recent version of master. Please rebase your changes against this repo's master branch.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Your pull request's branch is not based on the most recent version of master. Please rebase your changes against this repo's master branch.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Your pull request's branch is not based on the most recent version of master. Please rebase your changes against this repo's master branch.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Your pull request's branch is not based on the most recent version of master. Please rebase your changes against this repo's master branch.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Your pull request's branch is not based on the most recent version of master. Please rebase your changes against this repo's master branch.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@akornatskyy
Copy link
Contributor Author

@mesosphere-mergebot label Ready For Review

@asekretenko
Copy link
Contributor

asekretenko commented May 29, 2020

@akornatskyy note that the build the EE bump fails (the mergebot/enterprise/build-status/aggregate check below, also see the EE bump PR), because dcos-ee-mesos-modules still use the automake build.

Please incorporate your changes from
https://github.com/mesosphere/dcos-enterprise/pull/7843
into the EE bump PR created by mergebot
(https://github.com/mesosphere/dcos-enterprise/pull/7733, the link posted above by mesosphere-mergebot)

This is the case when we have to change OSS and EE together.

@akornatskyy
Copy link
Contributor Author

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

packages/mesos/build Outdated Show resolved Hide resolved
@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

asekretenko
asekretenko previously approved these changes May 29, 2020
Copy link
Contributor

@asekretenko asekretenko left a comment

Choose a reason for hiding this comment

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

@akornatskyy Thanks for all this work!

If the remaining integration tests pass (I think now it is reasonable to hope that they will), the only thing that prevents us from merging this patch will be the Mesos bump that brings in the libprocess configuration fix that will resolve the failing memory profiler test.

After that bump gets into DCOS and if nothing else comes up, we (I/Greg/someone lese) will be able to land this PR pair.

@akornatskyy
Copy link
Contributor Author

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Your pull request's branch is not based on the most recent version of master. Please rebase your changes against this repo's master branch.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

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

@greggomann
Copy link
Contributor

@mesosphere-mergebot changelog-not-required No user-facing change.

@d2iq-mergebot
Copy link
Collaborator

CHANGES.MD check override issued with the provided reason: No user-facing change.

"git": "https://github.com/apache/mesos",
"ref": "3d68993c8743231b6067738050786e750edda9b1",
"ref_origin": "master"
"sources": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Hey @akornatskyy @asekretenko ,
Do you happen to remember what was the reason to change this from single_source to sources? I'm asking because unfortunatelly it broke our 'Bump Mesos CI' script: https://jenkins.mesosphere.com/service/jenkins/view/Mesos/job/mesos/job/BumpDCOS/job/Bump_DCOS_Mesos_And_Modules/645/console

Does it make sense to change it back to single_source? Any concerns?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't see much sense for this change... but generally speaking using single_source should work as well. no concerns.

abudnik added a commit to abudnik/dcos that referenced this pull request Jun 17, 2020
…ages.

This patch reverts back changes made in PR dcos#7190.
abudnik added a commit to abudnik/dcos that referenced this pull request Jun 17, 2020
…ages.

This patch reverts back some changes made in PR dcos#7190.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants