search

Boehringer-Ingelheim / toscaci

tosca command line tool and service that simplify tosca integration with Continuous Integration tools
Apache License 2.0
10 stars 7 forks source link

CLI should return (dump) remote workspace location when done with execution #48

Open clemensutschig opened 2 years ago

clemensutschig commented 2 years ago

In order to debug any errors easier from the client (edp side) - we should print out the remote workspace location from the CLI -so people just need to go to the agent vm and know where to go :)=

segator commented 2 years ago

the API already returns the project workspace path, you only need to bind it on the struct and log on execution. https://github.com/Boehringer-Ingelheim/toscaci/blob/6239ea8fd4ea92303202217c428a871aade2fdbf/client/src/go/toscactl/tosca/provider.go#L42

Also the current workspace session ID(already in logs in the cli is just a base64 of the workspace path)

clemensutschig commented 2 years ago

isaac - cool. It looks like it's already returned on the server side:

https://github.com/Boehringer-Ingelheim/toscaci/blob/6239ea8fd4ea92303202217c428a871aade2fdbf/server/CIService/Service/WorkspaceService.cs#L29

so the only thing needed is the json binding - correct?

segator commented 2 years ago

yes

clemensutschig commented 2 years ago

albert - can you take this up .. not super urgent - but helpfull