Skip to content

Conversation

@gcoxmoz
Copy link
Contributor

@gcoxmoz gcoxmoz commented Dec 21, 2025

Refractr PR Checklist

JIRA ticket: IO-3544

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.

@gcoxmoz gcoxmoz requested a review from a team as a code owner December 21, 2025 15:33
- www.viewsourceconf.com

# IO-3544
- brand.mozilla.com/hub/26:
Copy link
Member

Choose a reason for hiding this comment

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

I am like 95% sure this wants a trailing slash. If you run the tests locally with this change I'm guessing it will fail?

Suggested change
- brand.mozilla.com/hub/26:
- brand.mozilla.com/hub/26/:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Trying without the slash based on some prior-art above, and a handshake that I'll come crying if it turns out needed.

@gcoxmoz gcoxmoz merged commit 3849c7b into main Jan 5, 2026
1 check passed
@gcoxmoz gcoxmoz deleted the IO-3544 branch January 5, 2026 16:00
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