Skip to content

fix(server): missing firewalls in describe - #1482

Merged
phm07 merged 3 commits into
mainfrom
server-describe-firewall
Sep 14, 2026
Merged

phm07 merged 3 commits into
mainfrom
server-describe-firewall

Conversation

@phm07

@phm07 phm07 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Fixes #1481

@phm07 phm07 self-assigned this Sep 10, 2026
@phm07
phm07 requested a review from a team as a code owner September 10, 2026 08:34
@phm07
phm07 requested a review from jooola September 10, 2026 08:34
@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 60.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.81%. Comparing base (a3ff292) to head (dbe1f81).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
internal/cmd/server/describe.go 60.00% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1482      +/-   ##
==========================================
- Coverage   73.08%   72.81%   -0.28%     
==========================================
  Files         301      302       +1     
  Lines       11097    11177      +80     
==========================================
+ Hits         8110     8138      +28     
- Misses       2090     2141      +51     
- Partials      897      898       +1     
Flag Coverage Δ
e2e 51.24% <0.00%> (-0.23%) ⬇️
unit 67.71% <60.00%> (-0.23%) ⬇️

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

☔ View full report in Codecov by Harness.
📢 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.

Comment thread internal/cmd/server/describe.go Outdated
Comment on lines +121 to +123
Firewalls:
No Firewalls

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we also test with some firewall attached?

@phm07
phm07 merged commit 93f8381 into main Sep 14, 2026
8 checks passed
@phm07
phm07 deleted the server-describe-firewall branch September 14, 2026 14:15
jooola pushed a commit that referenced this pull request Sep 14, 2026
<!-- section-start changelog -->
### Features

- print deprecated Load Balancer Type information (#1461)
([22a60bc](22a60bc))
- add api command (#1457)
([bd9f704](bd9f704))
- **image**: print deprecated Image information (#1485)
([c7badc0](c7badc0))

### Bug Fixes

- **server**: missing firewalls in describe (#1482)
([93f8381](93f8381))

<!-- section-end changelog -->

---

<details>
<summary><h4>PR by <a
href="https://github.com/apricote/releaser-pleaser">releaser-pleaser</a>
🤖</h4></summary>

If you want to modify the proposed release, add you overrides here. You
can learn more about the options in the docs.

## Release Notes

### Prefix / Start

This will be added to the start of the release notes.

~~~~rp-prefix
~~~~

### Suffix / End

This will be added to the end of the release notes.

~~~~rp-suffix
~~~~

</details>

Co-authored-by: Hetzner Cloud Bot <>
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.

Missing "firewalls" in the desribe server command

2 participants