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

upgraded NAC's admin RDS DB instance #283

Merged
merged 2 commits into from
Jun 20, 2024
Merged

upgraded NAC's admin RDS DB instance #283

merged 2 commits into from
Jun 20, 2024

Conversation

juddin927
Copy link
Contributor

  • Upgraded DB instance class from T2 to T3
  • Enabled performance insights
  • Changed RDS Certificate Authority(CA)

@juddin927 juddin927 requested a review from a team as a code owner June 20, 2024 13:13
@juddin927 juddin927 requested review from mTouhid, kyphutruong and laurentb4 and removed request for a team June 20, 2024 13:13
sb21460
sb21460 approved these changes Jun 20, 2024
View reviewed changes
Copy link
Contributor

@sb21460 sb21460 left a comment

Choose a reason for hiding this comment

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

LGTM

@sb21460 sb21460 merged commit 13c75ac into main Jun 20, 2024
@sb21460 sb21460 deleted the ND-376-db-instance-upgrade branch June 20, 2024 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sb21460 sb21460 sb21460 approved these changes

@mTouhid mTouhid Awaiting requested review from mTouhid mTouhid is a code owner automatically assigned from ministryofjustice/nvvs-devops-admins

@kyphutruong kyphutruong Awaiting requested review from kyphutruong kyphutruong is a code owner automatically assigned from ministryofjustice/nvvs-devops-admins

@laurentb4 laurentb4 Awaiting requested review from laurentb4 laurentb4 is a code owner automatically assigned from ministryofjustice/nvvs-devops-admins

Assignees
No one assigned
Labels
network-access-control
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@juddin927 @sb21460