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.

Sign up for GitHub

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

Jump to bottom

Add CName for production database to DNS #310

Closed
2 of 4 tasks
acn-sbuad opened this issue Jan 16, 2024 · 4 comments
Closed
2 of 4 tasks

Add CName for production database to DNS #310

acn-sbuad opened this issue Jan 16, 2024 · 4 comments
Assignees
@SandGrainOne
Labels
kind/chore Typical issues like patching and deployment.

Comments

@acn-sbuad
Copy link
Contributor

acn-sbuad commented Jan 16, 2024 •
edited
Loading

Description

CName for the storage db in production is not registered in DNS and should be so.

Additional Information

For Storage in production the connection string now holds the actual hostname of the server instead of the CNAME.

Tasks

  • Add to DNS
  • Update connection string to use cname in values.yaml
  • Verification

Acceptance Criterias

  • Database is referred to by CNAME
@acn-sbuad acn-sbuad added the kind/chore Typical issues like patching and deployment. label Jan 16, 2024
@acn-sbuad acn-sbuad changed the title Add C-name for database to DNS Add CName for database to DNS Jan 16, 2024
@SandGrainOne SandGrainOne changed the title Add CName for database to DNS Add CName for production database to DNS Feb 21, 2024
@SandGrainOne
Copy link
Member

@ootneim Is this a quick fix? Maybe already done?

@ootneim
Copy link
Member

ootneim commented Feb 26, 2024

Looks like the CNAME is incorrect in DNS.

storage-postgres.altinn.no -> altinn-storage-db-pgflex.postgres.database.azure.com

Correct setting is:
storage-postgres.altinn.no -> altinn-storage-db-prod-pgflex.postgres.database.azure.com

@bengtfredh or @Herskis have access to update the DNS record.

@bengtfredh
Copy link
Member

@ootneim CNAME is updated - record has TTL 2h - but can take up to 24 hour for all dns to be updated

@acn-sbuad
Copy link
Contributor Author

Using CName in settings for Storage. Will be rolled out to production with Release 743

@SandGrainOne SandGrainOne self-assigned this Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SandGrainOne SandGrainOne

Labels
kind/chore Typical issues like patching and deployment.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@SandGrainOne @bengtfredh @acn-sbuad @ootneim