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

run jenkins java builds in memory constrained pods #17832

Merged
merged 2 commits into from
Jan 8, 2018
Merged

run jenkins java builds in memory constrained pods #17832

merged 2 commits into from
Jan 8, 2018

Conversation

jim-minter
Copy link
Contributor

No description provided.

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Dec 15, 2017
@jim-minter
Copy link
Contributor Author

/hold
(until image ecosystem tests pass)

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 15, 2017
@jim-minter
Copy link
Contributor Author

/test image_ecosystem

@jim-minter jim-minter changed the title run jenkins maven build in memory constrained pod [WIP] run jenkins maven build in memory constrained pod Dec 15, 2017
@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 15, 2017
@bparees
Copy link
Contributor

bparees commented Dec 15, 2017

/lgtm
/hold cancel
(image eco failure was not related to pipelines/jenkins)

@openshift-ci-robot openshift-ci-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Dec 15, 2017
@bparees
Copy link
Contributor

bparees commented Dec 15, 2017

/hold
/test extended_builds
looks like it's the builds test that exercises the pipeline?

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 15, 2017
@bparees
Copy link
Contributor

bparees commented Dec 15, 2017

(feel free to remove the hold once the builds test passes)

@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 18, 2017
@openshift-merge-robot openshift-merge-robot removed the lgtm Indicates that a PR is ready to be merged. label Dec 18, 2017
@jim-minter jim-minter changed the title [WIP] run jenkins maven build in memory constrained pod run jenkins maven build in memory constrained pod Dec 18, 2017
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 18, 2017
@jim-minter
Copy link
Contributor Author

/unhold

@jim-minter
Copy link
Contributor Author

/hold
/test extended_builds

@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 18, 2017
@jim-minter jim-minter changed the title run jenkins maven build in memory constrained pod run jenkins java builds in memory constrained pods Dec 18, 2017
@jim-minter
Copy link
Contributor Author

@bparees @gabemontero this adds an additional gradle build test case which will pass once openshift/jenkins#442 is merged and a new centos image is pushed.

@bparees
Copy link
Contributor

bparees commented Dec 19, 2017

/lgtm
/retest
(remove the hold when you're happy it's passing)

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Dec 19, 2017
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bparees, jim-minter

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@bparees
Copy link
Contributor

bparees commented Dec 19, 2017

(syntax for unholding is "/hold cancel")

@jim-minter
Copy link
Contributor Author

/retest

@bparees
Copy link
Contributor

bparees commented Jan 4, 2018

extended builds passed.

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 4, 2018
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-ci-robot
Copy link

openshift-ci-robot commented Jan 6, 2018 •
edited
Loading

@jim-minter: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
ci/openshift-jenkins/extended_clusterup bcf5e70 link /test extended_clusterup
ci/openshift-jenkins/extended_image_ecosystem bcf5e70 link /test extended_image_ecosystem

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit 1bcce15 into openshift:master Jan 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sspeiche sspeiche Awaiting requested review from sspeiche

@stevekuznetsov stevekuznetsov Awaiting requested review from stevekuznetsov

Assignees

@bparees bparees

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jim-minter @bparees @openshift-ci-robot @openshift-bot @openshift-merge-robot