karmada icon indicating copy to clipboard operation
karmada copied to clipboard

Check file path before Initialize crd

Open my-git9 opened this issue 3 years ago • 4 comments

What type of PR is this? /kind bug

What this PR does / why we need it: 1.When we run karmadactl init --kube-image-registry=registry.cn-hangzhou.aliyuncs.com/google_containers --crds /root/crds.tar.gz 2.If the crds dir do not has patches or bases, like

$ ls crds
crd.yaml  net.yaml
  1. The output: image

  2. What do we expect: Command output file does not exist, and exit

Which issue(s) this PR fixes: Fixes #

Special notes for your reviewer: The test result: image

Does this PR introduce a user-facing change?:

Check if the file path exists before Initialize crd.

my-git9 avatar Jul 17 '22 05:07 my-git9

/cc @lonelyCZ @RainbowMango

my-git9 avatar Jul 18 '22 01:07 my-git9

/cc @prodanlabs

my-git9 avatar Aug 01 '22 12:08 my-git9

/assign @prodanlabs

my-git9 avatar Aug 01 '22 12:08 my-git9

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please ask for approval from prodanlabs after the PR has been reviewed.

The full list of commands accepted by this bot can be found 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

karmada-bot avatar Aug 02 '22 07:08 karmada-bot

Sorry for letting this sit. I think this is a reasonable improvement. @my-git9 would you like to rebase and re-open this PR? You can just assign it to me.

RainbowMango avatar Jul 12 '23 03:07 RainbowMango

Sorry for letting this sit. I think this is a reasonable improvement. @my-git9 would you like to rebase and re-open this PR? You can just assign it to me.

Sure, thanks

my-git9 avatar Jul 12 '23 09:07 my-git9

/assign @RainbowMango

my-git9 avatar Jul 12 '23 09:07 my-git9

Yes, it seems that this PR has a lot of limitations

my-git9 avatar Jul 17 '23 11:07 my-git9

/close

Please reopen it if you think we still need it.

RainbowMango avatar Apr 26 '24 07:04 RainbowMango

@RainbowMango: Closed this PR.

In response to this:

/close

Please reopen it if you think we still need it.

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.

karmada-bot avatar Apr 26 '24 07:04 karmada-bot