Skip to content

controller/replication: add UnimplementedError for getVolRepInfo()#295

Merged
mergify[bot] merged 1 commit into
csi-addons:mainfrom
Rakshith-R:check-unimplemented-error
Feb 7, 2023
Merged

controller/replication: add UnimplementedError for getVolRepInfo()#295
mergify[bot] merged 1 commit into
csi-addons:mainfrom
Rakshith-R:check-unimplemented-error

Conversation

@Rakshith-R

Copy link
Copy Markdown
Member

This commit adds code to handle UnimplmentedError from getVolumeReplicationInfo rpc. The controller wil now ignore this error and not requeue the request.

Fixes: #236

Signed-off-by: Rakshith R rar@redhat.com

@mergify mergify Bot requested a review from Yuggupta27 February 6, 2023 09:24
Madhu-1
Madhu-1 previously approved these changes Feb 6, 2023
Comment thread internal/util/error.go Outdated
return s.Message()
}

// IsUmplementedError returns true if the error is Unimplemented error.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

IsUmplementedError-> IsUnimplementedError

This commit adds code to handle UnimplmentedError from
getVolumeReplicationInfo rpc. The controller wil now
ignore this error and not requeue the request.

Fixes: csi-addons#236

Signed-off-by: Rakshith R <rar@redhat.com>
@Rakshith-R Rakshith-R force-pushed the check-unimplemented-error branch from ba70a84 to 8f2a31d Compare February 7, 2023 04:02
@Rakshith-R Rakshith-R requested a review from yati1998 February 7, 2023 04:03
@mergify mergify Bot dismissed Madhu-1’s stale review February 7, 2023 04:03

Pull request has been modified.

@Rakshith-R Rakshith-R requested a review from Madhu-1 February 7, 2023 04:03
@mergify mergify Bot merged commit 90f59d1 into csi-addons:main Feb 7, 2023
Nikhil-Ladha pushed a commit to Nikhil-Ladha/kubernetes-csi-addons that referenced this pull request Jun 26, 2025
[release-4.19] DFBUGS-2424: check vgrcontent before removing finalizer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle non implemented error for getVolumeReplicationInfo RPC

3 participants