Skip to content

WIP: code base snapshot to be moved to NTO#862

Closed
yanirq wants to merge 1 commit intoopenshift-kni:masterfrom
yanirq:move_pao
Closed

WIP: code base snapshot to be moved to NTO#862
yanirq wants to merge 1 commit intoopenshift-kni:masterfrom
yanirq:move_pao

Conversation

@yanirq
Copy link
Member

@yanirq yanirq commented Mar 13, 2022

This is a snapshot of the PAO code to be moved to NTO.

Removed:

  • Most of the OLM related parts
  • Must gather
  • CI cluster and config kustomized yamls
  • Scripts that will changed drastically form 4.10 and below
  • Build files

Kept:

  • Performance profile creator
  • Vendor (just for this PR not to be large)

This PR should NOT be merged

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 13, 2022
@yanirq
Copy link
Member Author

yanirq commented Mar 13, 2022

/hold - do not merge this PR

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 13, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: yanirq

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 openshift-ci bot 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 Mar 13, 2022
@yanirq
Copy link
Member Author

yanirq commented Mar 13, 2022

@ffromani
Copy link
Member

This is a snapshot of the PAO code to be moved to NTO.

Removed:

* Most of the OLM related parts

* Must gather

* CI cluster and config kustomized yamls

* Scripts that will changed drastically form 4.10 and below

* Build files

Kept:

* Performance profile creator

* Vendor (just for this PR not to be large)

This PR should NOT be merged

So this is to demonstrate the code which we want to move to NTO? The goal of the reviews is to verify that's the subset we need, or do you expect something more/different?

@yanirq
Copy link
Member Author

yanirq commented Mar 14, 2022

So this is to demonstrate the code which we want to move to NTO? The goal of the reviews is to verify that's the subset we need, or do you expect something more/different?

Yes, the goal is to see if there are redundancies or missing parts before copying it over to NTO.
It will be easier to have a snapshot of the code before the copy and movement of paths in NTO.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 14, 2022

@yanirq: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/ci-ppc aba41fb link false /test ci-ppc
ci/prow/ci aba41fb link true /test ci
ci/prow/e2e-render-cmd aba41fb link true /test e2e-render-cmd
ci/prow/ci-tools aba41fb link false /test ci-tools
ci/prow/e2e-gcp-operator-upgrade aba41fb link true /test e2e-gcp-operator-upgrade
ci/prow/ci-index aba41fb link true /test ci-index
ci/prow/images aba41fb link true /test images
ci/prow/e2e-gcp aba41fb link true /test e2e-gcp

Full PR test history. Your PR dashboard.

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.

@yanirq
Copy link
Member Author

yanirq commented Apr 13, 2022

PAO move to NTO is complete

@yanirq yanirq closed this Apr 13, 2022
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. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments