Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 333077 - update-virgo-build-all.rb script fails when no permissions on repository are granted
Summary: update-virgo-build-all.rb script fails when no permissions on repository are ...
Status: CLOSED FIXED
Alias: None
Product: Virgo
Classification: RT
Component: virgo-build (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 minor (vote)
Target Milestone: 2.2.0.RELEASE   Edit
Assignee: Hristo Iliev CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-22 06:51 EST by Hristo Iliev CLA
Modified: 2011-01-10 04:32 EST (History)
2 users (show)

See Also:


Attachments
Gemini web will not stop the push, but just produce warning (3.01 KB, patch)
2011-01-10 01:46 EST, Hristo Iliev CLA
glyn.normington: review+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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