Skip to content
This repository was archived by the owner on Jun 2, 2025. It is now read-only.

Conversation

xiangjingli
Copy link
Collaborator

@xiangjingli xiangjingli commented Feb 17, 2022

Signed-off-by: Xiangjing Li xiangli@redhat.com

  1. The channel controller is supported to specify an external kubeconfig argument, so that the channel controller running on a management cluster can reconcile resources deployed on the hosted clusters.

  2. Two clients (inCluster Client and outCluster Client) are maintained, so that the channel controller is able to update the webhook configuration service in the hosted cluster by outClient client and update the webhook server service in the management cluster by inCluster client

  3. If the kubeconfig argument is not specified, the channel controller is to use inCluster client by default. This is to align with the current ACM installation.

Signed-off-by: Xiangjing Li <xiangli@redhat.com>
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 17, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: xiangjingli

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:

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

Signed-off-by: Xiangjing Li <xiangli@redhat.com>
Signed-off-by: Xiangjing Li <xiangli@redhat.com>
@xiangjingli xiangjingli changed the title [WIP] be able to run channel controller outside of cluster Allow the channel controller to be able to run outside of cluster Feb 22, 2022
@xiangjingli xiangjingli requested a review from rokej February 22, 2022 22:49
@xiangjingli xiangjingli merged commit d96e3a2 into main Feb 24, 2022
@xiangjingli xiangjingli deleted the hypershiftPrototype branch February 24, 2022 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant