Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Include user in git commit info #695

Closed
leth opened this issue Aug 9, 2017 · 4 comments
Closed

Include user in git commit info #695

leth opened this issue Aug 9, 2017 · 4 comments

Comments

@leth
Copy link
Contributor

leth commented Aug 9, 2017

A thought that just occured to me (sorry if I missed an issue for it already):
For user-requesed changes, I'd be great if we could set the commit author to the user, and the commit committer to flux-bot.

@squaremo
Copy link
Member

squaremo commented Aug 9, 2017

An adjacent idea: get the username sent by fluxctl from git config (currently it uses the logged-in user).

@squaremo
Copy link
Member

squaremo commented Sep 7, 2017

@leth To check my interpretation:

  • for automated upgrades, use flux-bot as the committer and author
  • for user-initiated changes (releases, lock/unlock, automate/deautomate), use flux-bot as the committer and the user as the author.

Is this right?

@leth
Copy link
Contributor Author

leth commented Sep 7, 2017

Exactly (assuming my previous research on the semantics of author vs comitter was correct)! :)

@squaremo
Copy link
Member

Addressed by #735, which introduces a flag for the daemon --git-set-author. When supplied (it's false by default, so people don't suddenly have broken systems), the daemon will use the user as the commit author.

It also makes fluxctl populate the user information by looking in git config, rather than using the logged-in user.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants