Skip to content

Conversation

@hongkailiu
Copy link
Member

@hongkailiu hongkailiu commented Feb 3, 2020

https://issues.redhat.com/browse/DPTP-704

python scripts used in the commits:
https://github.com/hongkailiu/dptp-doc/tree/master/hack

/cc @openshift/openshift-team-developer-productivity-test-platform

@openshift-ci-robot openshift-ci-robot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Feb 3, 2020
@hongkailiu
Copy link
Member Author

/hold

require openshift/ci-tools#341

@openshift-ci-robot openshift-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Feb 3, 2020
@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 3, 2020

See if we still need c9ec53e

Seems that we still need to increase the MEM even for 4.3.

#1g as 3.11
oc get pod -n ci-op-x208b3yg --context build01
NAME                           READY   STATUS      RESTARTS   AGE
artifacts-build                0/1     OOMKilled   0          5m42s

Will update https://bugzilla.redhat.com/show_bug.cgi?id=1784163


Update OOMKilled about is fixed by e3a07db

Still need to change requests though.

Screen Shot 2020-02-03 at 2 37 57 PM

Screen Shot 2020-02-03 at 4 02 36 PM

@hongkailiu
Copy link
Member Author

/test all

@hongkailiu hongkailiu force-pushed the migrate_origin_master branch 3 times, most recently from a06841b to 906f4d8 Compare February 3, 2020 19:37
@openshift-ci-robot openshift-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Feb 3, 2020
@hongkailiu hongkailiu force-pushed the migrate_origin_master branch from 906f4d8 to 75b1f44 Compare February 3, 2020 19:56
@hongkailiu hongkailiu force-pushed the migrate_origin_master branch from 75b1f44 to a03cdec Compare February 3, 2020 20:18
@hongkailiu
Copy link
Member Author

/test all

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 3, 2020
@hongkailiu hongkailiu force-pushed the migrate_origin_master branch 3 times, most recently from c91b518 to 5b52d35 Compare February 4, 2020 02:31
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 4, 2020
@hongkailiu hongkailiu force-pushed the migrate_origin_master branch 3 times, most recently from 0479474 to 211e19a Compare February 4, 2020 02:49
@openshift-ci-robot openshift-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Feb 4, 2020
@hongkailiu hongkailiu force-pushed the migrate_origin_master branch from b4937e4 to e3e07c4 Compare February 4, 2020 03:09
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 4, 2020
@hongkailiu hongkailiu force-pushed the migrate_origin_master branch 2 times, most recently from 2d0ca90 to af21c5b Compare February 4, 2020 20:03
Copy link
Member Author

Choose a reason for hiding this comment

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

We keep this job on api.ci because 6G of Mem is going to hurt our scale up as discussed.
And the rehearsal is indeed on default:
https://prow.svc.ci.openshift.org/prowjob?prowjob=905d78bf-4789-11ea-b42d-0a58ac103b5d

@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 4, 2020
…elease_repo} -name "*openshift-origin-master-presubmits.yaml" -exec python3 hack/migrate_non_prowgen_jobs.py {} \;'
…elease_repo} -name "*openshift-origin-master-presubmits.yaml" -exec python3 hack/duplicate_prowgen_jobs_on_build01.py {} \;'
@hongkailiu hongkailiu force-pushed the migrate_origin_master branch from af21c5b to bc52bcf Compare February 4, 2020 21:48
@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 5, 2020

@hongkailiu hongkailiu force-pushed the migrate_origin_master branch from 457bbfa to b67e8b8 Compare February 5, 2020 15:58
Copy link
Contributor

@stevekuznetsov stevekuznetsov left a comment

Choose a reason for hiding this comment

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

/hold

Copy link
Contributor

Choose a reason for hiding this comment

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

Why?

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 5, 2020
@hongkailiu hongkailiu force-pushed the migrate_origin_master branch from b67e8b8 to 2472360 Compare February 5, 2020 16:35
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 5, 2020
@openshift-ci-robot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hongkailiu, stevekuznetsov

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hongkailiu
Copy link
Member Author

/test all

@openshift-bot
Copy link
Contributor

/test prow-config-filenames

@openshift-ci-robot
Copy link
Contributor

@hongkailiu: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/ordered-prow-config 2472360 link /test ordered-prow-config
ci/prow/generated-config 2472360 link /test generated-config
ci/rehearse/openshift/origin/master/e2e-azure 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-aws-disruptive 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-aws-image-registry 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-aws 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-aws-jenkins 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-aws-serial 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-conformance-k8s 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-gcp-builds 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-cmd 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-gcp 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-gcp-image-ecosystem 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-gcp-upgrade 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-vsphere 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/launch-vsphere 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-aws-fips 2472360 link /test pj-rehearse
ci/rehearse/openshift/origin/master/e2e-aws-ovn 2472360 link /test pj-rehearse
ci/prow/pj-rehearse 2472360 link /test pj-rehearse
ci/prow/prow-config-filenames 2472360 link /test prow-config-filenames

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.

Details

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-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 13, 2020
@openshift-ci-robot
Copy link
Contributor

@hongkailiu: PR needs rebase.

Details

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.

@hongkailiu
Copy link
Member Author

/close

@openshift-ci-robot
Copy link
Contributor

@hongkailiu: Closed this PR.

Details

In response to this:

/close

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.

@hongkailiu hongkailiu deleted the migrate_origin_master branch June 12, 2024 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants