Module name changed to sigs.k8s.io/ibm-powervs-block-csi-driver

[Madhan-SWE]

• Module name changed to sigs.k8s.io/ibm-powervs-block-csi-driver
• Internal names changed from powervs-csi-driver to ibm-powervs-block-csi-driver

Other fix in Makefile :

• The Version for the image was hardcoded as latest earlier. Now changed as per variable VERSION

[mkumatag]

/lgtm
/approve

[mkumatag]

@Madhan-SWE please change the repo name in the github as well.

[Madhan-SWE]

@mkumatag repo name changed

[mkumatag]

@Rajalakshmi-Girish can you check why PR is not merging even after having lgtm and approval tags

[Rajalakshmi-Girish]

@Rajalakshmi-Girish can you check why PR is not merging even after having lgtm and approval tags

I see below error in tide log:

{"base-sha":"0dc89b554512c7f549c108198c426ff65995472e","branch":"main","component":"tide","controller":"sync","error":"status code 404 not one of [200 405 409], body: {\"message\":\"Not Found\",\"documentation_url\":\https://docs.github.com/rest/reference/pulls#merge-a-pull-request\}","file":"prow/tide/tide.go:1247","func":"k8s.io/test-infra/prow/tide.(*Controller).mergePRs","level":"debug","merge-targets":[72],"msg":"Merge failed.","org":"ppc64le-cloud","pr":72,"repo":"ibm-powervs-block-csi-driver","severity":"debug","sha":"666be324fcd0816dd05983c2210e7527811dfd68","time":"2022-01-04T09:29:58Z"}

404 might be due to change of org name from powervs-csi-driver to ibm-powervs-block-csi-driver while the PR review is in progress..
@mkumatag Can we try re-creating a new PR and check?

[mkumatag]

/meow

[ppc64le-cloud-bot]

@mkumatag:

In response to this:

/meow

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.

[mkumatag]

/approve

[ppc64le-cloud-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Madhan-SWE, mkumatag

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 [mkumatag]

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

[mkumatag]

@Madhan-SWE can you please resolve the conflicts?

[ppc64le-cloud-bot]

New changes are detected. LGTM label has been removed.

[ppc64le-cloud-bot]

@Madhan-SWE: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

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.

[Madhan-SWE]

Resolving conflicts from UI creates merge commits and that leads to do-not-merge label addition.
Created a new PR with proper rebase: #75

[mkumatag]

merged via #75 , hence can be closed
/close

[ppc64le-cloud-bot]

@mkumatag: Closed this PR.

In response to this:

merged via #75 , hence can be closed
/close

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.