Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR integration race condition? #8871

Closed
davidlange6 opened this issue Apr 24, 2015 · 3 comments
Closed

PR integration race condition? #8871

davidlange6 opened this issue Apr 24, 2015 · 3 comments

Comments

@davidlange6
Copy link
Contributor

The CMSSW_7_5_X_2015-04-23-2300 build appears to have picked up only part of #8833, causing some run time problems. I guess there is a race condition either on our side or on the git hub side that brought in only part of a commit. The repo is consistent now, so it was a transient effect.

Anything that we can do about this beyond not accepting PRs at 10:59:59.99999?

@ktf, @smuzaffar, @nclopezo @Degano

@davidlt
Copy link
Contributor

davidlt commented Apr 24, 2015

Git itself cannot have inconsistent state. Maybe its -x ours? That is merge conflict, and -x ours made a mess out of it.

@nclopezo
Copy link
Contributor

It seems that it was caused by the porting of #8805 which contains 7be30ee

@davidlange6
Copy link
Contributor Author

On Apr 24, 2015, at 1:55 PM, davidlt notifications@github.com wrote:

Git itself cannot have inconsistent state. Maybe its -x ours? That is merge conflict, and -x ours made a mess out of it.

to be clear - the state of the repo is now fully consistent. - just not at the moment that the IB was triggered.


Reply to this email directly or view it on GitHub.

@ghost ghost closed this as completed Aug 21, 2015
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants