Skip to content

Conversation

@martyngigg
Copy link
Contributor

@martyngigg martyngigg commented Jan 24, 2026

Summary by CodeRabbit

Release Notes

  • Documentation

    • Updated certificate installation guide with requirement to update Ansible vault after obtaining the certificate bundle.
  • Chores

    • Updated certificate configuration for dev-analytics environment.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 24, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Updated certificate-related files: the README documentation was modified to include instructions for updating the Ansible vault after obtaining the certificate bundle, and a certificate configuration file's common name was changed from data-accelerator.isis.cclrc.ac.uk to dev-analytics.isis.cclrc.ac.uk.

Changes

Cohort / File(s) Summary
Documentation
certs/README.md
Added instruction to update Ansible vault with new certificate bundle content following the "Installing the new certificate chain" section
Certificate Configuration
certs/dev-analytics.isis.cclrc.ac.uk/2026-01/dev-analytics_isis_cclrc_ac_uk.cnf
Updated certificate common name (CN) from data-accelerator.isis.cclrc.ac.uk to dev-analytics.isis.cclrc.ac.uk

Poem

🐰 A certificate hops into place so fine,
From data-accelerator to dev-analytics divine,
Vault instructions now written clear and bright,
Security chains configured just right,
Domain names dance in the January night! ✨


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@martyngigg martyngigg enabled auto-merge (squash) January 24, 2026 10:04
@martyngigg martyngigg merged commit 22acfd7 into main Jan 24, 2026
1 of 2 checks passed
@martyngigg martyngigg deleted the dev-cert branch January 24, 2026 10:05
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