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

[21.05] Switch MX record for gocept.net #1080

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

zagy
Copy link
Member

@zagy zagy commented Aug 7, 2024

@flyingcircusio/release-managers

FC-23766

Release process

Impact: n/a

Changelog: n/a

PR release workflow (internal)

  • PR has internal ticket
  • internal issue ID (PL-…) part of branch name
  • internal issue ID mentioned in PR description text
  • ticket is on Platform agile board
  • ticket state set to Pull request ready
  • if ticket is more urgent than within the next few days, directly contact a member of the Platform team

Design notes

  • Provide a feature toggle if the change might need to be adjusted/reverted quickly depending on context. Consider whether the default should be on or off. Example: rate limiting.
  • All customer-facing features and (NixOS) options need to be discoverable from documentation. Add or update relevant documentation such that hosted and guided customers can understand it as well.

Security implications

  • Security requirements defined? (WHERE)
    • Keep systems up to date. The old mailserver should be decommissioned.
  • Security requirements tested? (EVIDENCE)
    • New server is set up and accepts mail for gocept.net.

@zagy zagy requested a review from ctheune August 7, 2024 05:19
@sysvinit sysvinit changed the base branch from fc-24.05-dev to fc-21.05-dev August 7, 2024 07:18
@sysvinit sysvinit changed the title Fc 23766 switch mailserver [21.05] Switch MX record for gocept.net Aug 7, 2024
@@ -9,7 +9,7 @@ ns2 CNAME ns2.gocept.com.
; generic services
; these aliases must match values in location.pp (if applicable)
mail AAAA 2a02:248:101:62::19
MX 10 mail
MX 10 mail.flyingcircus.io.
Copy link
Member

Choose a reason for hiding this comment

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

This change updates the MX record for mail.gocept.net. instead of the top-level MX record for gocept.net.. I presume this is probably not what you intended?

Copy link
Member

Choose a reason for hiding this comment

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

(Does mail.gocept.net itself need an MX record? Can we remove that record from the zone file?)

@ctheune ctheune marked this pull request as draft August 14, 2024 09:40
@zagy zagy force-pushed the FC-23766-switch-mailserver branch from 53ca711 to bfed68a Compare August 21, 2024 11:23
@zagy zagy marked this pull request as ready for review August 21, 2024 11:49
@zagy zagy requested a review from sysvinit August 21, 2024 11:49
@sysvinit sysvinit merged commit da1d3c6 into fc-21.05-dev Aug 26, 2024
2 checks passed
@sysvinit sysvinit deleted the FC-23766-switch-mailserver branch August 26, 2024 09:37
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.

2 participants