Skip to content

docs: fix certificate name plural#988

Merged
lukasmetzner merged 1 commit into
mainfrom
docs-certificate-name
Jul 21, 2025
Merged

docs: fix certificate name plural#988
lukasmetzner merged 1 commit into
mainfrom
docs-certificate-name

Conversation

@lukasmetzner
Copy link
Copy Markdown
Contributor

@lukasmetzner lukasmetzner commented Jul 21, 2025

All domains specified via the annotation load-balancer.hetzner.cloud/http-managed-certificate-domains, are used to create a single managed certificate. Therefore, the annotation load-balancer.hetzner.cloud/http-managed-certificate-name only accepts one name.

Additionally, fixed a small typo.

All domains specified via the annotation `load-balancer.hetzner.cloud/http-managed-certificate-domains`, are used to create a single managed certificate. Therefore, we only the annotation `load-balancer.hetzner.cloud/http-managed-certificate-name` only accepts one name.

Additionally, fixed a small typo.
@lukasmetzner lukasmetzner requested a review from a team as a code owner July 21, 2025 11:31
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.27%. Comparing base (82cbba0) to head (304956d).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #988      +/-   ##
==========================================
+ Coverage   70.05%   71.27%   +1.21%     
==========================================
  Files          22       22              
  Lines        3046     3046              
==========================================
+ Hits         2134     2171      +37     
+ Misses        743      706      -37     
  Partials      169      169              
Flag Coverage Δ
e2e 45.22% <ø> (+5.54%) ⬆️
unit 67.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lukasmetzner lukasmetzner merged commit 25021a2 into main Jul 21, 2025
10 checks passed
@lukasmetzner lukasmetzner deleted the docs-certificate-name branch July 21, 2025 11:53
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