WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@jbuck
Copy link
Contributor

@jbuck jbuck commented Nov 23, 2025

Refractr PR Checklist

JIRA ticket: SREIN-636

When creating a PR for Refractr, confirm you've done the following steps for a smooth CI and CD experience:

  • Have you updated the relevant YAML in the PR?
  • Have you checked the relevant YAML for any possible dupes regarding your domain?
  • Have you checked if there are any TLS cert concerns - e.g. if the domain being redirected already exists, and it is being changed to point at refractr, is a temporary TLS 'outage' while waiting for certification via HTTP challenge okay? If not, add a note to the JIRA ticket.
  • If desired, have you generated the nginx config manually to confirm updates work as expected?

After PR merge, next steps include:

  • A merge to the main branch will automatically deploy refractr's stage environment -- deploying the prod environment requires a GitHub release to be created.
  • Once deployed, refractr's certmap must be updated and DNS entries must be changed -- SRE can help with this. Please pull someone in on the JIRA ticket or ask for help in #sre on Slack.

@jbuck jbuck requested a review from a team as a code owner November 23, 2025 19:18
@jbuck jbuck force-pushed the jbuck/push-smzwwxnoyrly branch 2 times, most recently from d3577fa to a786c9c Compare November 23, 2025 21:08
@jbuck jbuck force-pushed the jbuck/push-smzwwxnoyrly branch from a786c9c to bf879ef Compare November 23, 2025 21:09
@jbuck jbuck merged commit 0559cfb into main Nov 24, 2025
1 check passed
@jbuck jbuck deleted the jbuck/push-smzwwxnoyrly branch November 24, 2025 20:06
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.

3 participants