Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 333077

Summary: update-virgo-build-all.rb script fails when no permissions on repository are granted
Product: [RT] Virgo Reporter: Hristo Iliev <hsiliev>
Component: virgo-buildAssignee: Hristo Iliev <hsiliev>
Status: CLOSED FIXED QA Contact:
Severity: minor    
Priority: P3 CC: eclipse, glyn.normington
Version: unspecified   
Target Milestone: 2.2.0.RELEASE   
Hardware: PC   
OS: All   
Whiteboard:
Attachments:
Description Flags
Gemini web will not stop the push, but just produce warning glyn.normington: review+

Description Hristo Iliev CLA 2010-12-22 06:51:01 EST
update-virgo-build-all.rb script fails on repositories where the commiter does not have permissions:

Pushing gemini-web-container
Counting objects: 3, done.
Delta compression using up to 2 threads.
Compressing objects: 100% (2/2), done.
Writing objects: 100% (2/2), 283 bytes, done.
Total 2 (delta 1), reused 0 (delta 0)
error: insufficient permission for adding an object to repository database ./objects

fatal: failed to write object
error: unpack failed: unpack-objects abnormal exit
To ssh://hiliev@git.eclipse.org/gitroot/gemini.web/org.eclipse.gemini.web.gemini-web-container.git
 ! [remote rejected] master -> master (n/a (unpacker error))
error: failed to push some refs to 'ssh://hiliev@git.eclipse.org/gitroot/gemini.web/org.eclipse.gemini.web.gemini-web-container.git'
Execution Failed

The script should push only the repos where the committer has permissions and skip (with warning/error?) the rest.

As a workaround the script can be edited to contain only the repositories that the committer has access to.
Comment 1 Glyn Normington CLA 2010-12-23 05:39:16 EST
I suggest that the script should issue a warning message if it hits this problem on Gemini Web (a special case admittedly) and issue an error message (and probably fail at the end with a bad return code) if it hits this problem on any Virgo git repo.
Comment 2 Hristo Iliev CLA 2011-01-10 01:46:52 EST
Created attachment 186358 [details]
Gemini web will not stop the push, but just produce warning
Comment 3 Glyn Normington CLA 2011-01-10 03:43:27 EST
Comment on attachment 186358 [details]
Gemini web will not stop the push, but just produce warning

Reviewed
Comment 4 Hristo Iliev CLA 2011-01-10 04:32:01 EST
Commit 5d3052652b91cc54fd4e4419aba2703b43c0933f