You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@airavata.apache.org by GitBox <gi...@apache.org> on 2020/06/19 18:10:02 UTC
[GitHub] [airavata-custos] virendrawali opened a new issue #79: Error while using delete_tenant python API
virendrawali opened a new issue #79:
URL: https://github.com/apache/airavata-custos/issues/79
## Description
Getting a **"Client ID should not be null"** error while trying to delete a client. Even though I just created the client using **create_admin_client** API
`def test_delete_tenant(tenant):
response = client.delete_tenant(token, tenant.client_id)
print(response)`
## Steps to Reproduce
```
def test_delete_tenant(tenant):
response = client.delete_tenant(token, "custos-al6tf9r1aggetx6nrt1p-10000519")
```
## Error Message
rpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
status = StatusCode.INVALID_ARGUMENT
details = "Error occurred at deleteTenant UNKNOWN: Error while validating method deleteClient Client Id should not be null"
debug_error_string = "{"created":"@1592589585.887821071","description":"Error received from peer ipv4:149.165.168.38:31499","file":"src/core/lib/surface/call.cc","file_line":1056,"grpc_message":"Error occurred at deleteTenant UNKNOWN: Error while validating method deleteClient Client Id should not be null","grpc_status":3}"
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airavata-custos] virendrawali commented on issue #79: Error while using delete_tenant python API
Posted by GitBox <gi...@apache.org>.
virendrawali commented on issue #79:
URL: https://github.com/apache/airavata-custos/issues/79#issuecomment-647603515
Thanks for your reply, Isuru.
I did not know that the tenant has to be activated before it can be deleted. I will check how to activate the tenant and test again.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airavata-custos] virendrawali commented on issue #79: Error while using delete_tenant python API
Posted by GitBox <gi...@apache.org>.
virendrawali commented on issue #79:
URL: https://github.com/apache/airavata-custos/issues/79#issuecomment-647045732
@isururanawaka Could you please look into it?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airavata-custos] virendrawali edited a comment on issue #79: Error while using delete_tenant python API
Posted by GitBox <gi...@apache.org>.
virendrawali edited a comment on issue #79:
URL: https://github.com/apache/airavata-custos/issues/79#issuecomment-647676841
Hi Isuru,
I am able to delete a child tenant but the response returned by the API is empty and I was expecting status == True or something like that. Could you please check if we need to modify the deleteTenant response? Will it be better if we make it like the delete user API?
Here is the code from the TenantManagementService.proto file.
```
rpc deleteTenant (DeleteTenantRequest) returns (google.protobuf.Empty) {
option (google.api.http) = {
delete: "/tenant-management/v1.0.0/oauth2/tenant"
};
}
```
```
rpc deleteUser (org.apache.custos.iam.service.UserSearchRequest) returns (org.apache.custos.iam.service.OperationStatus) {
option (google.api.http) = {
delete: "/user-management/v1.0.0/user"
body: "user"
};
}
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airavata-custos] isururanawaka closed issue #79: Error while using delete_tenant python API
Posted by GitBox <gi...@apache.org>.
isururanawaka closed issue #79:
URL: https://github.com/apache/airavata-custos/issues/79
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airavata-custos] isururanawaka commented on issue #79: Error while using delete_tenant python API
Posted by GitBox <gi...@apache.org>.
isururanawaka commented on issue #79:
URL: https://github.com/apache/airavata-custos/issues/79#issuecomment-647124297
Activated the tenant. Check now
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airavata-custos] virendrawali commented on issue #79: Error while using delete_tenant python API
Posted by GitBox <gi...@apache.org>.
virendrawali commented on issue #79:
URL: https://github.com/apache/airavata-custos/issues/79#issuecomment-647676841
Hi Isuru,
I am able to delete a child tenant but the response returned by the API is empty and I was expecting status == True or something like that. Could you please check if we need to modify the deleteTenant response?
Here is the code from the TenantManagementService.proto file.
```
rpc deleteTenant (DeleteTenantRequest) returns (google.protobuf.Empty) {
option (google.api.http) = {
delete: "/tenant-management/v1.0.0/oauth2/tenant"
};
}
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airavata-custos] virendrawali commented on issue #79: Error while using delete_tenant python API
Posted by GitBox <gi...@apache.org>.
virendrawali commented on issue #79:
URL: https://github.com/apache/airavata-custos/issues/79#issuecomment-649787095
@isururanawaka , do above issue need to fix? I can work on this issue.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org