Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOP-4242: Use release, not tag push for CI/CD #968

Merged
merged 160 commits into from
Jan 24, 2024
Merged

DOP-4242: Use release, not tag push for CI/CD #968

merged 160 commits into from
Jan 24, 2024

Conversation

mayaraman19
Copy link
Contributor

@mayaraman19 mayaraman19 commented Jan 19, 2024

DOP-4242

This PR changes all instances of on.push.tag to on.release so that the CI/CD workflow is only activated upon release. please ignore the commit history 😭

To test, I've created this dummy repository which has actions that are run only upon release. It seems to be working - feel free to clone, contribute, release, and play around with it.

Copy link

Your feature branch infrastructure has been deployed!

Your webhook URL is: https://z03hdnj1a6.execute-api.us-east-2.amazonaws.com/prod/webhook/githubEndpoint/trigger/build

For more information on how to use this endpoint, follow these instructions.

@mayaraman19 mayaraman19 marked this pull request as ready for review January 22, 2024 19:16
Copy link
Contributor

@seungpark seungpark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will be a good opportunity to update the readme for releases! 🙏

.github/workflows/deploy-prd-ecs.yml Show resolved Hide resolved
@mayaraman19 mayaraman19 requested a review from seungpark January 23, 2024 16:46
Copy link
Contributor

@seungpark seungpark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. minor notes below!

README.md Outdated
@@ -157,7 +157,8 @@ docs-worker-pool contains various triggers for release to higher environments. C

- Create release tags. We currently follow [semver](https://semver.org/) standards.
- If you don't have push access, open an issue or otherwise contact a contributor with administrator privileges.
- Verify that the deploy-production-ec2 workflow executed successfully for both job runs across both production instances.
- Create a release associated with the tag using the GitHub UI or by running `gh release create`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be an extra space at end of line, based on error

@@ -2,6 +2,14 @@

This README describes the various workflows defined for the docs-worker-pool repository.

## Releases

The release process occurs whenever someone releases a specific tag via the GitHub UI. Three separate workflows are run upon release:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor nitpick. not specific to the Github UI

@mayaraman19 mayaraman19 merged commit 640f813 into master Jan 24, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants