Skip to content

Add OpenStack cloud with ppc64le compute nodes inside Red Hat for 4.3 CI testing #8271

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
Merged
openshift-merge-robot merged 2 commits into from
May 5, 2020
Merged

Add OpenStack cloud with ppc64le compute nodes inside Red Hat for 4.3 CI testing #8271

openshift-merge-robot merged 2 commits into from
May 5, 2020

Conversation

@hamzy
Copy link
Contributor

@hamzy hamzy commented Apr 9, 2020

Add OpenStack cloud with ppc64le compute nodes inside Red Hat for 4.3 CI testing

@hamzy hamzy force-pushed the openstack-43-ppc64le-ci branch 4 times, most recently from fcbf825 to d493ee1 Compare April 9, 2020 21:23
@hamzy
Copy link
Contributor Author

hamzy commented Apr 9, 2020

/test pj-rehearse

@hamzy hamzy force-pushed the openstack-43-ppc64le-ci branch 8 times, most recently from dd6e960 to 05c4da2 Compare April 13, 2020 21:34
@hamzy
Copy link
Contributor Author

hamzy commented Apr 13, 2020

/test pj-rehearse

@hamzy hamzy force-pushed the openstack-43-ppc64le-ci branch 11 times, most recently from 3dff673 to ad07547 Compare April 16, 2020 00:37
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 16, 2020
@hamzy hamzy force-pushed the openstack-43-ppc64le-ci branch from ad07547 to 08fb41c Compare April 16, 2020 00:46
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 16, 2020
@crawford
Copy link
Contributor

crawford commented May 1, 2020

/approve

@hamzy hamzy force-pushed the openstack-43-ppc64le-ci branch 2 times, most recently from 24af677 to 9ac7dd3 Compare May 1, 2020 23:52
@hamzy
Copy link
Contributor Author

hamzy commented May 2, 2020

/test pj-rehearse

1 similar comment
@hamzy
Copy link
Contributor Author

hamzy commented May 2, 2020

/test pj-rehearse

@hamzy hamzy force-pushed the openstack-43-ppc64le-ci branch 3 times, most recently from a507474 to 702e554 Compare May 2, 2020 20:31
@hamzy
Copy link
Contributor Author

hamzy commented May 3, 2020

/hold

@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 May 3, 2020
@hamzy hamzy force-pushed the openstack-43-ppc64le-ci branch from 702e554 to ed7292e Compare May 4, 2020 13:20
@hamzy
Copy link
Contributor Author

hamzy commented May 4, 2020

/test pj-rehearse

1 similar comment
@hamzy
Copy link
Contributor Author

hamzy commented May 4, 2020

/test pj-rehearse

@hamzy
Copy link
Contributor Author

hamzy commented May 4, 2020

/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 May 4, 2020
@hamzy
Copy link
Contributor Author

hamzy commented May 4, 2020

/assign @petr-muller

@manojnkumar
Copy link
Contributor

manojnkumar commented May 4, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 4, 2020
petr-muller
petr-muller reviewed May 5, 2020
View reviewed changes
Copy link
Member

@petr-muller petr-muller left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Needs a few tweaks to adapt to recent changes, LGTM otherwise.

@hamzy @petr-muller
Apply suggestions from code review
9338510 
Taking core-services suggestions from Petr

Co-authored-by: Petr Muller <[email protected]>
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label May 5, 2020
@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 5, 2020
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented May 5, 2020

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: crawford, hamzy, manojnkumar, petr-muller

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

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 5, 2020
@openshift-merge-robot openshift-merge-robot merged commit eb75d4b into openshift:master May 5, 2020
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented May 5, 2020

@hamzy: Updated the following 4 configmaps:

  • job-config-4.3 configmap in namespace ci at cluster api.ci using the following files:
    • key openshift-release-release-4.3-periodics.yaml using file ci-operator/jobs/openshift/release/openshift-release-release-4.3-periodics.yaml
  • job-config-4.3 configmap in namespace ci at cluster app.ci using the following files:
    • key openshift-release-release-4.3-periodics.yaml using file ci-operator/jobs/openshift/release/openshift-release-release-4.3-periodics.yaml
  • plugins configmap in namespace ci at cluster api.ci using the following files:
    • key plugins.yaml using file core-services/prow/02_config/_plugins.yaml
  • plugins configmap in namespace ci at cluster app.ci using the following files:
    • key plugins.yaml using file core-services/prow/02_config/_plugins.yaml
Details

In response to this:

Add OpenStack cloud with ppc64le compute nodes inside Red Hat for 4.3 CI testing

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.

@wking wking mentioned this pull request Jun 19, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Prashanth684 Prashanth684 Prashanth684 left review comments

@droslean droslean Awaiting requested review from droslean

@jhixson74 jhixson74 Awaiting requested review from jhixson74

+3 more reviewers

@crawford crawford crawford left review comments

@jaypoulz jaypoulz jaypoulz left review comments

@petr-muller petr-muller petr-muller approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@crawford crawford

@petr-muller petr-muller

@manojnkumar manojnkumar

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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@hamzy @tbreeds @manojnkumar @crawford @openshift-ci-robot @petr-muller @jaypoulz @Prashanth684 @openshift-merge-robot