Skip to content

chore(certs): Remove date-based directories for certificate requests.#342

Closed
martyngigg wants to merge 1 commit into
mainfrom
certs-reorg
Closed

chore(certs): Remove date-based directories for certificate requests.#342
martyngigg wants to merge 1 commit into
mainfrom
certs-reorg

Conversation

@martyngigg

Copy link
Copy Markdown
Member

Summary

The requests are always the same so simply store them in the directory named after the domain.

@martyngigg martyngigg requested a review from a team as a code owner June 11, 2026 07:50
@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (4)
  • infra/certs/.gitignore
  • infra/certs/README.md
  • infra/certs/analytics.isis.cclrc.ac.uk/analytics_isis_cclrc_ac_uk.cnf
  • infra/certs/dev-analytics.isis.cclrc.ac.uk/dev-analytics_isis_cclrc_ac_uk.cnf
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8f4c46ce-e3ca-4d38-a97e-d6e2b34b76d2

📥 Commits

Reviewing files that changed from the base of the PR and between b93399d and 247c1d5.

📒 Files selected for processing (4)
  • infra/certs/.gitignore
  • infra/certs/README.md
  • infra/certs/analytics.isis.cclrc.ac.uk/analytics_isis_cclrc_ac_uk.cnf
  • infra/certs/dev-analytics.isis.cclrc.ac.uk/dev-analytics_isis_cclrc_ac_uk.cnf

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

Copy link
Copy Markdown
Member Author

Closing in favour of an Ansible playbook approach.

@martyngigg martyngigg closed this Jun 11, 2026
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.

1 participant