WIP: enable CSI Migration only on A/D Controller

[bertinatto]

This PR is a PoC that enabled CSI Migration only on A/D Controller.

Please don't review yet.

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 4d8a6f7|WIP: enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• ce54077|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 403d038|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• d2fb1e8|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 24e2759|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 4026ca9|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 77e1529|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 77e1529|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message
• 94ab506|Separate feature gates from control-plane: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 45df939|Separate feature gates from control-plane: does not specify an upstream backport in the commit message
• c4dfd47|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 286af31|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 4bfd398|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 340f8ea|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[sttts]

I hold back from reviewing. But please note that this has to go into upstream master before merging.

[bertinatto]

/retest

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 2a7f173|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

In response to this:

WIP: enable CSI Migration only on A/D Controller

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.

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits are valid:

• b62f11a|UPSTREAM: 96966: p&f e2e test: log response header for better troubleshooting: the upstream PR kubernetes/kubernetes#96966 has merged
• e0ef263|UPSTREAM: 96984: APF e2e: wait for steady state before proceeding: the upstream PR kubernetes/kubernetes#96984 has merged

The following commits could not be validated and must be approved by a top-level approver:

• 5c1a6ba|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message
• f4b2eca|UPSTREAM: : update test annotations: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• 5594362|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

In response to this:

WIP: enable CSI Migration only on A/D Controller

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.

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• c79fd9c|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bertinatto
To complete the pull request process, please assign mfojtik after the PR has been reviewed.
You can assign the PR to them by writing /assign @mfojtik in a comment when ready.

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

Needs approval from an approver in each of these files:

• DOWNSTREAM_OWNERS

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

[bertinatto]

/retest

[bertinatto]

/retest

[bertinatto]

/test e2e-aws-disruptive

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

• c79fd9c|UPSTREAM: : enable CSI Migration only on A/D Controller: does not specify an upstream backport in the commit message

[openshift-ci-robot]

@bertinatto: the contents of this pull request could not be automatically validated.

The following commits are valid:

• 05b10ad|UPSTREAM: 98699: Balance nodes in scheduling e2e: the upstream PR kubernetes/kubernetes#98699 has merged
• 0c9d5b8|UPSTREAM: 98939: fixes race in TestSyncPodsDeletesWhenSourcesAreReadyPerQOS: the upstream PR kubernetes/kubernetes#98939 has merged
• 10d0d06|UPSTREAM: 98742: Sync completed pods until their containers have been terminated: the upstream PR kubernetes/kubernetes#98742 has merged
• 110e007|UPSTREAM: 98073: (e2e/scheduler) Ensure minimum memory limit in createBalancedPodForNodes: the upstream PR kubernetes/kubernetes#98073 has merged
• 3308cc0|UPSTREAM: 96966: p&f e2e test: log response header for better troubleshooting: the upstream PR kubernetes/kubernetes#96966 has merged
• 3bf8f6b|UPSTREAM: 97819: Move deferred taint cleanup call to ensure all are removed: the upstream PR kubernetes/kubernetes#97819 has merged
• 41a7cea|UPSTREAM: 96984: APF e2e: wait for steady state before proceeding: the upstream PR kubernetes/kubernetes#96984 has merged
• 59f8ace|UPSTREAM: 93861: apiserver: add --permit-address-sharing flag to listen with SO_REUSEADDR: the upstream PR kubernetes/kubernetes#93861 has merged
• 60ee482|UPSTREAM: 99020: shorten scheduling priorities taint key: the upstream PR kubernetes/kubernetes#99020 has merged
• 718da61|UPSTREAM: 99199: delay serving CRD creates for a few seconds for HA consistency: the upstream PR kubernetes/kubernetes#99199 has merged
• a6b53d1|UPSTREAM: 98571: kubelet: Stop probing a pod during graceful shutdown: the upstream PR kubernetes/kubernetes#98571 has merged
• ab27f6e|UPSTREAM: 98933: kubelet: fix create create sandbox delete pod race: the upstream PR kubernetes/kubernetes#98933 has merged
• c2e8787|UPSTREAM: 98956: Fix race when KillPod followed by IsPodPendingTermination: the upstream PR kubernetes/kubernetes#98956 has merged
• d2346f2|UPSTREAM: 96958: kubelet: remove periodic messages from log-level 2: the upstream PR kubernetes/kubernetes#96958 has merged

The following commits could not be validated and must be approved by a top-level approver:

