This document is an extracted version of LMS/Studio Security Fix Process. Please consult this documentation for details and explanations, This document will only provide steps by step procedure and doesn't discuss technical details.
- Ensure that the
edx-platform-private:security-release
branch is up-to-date.- If the branch is out-of-date and not tracking
edx-platform:master
closely (as will likely be the case):- In GoCD, trigger the
edxapp_private_public_merge_sync
pipeline manually and ensure it completes successfully.
- In GoCD, trigger the
- If the branch is out-of-date and not tracking
- Clone edx-platform-private repo locally (Pull the latest changes if you have cloned the repo already).
- git clone git@github.com:edx/edx-platform-private.git
- Create a branch off
edx-platform-private:security-release
(repo:branch)git checkout -t origin/security-release
git checkout -b <branch_name>
- Add the security fix commits to your branch.
- Push the branch.
- git push --set-upstream origin <branch_name>
- Create a pull request using
edx-platform-private:security-release
as its base branch with the following template in the PR description.Code Block This is a security fix. Please read important process notices below. **NOTE:** Once the PR is marked as approved it will be deployed to production. DO NOT approve until you are ready for the patch to go! **NOTE:** Do NOT merge the PR after approval! Although there can be multiple reviewers, only one developer should approve the PR using the "Review changes"→ "Approve" → "Submit review" flow but should not merge it! Private Deployment: - [x] Squash changes. - [x] Approved and deployed to Production. After Private Deployment: - [ ] Send notification with patches. - [ ] Wait 48 hours. Once Deployment can be made Public: - [ ] Merge PR - A PR will automatically be created on public repo.
- Request reviews of your PR from one developer but do not merge it!!
- The developer should approve the PR using the "Review changes"→ "Approve" → "Submit review" flow
- NOTE: Once the PR is marked as approved it will be deployed to production. DO NOT approve until you are ready for the patch to go!
- Code Deployment.
- Deployment on
- Stage
- The next edxapp release pipeline deploys to the stage environment will include the approved private PR commits.
- Please verify your changes on stage.
- If you are unsure if Stage currently has your changes, visit this page for more information >> /wiki/spaces/RELEASES/pages/182780356
- Deployment on Production
- When the stage release candidate is advanced to production, the production release will include the approved private PR commits.
- Please verify your changes in production.
- If you are unsure if Prod currently has your changes, visit this page for more information >> /wiki/spaces/RELEASES/pages/182780356
- Send email notification with patches, for more details LMS/Studio Security Fix Process
- Wait for 48 hours after the email notification
- Make the private fix public
- Merge the private PR into its base branch,
edx-platform-private:security-release
. NOTE: Do NOT rebase the PR - or the PR's branch!- The bot creates a pull request against master in edx-platform.
- The tests will run against your PR
- The automation will merge the pull request automatically after the tests pass.
- Merge the private PR into its base branch,
- Send out general notifications as documented in Security Disclosure Reporting and Resolution.
...