microshift icon indicating copy to clipboard operation
microshift copied to clipboard

add user-facing config for ovn-kubernetes CNI

Open zshi-redhat opened this issue 2 years ago • 4 comments

zshi-redhat avatar Sep 16 '22 04:09 zshi-redhat

/test e2e-openshift-conformance-sig-api-machinery

zshi-redhat avatar Sep 16 '22 06:09 zshi-redhat

cc @fzdarsky @mangelajo

zshi-redhat avatar Sep 16 '22 06:09 zshi-redhat

/approve

mangelajo avatar Sep 29 '22 14:09 mangelajo

Putting a hold for the API team to take some time to consider how to handle this new configuration parameter going forward. We are currently working on rewriting the existing configuration to be more in line with OCP so adding a dependency on the existing configuration of MicroShift will be problematic for us.

/hold

dgrisonnet avatar Sep 29 '22 18:09 dgrisonnet

Putting a hold for the API team to take some time to consider how to handle this new configuration parameter going forward. We are currently working on rewriting the existing configuration to be more in line with OCP so adding a dependency on the existing configuration of MicroShift will be problematic for us.

/hold

@dgrisonnet The PR is updated to use a separate config file for ovn-kubernetes.

zshi-redhat avatar Oct 09 '22 03:10 zshi-redhat

/retest

zshi-redhat avatar Oct 31 '22 07:10 zshi-redhat

/jira refresh

zshi-redhat avatar Oct 31 '22 07:10 zshi-redhat

Unholding since the initial concerns were cleared in this thread: https://github.com/openshift/microshift/pull/1030#discussion_r1006118685

/unhold

dgrisonnet avatar Oct 31 '22 17:10 dgrisonnet

Waiting for last comments to be handled, looks good otherwise.

mangelajo avatar Nov 02 '22 09:11 mangelajo

/retest

ggiguash avatar Nov 02 '22 11:11 ggiguash

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mangelajo, zshi-redhat

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

The pull request process is described here

Needs approval from an approver in each of these files:
  • ~~OWNERS~~ [mangelajo,zshi-redhat]

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

openshift-ci[bot] avatar Nov 03 '22 08:11 openshift-ci[bot]

@zshi-redhat: The following test 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/periodic-ocp-4.13-images d5875a2d1ef76fa9076893bc8d7989ceed1542cb link true /test periodic-ocp-4.13-images

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.

openshift-ci[bot] avatar Nov 03 '22 09:11 openshift-ci[bot]

/retest

zshi-redhat avatar Nov 03 '22 09:11 zshi-redhat