• 01ab7fd|Merge pull request #580 from rphillips/backports/98933: does not specify an upstream backport in the commit message
• 070a94d|Merge pull request #558 from sttts/sttts-ignore-second-signal: does not specify an upstream backport in the commit message
• 1ce30fc|Merge pull request #593 from harche/bug_1885717: does not specify an upstream backport in the commit message
• 26f8888|Merge pull request #564 from rphillips/backports/98938: does not specify an upstream backport in the commit message
• 2ce2be0|Merge pull request #599 from tkashem/pick-96984-4.8: does not specify an upstream backport in the commit message
• 3d798bb|UPSTREAM: : update test annotations: does not specify an upstream backport in the commit message
• 4705571|Merge pull request #526 from damemi/openshift-fix-priority-balancedpods: does not specify an upstream backport in the commit message
• 4d85fcb|Merge pull request #309 from sttts/sttts-so-reuseaddr: does not specify an upstream backport in the commit message
• 52d6e7e|Merge pull request #557 from rphillips/fixes/ignore_not_found_error: does not specify an upstream backport in the commit message
• 59471d9|UPSTREAM: : update apiserver-library-go to clarify SCC failure details: does not specify an upstream backport in the commit message
• 68104e8|Merge pull request #573 from deads2k/apiserver-library: does not specify an upstream backport in the commit message
• 69d7e87|Merge pull request #584 from rphillips/backports/96958: does not specify an upstream backport in the commit message
• 6f8878d|Merge pull request #587 from deads2k/crd-test-patch: does not specify an upstream backport in the commit message
• 9978cc2|UPSTREAM: : bump library-go for metrics authorizer: does not specify an upstream backport in the commit message
• a94d343|UPSTREAM: : ignore not found errors in status messages: does not specify an upstream backport in the commit message
• aa519d9|Merge pull request #588 from deads2k/hardcoded-authorizer: does not specify an upstream backport in the commit message
• ac0db7d|Merge pull request #594 from ehashman/bz-1932097: does not specify an upstream backport in the commit message
• b0f37e6|UPSTREAM: : Enable CSI snapshot e2e tests: does not specify an upstream backport in the commit message
• c3cfb81|Merge pull request #547 from damemi/openshift-balance-nodes-scheduling-ubernetes-lite: does not specify an upstream backport in the commit message
• c4adb51|UPSTREAM: : kube-apiserver: ignore SIGTERM/INT after the first one: does not specify an upstream backport in the commit message
• c67907a|UPSTREAM: : revert ignore not found errors in status messages: does not specify an upstream backport in the commit message
• ca08086|Merge pull request #578 from rphillips/backports/98956: does not specify an upstream backport in the commit message
• cecf9da|UPSTREAM: : use hardcoded metrics scraping authorizer for delegated apiservers: does not specify an upstream backport in the commit message
• d1177da|UPSTREAM: : A/D Controller should rely on Node to decide whether volume is migrated to CSI: does not specify an upstream backport in the commit message
• dfe6c22|Merge pull request #513 from damemi/bz1876918: does not specify an upstream backport in the commit message
• e509061|Merge pull request #555 from jsafrane/enable-snapshot-tests: does not specify an upstream backport in the commit message

[openshift-ci]

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

Test name	Commit	Details	Rerun command
ci/prow/unit	d1177da	link	/test unit
ci/prow/integration	d1177da	link	/test integration
ci/prow/e2e-vsphere	d1177da	link	/test e2e-vsphere
ci/prow/e2e-agnostic-cmd	d1177da	link	/test e2e-agnostic-cmd
ci/prow/e2e-azure	d1177da	link	/test e2e-azure
ci/prow/e2e-aws	d1177da	link	/test e2e-aws
ci/prow/e2e-aws-ovn	d1177da	link	/test e2e-aws-ovn
ci/prow/e2e-aws-multitenant	d1177da	link	/test e2e-aws-multitenant
ci/prow/e2e-gcp	d1177da	link	/test e2e-gcp
ci/prow/e2e-aws-disruptive	d1177da	link	/test e2e-aws-disruptive
ci/prow/e2e-aws-fips	d1177da	link	/test e2e-aws-fips
ci/prow/e2e-aws-csi	d1177da	link	/test e2e-aws-csi
ci/prow/e2e-metal-ipi-ovn-ipv6	d1177da	link	/test e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-metal-ipi	d1177da	link	/test e2e-metal-ipi
ci/prow/e2e-aws-serial	d1177da	link	/test e2e-aws-serial
ci/prow/k8s-e2e-gcp	d1177da	link	/test k8s-e2e-gcp
ci/prow/k8s-e2e-gcp-serial	d1177da	link	/test k8s-e2e-gcp-serial

Full PR test history. Your PR dashboard.

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.

[bertinatto]

We're not going to take this approach anymore.