Skip to content

Conversation

@sameshai
Copy link
Member

@sameshai sameshai commented May 29, 2024

If for some reason the File share is created but target creation fails. Lets retry creating target so that we have way to recover the file share created.

Results : https://github.ibm.com/alchemy-containers/armada-storage/issues/6106

…hare creation fails

Signed-off-by: Sameer Shaikh <sameer.shaikh@ibm.com>
@sameshai sameshai changed the title Add support for VolumeAccessPoint(FileShareTarget Creation) if file s… Add support for VolumeAccessPoint(FileShareTarget Creation) if volume creation fails but creates (File share) May 29, 2024
GunaKKIBM
GunaKKIBM previously approved these changes May 29, 2024
@contsto
Copy link

contsto commented May 29, 2024

👍 Coverage increased from [%] to [82.7875%]

@contsto
Copy link

contsto commented May 29, 2024

👍 Coverage increased from [%] to [83.3%]

…hare creation fails

Signed-off-by: Sameer Shaikh <sameer.shaikh@ibm.com>
@contsto
Copy link

contsto commented May 30, 2024

👍 Coverage increased from [%] to [83%]

…hare creation fails

Signed-off-by: Sameer Shaikh <sameer.shaikh@ibm.com>
@contsto
Copy link

contsto commented May 30, 2024

👍 Coverage increased from [%] to [%]

1 similar comment
@contsto
Copy link

contsto commented May 30, 2024

👍 Coverage increased from [%] to [%]

Signed-off-by: Sameer Shaikh <sameer.shaikh@ibm.com>
@contsto
Copy link

contsto commented May 30, 2024

👍 Coverage increased from [%] to [82.875%]

Signed-off-by: Sameer Shaikh <sameer.shaikh@ibm.com>
@contsto
Copy link

contsto commented May 30, 2024

👍 Coverage increased from [%] to [82.675%]

Signed-off-by: Sameer Shaikh <sameer.shaikh@ibm.com>
@contsto
Copy link

contsto commented May 30, 2024

👍 Coverage increased from [%] to [83.3125%]

prankulmahajan
prankulmahajan previously approved these changes May 30, 2024
Copy link
Member

@prankulmahajan prankulmahajan left a comment

Choose a reason for hiding this comment

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

lgtm

Signed-off-by: Sameer Shaikh <sameer.shaikh@ibm.com>
@sameshai sameshai requested a review from prankulmahajan May 30, 2024 10:14
Copy link
Member

@prankulmahajan prankulmahajan left a comment

Choose a reason for hiding this comment

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

lgtm

@contsto
Copy link

contsto commented May 30, 2024

👍 Coverage increased from [%] to [83.1%]

Copy link
Collaborator

@arahamad arahamad left a comment

Choose a reason for hiding this comment

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

lgtm

@arahamad arahamad merged commit d6cfbba into master May 30, 2024
@arahamad arahamad deleted the seperfix branch October 15, 2024 12:38
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.

6 participants