Always add member instead
[ZITADOPER-5]
This commit is contained in:
@@ -199,20 +199,6 @@ func (wr *wrappedOrganizationReconciler) reconcileInitialAdmin(ctx context.Conte
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if adminClient != nil {
|
|
||||||
{
|
|
||||||
if _, err := adminClient.AddIAMMember(ctx, &adm.AddIAMMemberRequest{
|
|
||||||
UserId: userid,
|
|
||||||
Roles: []string{
|
|
||||||
"IAM_OWNER",
|
|
||||||
},
|
|
||||||
}); err != nil {
|
|
||||||
if !strings.Contains(err.Error(), "RolesNotChanged") {
|
|
||||||
return fmt.Errorf("Error adding org member: %v", err)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
} else {
|
} else {
|
||||||
userid = adminUser.User.Id
|
userid = adminUser.User.Id
|
||||||
@@ -231,6 +217,18 @@ func (wr *wrappedOrganizationReconciler) reconcileInitialAdmin(ctx context.Conte
|
|||||||
}
|
}
|
||||||
|
|
||||||
if adminClient != nil {
|
if adminClient != nil {
|
||||||
|
{
|
||||||
|
if _, err := adminClient.AddIAMMember(ctx, &adm.AddIAMMemberRequest{
|
||||||
|
UserId: userid,
|
||||||
|
Roles: []string{
|
||||||
|
"IAM_OWNER",
|
||||||
|
},
|
||||||
|
}); err != nil {
|
||||||
|
if !strings.Contains(err.Error(), "RolesNotChanged") || !strings.Contains(err.Error(), "AlreadyExists") {
|
||||||
|
return fmt.Errorf("Error adding org member: %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
{
|
{
|
||||||
if _, err := adminClient.UpdateIAMMember(ctx, &adm.UpdateIAMMemberRequest{
|
if _, err := adminClient.UpdateIAMMember(ctx, &adm.UpdateIAMMemberRequest{
|
||||||
UserId: userid,
|
UserId: userid,
|
||||||
|
|||||||
Reference in New Issue
Block a user