Galaxy: github webhook seems not work

[jeffrey4l]

Bug Report

SUMMARY

I wanna to trigger role import when the github repo is updated. But github webhook seems not work. And there is no detail doc at https://docs.ansible.com/ansible/latest/reference_appendices/galaxy.html#travis-integrations

And there is no code to support ansible-galaxy setup github source.

Travis works. But travis only support notification when build start/cancel/error/success/failure. But this is meaningless, because the pull request is not merged.

STEPS TO REPRODUCE

1. create webhook on github project. and use the token
2. setup github webhook on galaxy side by following
   ansible-galaxy setup github jeffrey4l ansible-alertmanager-role
3. trigger webhook on github side.

EXPECTED RESULTS

Response should be 200.

ACTUAL RESULTS

Get 500 error

Headers
Content-Length: 27
Content-Type: text/html
Date: Wed, 28 Nov 2018 01:26:01 GMT
Server: gunicorn/19.7.1
Set-Cookie: xxxx
Vary: Cookie
X-Frame-Options: SAMEORIGIN
X-REQUEST-ID: bbbb

Body
<h1>Server Error (500)</h1>

[chouseknecht]

@jeffrey4l

The Galaxy web hook only supports Travis. We're not going to extend it to work directly from GitHub at this point, which is what I think you're asking for.

[chouseknecht]

@jeffrey4l

There's some updated info here regarding Travis configuration. Ignore the note about travis-ci.org vs travis-ci.com. It works with both servers.

[jeffrey4l]

Thanks Chris.

On Thu, Nov 29, 2018 at 6:31 PM Chris Houseknecht [e-mail]
wrote:

Closed #1379 https://github.com/ansible/galaxy/issues/1379.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ansible/galaxy/issues/1379#event-1994832657, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVp0LOiNbog9aaM3_cL3IvhHHxx21Jhks5uz7dzgaJpZM4Y2rMs
.

--
Regards,
Jeffrey Zhang
Blog: http://xcodest.me

[geekbass]

The Galaxy web hook only supports Travis. We're not going to extend it to work directly from GitHub at this point, which is what I think you're asking for.

Is there any reason for not supporting Github webhooks now and in the future @chouseknecht ? Any background or context for this?

[stefanhorning]

Looks like the HTTP POST payload Travis sends it documented here: https://docs.travis-ci.com/user/notifications/#webhooks-delivery-format

So perhaps we can mimic this behavior in a Github action. However it would be nice to know which fields are actually used by Galaxy to trigger a re-import. So we don't have to build the entire structure and maybe just send a subset of that Travis payload. At least until Galaxy supports GH actions natively...