{"action":"opened","issue":{"url":"https://api.github.com/repos/akretion/docky/issues/103","repository_url":"https://api.github.com/repos/akretion/docky","labels_url":"https://api.github.com/repos/akretion/docky/issues/103/labels{/name}","comments_url":"https://api.github.com/repos/akretion/docky/issues/103/comments","events_url":"https://api.github.com/repos/akretion/docky/issues/103/events","html_url":"https://github.com/akretion/docky/issues/103","id":353191371,"node_id":"MDU6SXNzdWUzNTMxOTEzNzE=","number":103,"title":"missing git config --global user.email/name ?","user":{"login":"rvalyi","id":16926,"node_id":"MDQ6VXNlcjE2OTI2","avatar_url":"https://avatars2.githubusercontent.com/u/16926?v=4","gravatar_id":"","url":"https://api.github.com/users/rvalyi","html_url":"https://github.com/rvalyi","followers_url":"https://api.github.com/users/rvalyi/followers","following_url":"https://api.github.com/users/rvalyi/following{/other_user}","gists_url":"https://api.github.com/users/rvalyi/gists{/gist_id}","starred_url":"https://api.github.com/users/rvalyi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rvalyi/subscriptions","organizations_url":"https://api.github.com/users/rvalyi/orgs","repos_url":"https://api.github.com/users/rvalyi/repos","events_url":"https://api.github.com/users/rvalyi/events{/privacy}","received_events_url":"https://api.github.com/users/rvalyi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-08-23T02:00:27Z","updated_at":"2018-08-23T02:00:27Z","closed_at":null,"author_association":"MEMBER","body":"Hello, when running ak --verbose build with **merge** settings in my spec.yaml to be made (for instance merging branch akretion, 10.0-migration/l10n_br_account into branch OCA 10.0), git would fail with the following message:\r\n\r\n```\r\n(INFO) [01:08:59] git_aggregator.repo  Reset branch to kmee 10.0-mig-l10n_br_stock\r\n(INFO) [01:08:01] git_aggregator.repo  Pull akretion, 10.0-migration/l10n_br_account\r\n\r\n*** Please tell me who you are.\r\n\r\nRun\r\n\r\n  git config --global user.email \"you@example.com\"\r\n  git config --global user.name \"Your Name\"\r\n\r\nto set your account's default identity.\r\nOmit --global to set the identity only in this repository.\r\n\r\nfatal: empty ident name (for <odoo@a748a62ae312.(none)>) not allowed\r\nTraceback (most recent call last):\r\n  File \"/usr/local/bin/gitaggregate\", line 11, in <module>\r\n    sys.exit(main())\r\n  File \"/usr/local/lib/python2.7/dist-packages/git_aggregator/main.py\", line 131, in main\r\n    run(args)\r\n  File \"/usr/local/lib/python2.7/dist-packages/git_aggregator/main.py\", line 174, in run\r\n    r.aggregate()\r\n  File \"/usr/local/lib/python2.7/dist-packages/git_aggregator/repo.py\", line 186, in aggregate\r\n    self._merge(merge)\r\n  File \"/usr/local/lib/python2.7/dist-packages/git_aggregator/repo.py\", line 253, in _merge\r\n    self.log_call(cmd)\r\n  File \"/usr/local/lib/python2.7/dist-packages/git_aggregator/repo.py\", line 156, in log_call\r\n    ret = callwith(cmd, **kw)\r\n  File \"/usr/lib/python2.7/subprocess.py\", line 540, in check_call\r\n    raise CalledProcessError(retcode, cmd)\r\nsubprocess.CalledProcessError: Command '(u'git', u'pull', u'--no-edit', u'--quiet', 'akretion', '10.0-migration/l10n_br_account')' returned non-zero exit status 128\r\nTraceback (most recent call last):\r\n```\r\n\r\nand indeed doing\r\n```\r\n  git config --global user.email \"you@example.com\"\r\n  git config --global user.name \"Your Name\"\r\n```\r\nmakes the next try to succeed.\r\n\r\nSo shouldn't we not have a fake git config in our base image just like we had in Voodoo? Something that could be overriden if we mount some config file or may be pass some ENV VAR. What do you think?"}}
{"actor":{"display_login":"rvalyi"}}