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

Implement CI for MystBin #40

Merged
merged 2 commits into from
May 16, 2024
Merged

Implement CI for MystBin #40

merged 2 commits into from
May 16, 2024

Conversation

AbstractUmbra
Copy link
Collaborator

@AbstractUmbra AbstractUmbra commented May 16, 2024

Description

This PR adds capability for CI for MystBin using GitHub actions.

The steps it follows are:-

  1. A push to main will build a Docker image of the MystBin software and publish this to GHCR under the PythonistaGuild organization. This will be a public image so that anyone may use it.
  2. On a successful build and publish of this image, it will execute a deployment action and deploy this newly built image to the current host of MystBin.
    2a) This deployment stage will only restart the mystbin service, not the pg database nor the redis cache.

This is currently not very testable due to the limitations of GitHub actions and may require subsequent PRs. In such cases I will use my break-glass privileges to squash and force push any fixes I must do.

Non-goals of this PR:-

  • HADR / Replication of the database for a staging environment.
    • I plan to tackle this at a later date.

Checklist

  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
    • I have updated the changelog with a quick recap of my changes.
  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed)
  • This PR is not a code change (e.g. documentation, README, ...)
  • I have read and agree to the Developer Certificate of Origin for this contribution

@AbstractUmbra AbstractUmbra marked this pull request as ready for review May 16, 2024 21:22
@AbstractUmbra AbstractUmbra merged commit 6358119 into main May 16, 2024
2 checks passed
@AbstractUmbra AbstractUmbra deleted the feature/ci branch May 16, 2024 21:34
AbstractUmbra added a commit that referenced this pull request May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants