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

Set a git user and e-mail for the CI rebase. #2535

Merged
merged 1 commit into from
Aug 15, 2018

Conversation

bowlofeggs
Copy link
Contributor

CI began to fail recently at the git rebase step with this message:

  • git rebase --preserve-merges origin/develop

*** Please tell me who you are.

Run

git config --global user.email "[email protected]"
git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'bodhi@slave03.(none)')
You need to set your committer info first

This commit sets the e-mail and name so that git will be happy with
the rebase.

Signed-off-by: Randy Barlow [email protected]

CI began to fail recently at the git rebase step with this message:

+ git rebase --preserve-merges origin/develop

*** Please tell me who you are.

Run

  git config --global user.email "[email protected]"
  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'bodhi@slave03.(none)')
You need to set your committer info first

This commit sets the e-mail and name so that git will be happy with
the rebase.

Signed-off-by: Randy Barlow <[email protected]>
@bowlofeggs bowlofeggs added Critical We can't go on living in this sqalor, drop everything and fix it! Tests Issues pertaining to Bodhi's tests Crash Issues related to an unhandled crash labels Aug 15, 2018
@bowlofeggs bowlofeggs requested a review from a team as a code owner August 15, 2018 12:51
@fedora-infra fedora-infra deleted a comment from centos-ci Aug 15, 2018
@bowlofeggs
Copy link
Contributor Author

This PR will not pass CI before it is merged, because it edits the script that CI runs. The CI system runs the merged version of the script, not the proposed version, so there is no way to get this PR to pass CI even though it should fix the CI failures.

Once it is merged, I should be able to rebase all the open PRs to get them passing.

@bowlofeggs bowlofeggs merged commit a0a65ef into fedora-infra:develop Aug 15, 2018
@bowlofeggs bowlofeggs deleted the ci-set_git_user branch August 15, 2018 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Crash Issues related to an unhandled crash Critical We can't go on living in this sqalor, drop everything and fix it! Tests Issues pertaining to Bodhi's tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants