elastic / ansible-elasticsearch

Ansible playbook for Elasticsearch
Other
1.59k stars 857 forks source link

Support elasticsearch-keystore entries #767

Closed Bernhard-Fluehmann closed 3 years ago

Bernhard-Fluehmann commented 3 years ago

@jmlrt Here you go

elasticmachine commented 3 years ago

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

elasticmachine commented 3 years ago

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

cla-checker-service[bot] commented 3 years ago

❌ Author of the following commits did not sign a Contributor Agreement: e66f4665ccac1a4d78922473e0fca4239fc90a0e, 6ed512f4b381852647ec9e2c8e44f2e582ee1473, 94a4686edfcf6e45b7ca93bf149e22d06cb71025, , , ed0427cd3821342c50604a542b465d640decd1fb, , , , 141f8263e312f7f06f72dbb0415907020e713fca

Please, read and sign the above mentioned agreement if you want to contribute to this project

Bernhard-Fluehmann commented 3 years ago

@jmlrt Can you please check what's still wrong with the Contribution license?

jmlrt commented 3 years ago

@Bernhard-Fluehmann, it seems that you have some issue with your git configuration and GitHub doesn't recognize these commits as coming from your GH user:

Screenshot 2021-02-02 at 14 54 09

(GH see 2 authors and don't match them with your profile picture).

Also I found the CLA you signed but I think you did a typo in your email.

Could you fix your GH config and the commits and eventually re-sign CLA to fix the typo in your email?

jmlrt commented 3 years ago

@jmlrt Can you tell me please what's the typo in sla, since it used to work

I think your email adress in CLA contains struff while that should be stuff.

Bernhard-Fluehmann commented 3 years ago

@jmlrt New CLA submitted. Please confirm

jmlrt commented 3 years ago

cla/check

jmlrt commented 3 years ago

@Bernhard-Fluehmann, this is still failling and your commits are still not recognized by GitHub as coming from you:

Screenshot 2021-02-03 at 11 46 17
Bernhard-Fluehmann commented 3 years ago

@jmlrt Ok, now I am getting puzzled. I have set the username and mail correctly in my git client and I don't know what do do else. I am using Bernhard-Fluehmann (my GitHub user.name) and bernhard.fluehmann@realstuff.ch (My Git user.mail). What's still wrong with it?

Bernhard-Fluehmann commented 3 years ago

@jmlrt The only thing is that I am commiting to our fork of the repo (RealStuff/ansible-elasticsearch) and create the pull-request from there. It's not a private repo but instead a repo of our company account

Bernhard-Fluehmann commented 3 years ago

@jmlrt BTW, this is the output of my console

Bildschirmfoto 2021-02-03 um 12 07 11
jmlrt commented 3 years ago

Using GitHub API, here is what I see for one of your previous commit which was recognized by GitHub:

https://github.com/elastic/ansible-elasticsearch/pull/757/commits/aa95b67941cc6694d67eaec317683e42db0af944

➜  ~ curl -H "Accept: application/vnd.github.v3+json" \
 https://api.github.com/repos/elastic/ansible-elasticsearch/commits/aa95b67941cc6694d67eaec317683e42db0af944|jq '.commit'
{
  "author": {
    "name": "bernhard.fluehmann",
    "email": "bernhard.fluehmann@realstuff.ch",
    "date": "2021-01-20T18:42:39Z"
  },
  "committer": {
    "name": "bernhard.fluehmann",
    "email": "bernhard.fluehmann@realstuff.ch",
    "date": "2021-01-20T18:52:50Z"
  },
  "message": "Added keystore entry support",
  "tree": {
    "sha": "34a84102475215b117ab7b9b43edebe5c8db94a7",
    "url": "https://api.github.com/repos/elastic/ansible-elasticsearch/git/trees/34a84102475215b117ab7b9b43edebe5c8db94a7"
  },
  "url": "https://api.github.com/repos/elastic/ansible-elasticsearch/git/commits/aa95b67941cc6694d67eaec317683e42db0af944",
  "comment_count": 0,
  "verification": {
    "verified": false,
    "reason": "unsigned",
    "signature": null,
    "payload": null
  }
}

Here is what I get for one of your new commits which are not recognized:

https://github.com/elastic/ansible-elasticsearch/pull/767/commits/e66f4665ccac1a4d78922473e0fca4239fc90a0e

➜  ~ curl -H "Accept: application/vnd.github.v3+json" \
 https://api.github.com/repos/elastic/ansible-elasticsearch/commits/e66f4665ccac1a4d78922473e0fca4239fc90a0e|jq .commit
{
  "author": {
    "name": "Bernhard-Fluehmann",
    "email": "Bernhard.Fluehmann@<ANOTHER_EMAIL>.com",
    "date": "2021-02-02T13:00:59Z"
  },
  "committer": {
    "name": "Bernhard-Fluehmann",
    "email": "Bernhard.Fluehmann@<ANOTHER_EMAIL>.com",
    "date": "2021-02-02T13:00:59Z"
  },
  "message": "Support elasticsearch-keystore entries",
  "tree": {
    "sha": "20e7dda3560dc0a03621c86c0dd7d431606e32e9",
    "url": "https://api.github.com/repos/elastic/ansible-elasticsearch/git/trees/20e7dda3560dc0a03621c86c0dd7d431606e32e9"
  },
  "url": "https://api.github.com/repos/elastic/ansible-elasticsearch/git/commits/e66f4665ccac1a4d78922473e0fca4239fc90a0e",
  "comment_count": 0,
  "verification": {
    "verified": false,
    "reason": "unsigned",
    "signature": null,
    "payload": null
  }
}
Bernhard-Fluehmann commented 3 years ago

@jmlrt Now?