-
Type:
Improvement
-
Status: Open
-
Priority:
Major
-
Resolution: Unresolved
-
Affects Version/s: None
-
Fix Version/s: QualifiedToSchedule
-
Component/s: Dev Tools
-
Tags:
-
Backlog priority:1,000
-
Sprint:DevOps-15, DevOps-16
-
Story Points:2
The current behavior is not properly raising errors and the output can be improved:
current output on package upload
$> curl -i -n -F package=@/opt/jenkins/workspace/IT-release-on-demand-build/nuxeo/marketplace/marketplace-template-rendering/package/target/package-6.6.1.zip https://connect-test.nuxeo.com/nuxeo/site/marketplace/upload?batch=true
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0
0 1135k 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0
HTTP/1.1 100 Continue
HTTP/1.1 503 Service Unavailable: Back-end server is at capacity
Content-Length: 0
Connection: keep-alive
Configuration saved: /opt/jenkins/workspace/IT-release-on-demand-build/nuxeo/marketplace/release.ini
$> curl -i -n -F package=@/opt/jenkins/workspace/IT-release-on-demand-build/nuxeo/marketplace/marketplace-template-rendering/package-samples/target/package-samples-6.6.1.zip https://connect-test.nuxeo.com/nuxeo/site/marketplace/upload?batch=true
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0
0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0
0 1500k 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0
HTTP/1.1 100 Continue
HTTP/1.1 503 Service Unavailable: Back-end server is at capacity
Content-Length: 0
Connection: keep-alive
Configuration saved: /opt/jenkins/workspace/IT-release-on-demand-build/nuxeo/marketplace/release.ini
(...)
+ grep uploaded release.ini
(...)
uploaded = https://connect-test.nuxeo.com/nuxeo: /opt/jenkins/workspace/IT-release-on-demand-build/nuxeo/marketplace/marketplace-template-rendering/package-samples/target/package-samples-6.6.1.zip
Issues with current behavior:
- HTTP error 503 was not detected => see -f curl option and the return code 22
- output is not adapted to scripting => same as above
- uploaded only reports the last uploaded package per repository
- uploaded is not very useful as is; change it to provide the remote URL (in case the upload went fine): uploaded = nuxeo/marketplace/marketplace-template-rendering/package-samples/target/package-samples-6.6.1.zip https://nos-test-connect.nos.nuxeo.com/nuxeo/site/marketplace/package/nuxeo-template-rendering-samples?version=6.6.1
Related lines of code: https://github.com/nuxeo/nuxeo/blob/master/scripts/release_mp.py#L147-L149
