Refactored DeleteSecret.

This commit is contained in:
Aleksandr Melnikov
2020-02-06 14:37:43 -08:00
parent f0265caf85
commit f99260fa93
3 changed files with 12 additions and 9 deletions

View File

@@ -64,12 +64,8 @@ func (c *Client) ListSecrets(namespace string) (secrets []*model.Secret, err err
return
}
func (c *Client) DeleteSecret(namespace string, name string) (deleted bool, err error) {
err = c.CoreV1().Secrets(namespace).Delete(name, &metav1.DeleteOptions{})
if err != nil {
return false, err
}
return true, nil
func (c *Client) DeleteSecret(namespace string, secret *model.Secret) (err error) {
return c.CoreV1().Secrets(namespace).Delete(secret.Name, &metav1.DeleteOptions{})
}
func (c *Client) DeleteSecretKey(namespace string, name string, key string) (deleted bool, err error) {

View File

@@ -70,7 +70,14 @@ func (r *ResourceManager) ListSecrets(namespace string) (secrets []*model.Secret
}
func (r *ResourceManager) DeleteSecret(namespace string, name string) (deleted bool, err error) {
return r.kubeClient.DeleteSecret(namespace, name)
secret := &model.Secret{
Name: name,
}
err = r.kubeClient.DeleteSecret(namespace, secret)
if err != nil {
return false, util.NewUserError(codes.Unknown, "Secret unable to be deleted.")
}
return true, nil
}
func (r *ResourceManager) DeleteSecretKey(namespace string, name string, key string) (deleted bool, err error) {

View File

@@ -79,10 +79,10 @@ func (s *SecretServer) ListSecrets(ctx context.Context, req *api.ListSecretsRequ
func (s *SecretServer) DeleteSecret(ctx context.Context, req *api.DeleteSecretRequest) (deleted *api.DeleteSecretResponse, err error) {
var isDeleted bool
isDeleted, err = s.resourceManager.DeleteSecret(req.Namespace, req.Name)
if err != nil {
if errors.As(err, &userError) {
return &api.DeleteSecretResponse{
Deleted: false,
}, util.NewUserError(codes.Unknown, err.Error())
}, userError.GRPCError()
}
return &api.DeleteSecretResponse{
Deleted: isDeleted,