qaprosoft / qps-pipeline

Jenkins pipeline and jobdsl scripts
11 stars 15 forks source link

provide stable solution for iam auth refresh end-point #863

Closed vdelendik closed 4 years ago

vdelendik commented 4 years ago

https://github.com/qaprosoft/qps-pipeline/blob/a7b85726bdcd03b10025378ea2b7f9fc9e8c5833/src/com/qaprosoft/jenkins/pipeline/integration/zafira/ZafiraClient.groovy#L275

serviceURL.split("reporting-service")[0] + "api/iam"

that's not good code as we register zafira by default using "reporting-service" container name

vdelendik commented 4 years ago
  1. there no anymore dedicated reporting service end-point like /zafira-ws or /reporting-service
  2. we can reuse as serviceUrl for the ZafiraClient just INFRA_HOST url
  3. default reportingServiceUrl parameter value should be updated for RegisterOrganization ticket
vdelendik commented 4 years ago

done. ZafiraClient adjusted in accordance with new end-point. Major note: service url should be registered in short format starting from zebrunner 1.0 (5.2-qps)

ghost commented 4 years ago

Verified. Fixed.