Webhooks for Open edX integrating JIRA and Github. Designed to be deployed at Heroku.
Access the app at https://openedx-webhooks.herokuapp.com.
Make sure you've installed:
Python 2.7.x development environment (virtualenv strongly recommended)
-
All
heroku
commands can be performed through the Heroku web-based dashboard as well, if you don't want to use the CLI.
Log in using the email address and password you used when creating your Heroku account:
heroku login
Authenticating is required to allow both the
heroku
andgit
commands to operate.Add the Heroku app repo as a git remote:
heroku git:remote -a openedx-webhooks-staging
Verify that the remote is added properly:
git remote -v
You should see output similar to:
heroku https://git.heroku.com/openedx-webhooks-staging.git (push) heroku https://git.heroku.com/openedx-webhooks-staging.git (fetch) origin [email protected]:edx/openedx-webhooks.git (fetch) origin [email protected]:edx/openedx-webhooks.git (push)
This app relies on the following addons from Heroku:
- PostgreSQL
- Redis
- Papertrail
- Scheduler
While it's possible to replicate the entire stack locally, it'll be difficult to ensure consistent experience for each developer. Instead, we utilize the pipeline facility offered by Heroku to handle our development needs.
The general development cycle is:
Code → Deploy branch to staging → Test → Iterate
To deploy a local branch to staging:
git push heroku [branch_or_tag_or_hash:]master
In most cases, to push your current working branch, use:
git push heroku @:master
Once you're satisfied with your changes, go ahead and open a pull request per normal development procedures.
Navigate to https://openedx-webhooks-staging.herokuapp.com to make sure the app has started. If the URL is too hard to remember, you can also use:
heroku open
If the application isn't running, visit the openedx-webhooks-staging Resources page to make sure there are dynos running.
make install-dev-requirements make test
In most cases, you'll want to deploy by promoting from staging to production. The general workflow is:
Merge to master
→ Deploy master
to staging → Test → Promote to
production
Prior to the promotion, make sure all the changes have been merged
to master
, and you've deployed the master
branch successfully to
staging:
git push heroku master
When you're ready to promote from staging to production:
heroku pipelines:promote -r heroku
Make sure the same git commit is deployed to both environments. First see what's deployed on staging:
heroku releases -n 1
Then see what's deployed on production:
heroku releases -a openedx-webhooks -n 1
Make sure the abbreviated git SHAs match.
Navigate to https://openedx-webhooks.herokuapp.com to make sure the app has started. If the URL is too hard to remember, you can also use:
heroku open -a openedx-webhooks
This should no longer be an issue as of July 9th, 2018.
If you re-process pull requests, an unfortunate thing can happen: it will find stale pull requests that were written by edX employees who have now left. The bot will see that the author has no contributor agreement, and will make a new JIRA issue for the pull request. This is needless noise.
The bot looks for comments it wrote that have a JIRA issue id in them. You can leave the bot comment on the stale pull request so that at least it won't happen again in the future.
On GitHub, visit the repo webhooks
(https://github.com/<ORG>/<REPO>/settings/hooks
) or organization webhooks
(https://github.com/organizations/<ORG>/settings/hooks
) page.
Create or edit a webhook.
- Payload URL: https://openedx-webhooks.herokuapp.com/github/hook-receiver
- Content type: application/json
- Secret: same as setting GITHUB_WEBHOOK_SECRET in Heroku
- Events:
- Issue comments
- Pull requests
- Pull request reviews
- Pull request review comments
See the fragment files (if any) in the changelog.d directory.
- The number of lines added and deleted by a pull request are recorded in custom Jira fields.
- Core Committer pull requests now start with a Jira status of "Waiting on Author" rather than "Open edX Community Review".
- Draft pull requests start with a status of "Waiting on Author". Once the pull request is no longer a draft, the status is set to the initial status it would have originally had.
- BUG: if the PR description was edited, the Jira issue status would be incorrectly reset to its initial value [OPENEDX-424]. This is now fixed.
- When a core committer merges a pull request, the bot will add a comment pinging the committer's edX champions to let them know the merge has happened.
- BUG: previously the bot could clobber ad-hoc labels on Jira issues when it set its own labels. This is now fixed. The bot will preserve any labels it didn't make.
- Removed the code that managed webhooks in repos.
- Refactored some code that handles pull requests being closed, so now it operates on any change to the pull request. The behavior should be the same, except now if a pull request is closed or merged after the Jira issue has been manually deleted, the bot will create a new issue so that it can mark it Rejected or Merged.
- BUG: previously, the bot might change GitHub labels and incorrectly drop ad-hoc labels that people had put on the pull request. This is now fixed.
- GitHub very occasionally sends us a pull request event, but then serves us a 404 error when we ask it about the pull request. Now the bot will retry GET requests that return 404, to give GitHub a chance to get its act together.
- BUG: when a pull request was edited, the associated Jira issue would be reset to its initial status. This is now fixed: the Jira status is unchanged.
- Previously, if an OSPR issue had been manually moved to BLENDED, and then the title of the pull request amended to have "[BD-xx]", the bot would try and fail to delete the moved issue. Now it understands the move, and doesn't try to delete the original issue. It also updates the issue with Blended information.
- Changes to the title or description of a pull request are copied over to the associated Jira issue to keep them in sync.
- If a change to a pull request requires a different Jira issue, the old issue is deleted, and a new one made. For example, if a blended pull request doesn't have "[BD-xx]" in the title, an OSPR issue gets made initially. Now when the developer updates the title, the OSPR issue is deleted, and a new BLENDED issue is created for it.
- The "expires_on" key in people.yaml is officially obsolete, and no longer interpreted.
- Some incorrect CLA logic was fixed. An entry in people.yaml with no "expires_on" key would be considered to have a signed CLA, even if the agreement was "none".
- If an opened pull request has a CLA, then the bot will comment "jenkins ok to test" on it to get the tests started automatically.
- Blended workflow: if "[BD-XX]" is found in the title of an opened pull request, then the Jira ticket will be in the BLENDED project, with links to the correct epic, etc.
- Core committer logic has to be particular to specific repos, it's not a blanket right. Now "committer" isn't a simple boolean, it's an object with subkeys: "repos" is a list of repos the user can commit to, and "orgs" is a list of GitHub organizations the user can commit to (any repo).
- Slight change to people.yaml schema: "internal:true" is used to indicate edX people (or Arbisoft). The "committer:true" flag indicates core committers.
- Core committer pull request handling: a different welcome message is used, OSPR issues are started in the "Open edX Community Review" status, and "core committer" GitHub and Jira labels are applied.
- We used to have two GitHub webhooks. They have been combined. Only /github/hook-receiver is needed now. The obsolete /github/pr endpoint still exists just to log unneeded webhook action so we can fix the GitHub configuration.
- Labels in GitHub repos are synchronized from repo-tools-data/labels.yaml before any labels are adjusted in the repo.
- Data read from repo-tools-data (people.yaml, label.yaml) is only cached for 15 minutes. It used to be until the bot was restarted.
- Pull requests that need a CLA signed now create Jira tickets in the "Community Manager Review" status.
- Describe the different processes that are run on Heroku
- Describe how to access logs
- Make sure
docs/
is up to date