Return updated status back to client on error

components/usermgmt/pkg/server/group.go

@@ -20,7 +20,7 @@ func NewGroupServer(ps service.GroupService) rpcv3.GroupServer {
 func (s *groupServer) CreateGroup(ctx context.Context, p *userpbv3.Group) (*userpbv3.Group, error) {
 	group, err := s.Create(ctx, p)
 	if err != nil {
-		return nil, err
+		return group, err
 	}
 	return group, nil
 }
@@ -28,7 +28,7 @@ func (s *groupServer) CreateGroup(ctx context.Context, p *userpbv3.Group) (*user
 func (s *groupServer) GetGroups(ctx context.Context, p *userpbv3.Group) (*userpbv3.GroupList, error) {
 	groups, err := s.List(ctx, p)
 	if err != nil {
-		return nil, err
+		return groups, err
 	}
 	return groups, nil
 }
@@ -38,7 +38,7 @@ func (s *groupServer) GetGroup(ctx context.Context, p *userpbv3.Group) (*userpbv
 	if err != nil {
 		group, err = s.GetByID(ctx, p.Metadata.Id)
 		if err != nil {
-			return nil, err
+			return group, err
 		}
 	}
 
@@ -56,7 +56,7 @@ func (s *groupServer) DeleteGroup(ctx context.Context, p *userpbv3.Group) (*user
 func (s *groupServer) UpdateGroup(ctx context.Context, p *userpbv3.Group) (*userpbv3.Group, error) {
 	group, err := s.Update(ctx, p)
 	if err != nil {
-		return nil, err
+		return group, err
 	}
 	return group, nil
 }

components/usermgmt/pkg/server/role.go

@@ -20,7 +20,7 @@ func NewRoleServer(ps service.RoleService) rpcv3.RoleServer {
 func (s *roleServer) CreateRole(ctx context.Context, p *userpbv3.Role) (*userpbv3.Role, error) {
 	role, err := s.Create(ctx, p)
 	if err != nil {
-		return nil, err
+		return role, err
 	}
 	return role, nil
 }
@@ -28,7 +28,7 @@ func (s *roleServer) CreateRole(ctx context.Context, p *userpbv3.Role) (*userpbv
 func (s *roleServer) GetRoles(ctx context.Context, p *userpbv3.Role) (*userpbv3.RoleList, error) {
 	roles, err := s.List(ctx, p)
 	if err != nil {
-		return nil, err
+		return roles, err
 	}
 	return roles, nil
 }
@@ -38,7 +38,7 @@ func (s *roleServer) GetRole(ctx context.Context, p *userpbv3.Role) (*userpbv3.R
 	if err != nil {
 		role, err = s.GetByID(ctx, p.Metadata.Id)
 		if err != nil {
-			return nil, err
+			return role, err
 		}
 	}
 
@@ -56,7 +56,7 @@ func (s *roleServer) DeleteRole(ctx context.Context, p *userpbv3.Role) (*userpbv
 func (s *roleServer) UpdateRole(ctx context.Context, p *userpbv3.Role) (*userpbv3.Role, error) {
 	role, err := s.Update(ctx, p)
 	if err != nil {
-		return nil, err
+		return role, err
 	}
 	return role, nil
 }

components/usermgmt/pkg/server/user.go

@@ -20,7 +20,7 @@ func NewUserServer(ps service.UserService) rpcv3.UserServer {
 func (s *userServer) CreateUser(ctx context.Context, p *userpbv3.User) (*userpbv3.User, error) {
 	user, err := s.Create(ctx, p)
 	if err != nil {
-		return nil, err
+		return user, err
 	}
 	return user, nil
 }
@@ -28,7 +28,7 @@ func (s *userServer) CreateUser(ctx context.Context, p *userpbv3.User) (*userpbv
 func (s *userServer) GetUsers(ctx context.Context, p *userpbv3.User) (*userpbv3.UserList, error) {
 	users, err := s.List(ctx, p)
 	if err != nil {
-		return nil, err
+		return users, err
 	}
 	return users, nil
 }
@@ -38,7 +38,7 @@ func (s *userServer) GetUser(ctx context.Context, p *userpbv3.User) (*userpbv3.U
 	// if err != nil {
 	user, err := s.GetByID(ctx, p.Metadata.Id)
 	if err != nil {
-		return nil, err
+		return user, err
 	}
 	// }
 

components/usermgmt/pkg/service/role.go

@@ -159,8 +159,6 @@ func (s *roleService) GetByID(ctx context.Context, id string) (*userv3.Role, err
 }
 
 func (s *roleService) GetByName(ctx context.Context, name string) (*userv3.Role, error) {
-	fmt.Println("name:", name)
-
 	role := &userv3.Role{
 		ApiVersion: apiVersion,
 		Kind:       roleKind,