Skip to content

Conversation

@ycyaoxdu
Copy link
Member

@ycyaoxdu ycyaoxdu commented Oct 8, 2022

Signed-off-by: ycyaoxdu yaoyuchen0626@163.com

@openshift-ci openshift-ci bot requested review from qiujian16 and yue9944882 October 8, 2022 06:01
@ycyaoxdu
Copy link
Member Author

ycyaoxdu commented Oct 8, 2022

related issue #186

@ycyaoxdu ycyaoxdu changed the title allow install hub-addon specify ns [wip]allow install hub-addon specify ns Oct 8, 2022
@ycyaoxdu ycyaoxdu force-pushed the enable-install-hub-addon-namespace branch 3 times, most recently from d0ac49e to fc17b85 Compare October 8, 2022 08:56
@ycyaoxdu ycyaoxdu changed the title [wip]allow install hub-addon specify ns allow install hub-addon specify ns Oct 8, 2022
@ycyaoxdu
Copy link
Member Author

ycyaoxdu commented Oct 8, 2022

/assign @qiujian16

if err != nil {
return err
}
_, err = kubeClient.CoreV1().Namespaces().Get(context.TODO(), o.values.Namespace, metav1.GetOptions{})
Copy link
Member

Choose a reason for hiding this comment

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

this check is not necessary

Signed-off-by: ycyaoxdu <yaoyuchen0626@163.com>
@ycyaoxdu ycyaoxdu force-pushed the enable-install-hub-addon-namespace branch from fc17b85 to dc103e0 Compare October 9, 2022 03:41
@qiujian16
Copy link
Member

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm label Oct 9, 2022
@openshift-ci
Copy link

openshift-ci bot commented Oct 9, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: qiujian16, ycyaoxdu

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

@openshift-ci openshift-ci bot added the approved label Oct 9, 2022
@openshift-merge-robot openshift-merge-robot merged commit c7fa5a7 into open-cluster-management-io:main Oct 9, 2022
@ycyaoxdu ycyaoxdu deleted the enable-install-hub-addon-namespace branch October 13, 2022 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants