Skip to content

Add/ckg2 rm #3261

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.

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

Merged
merged 8 commits into from
Jun 24, 2025
Merged

Add/ckg2 rm #3261

merged 8 commits into from
Jun 24, 2025

Conversation

kquinsland
Copy link
Contributor

rack mount adapter for cloud key gen2 devices

Karl Quinsland added 3 commits June 19, 2025 17:11
With images I was able to rip from amazon product listing.
Powered Device (passthrough...)
@Copilot Copilot AI review requested due to automatic review settings June 20, 2025 00:24
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Add a new device type YAML for the Ubiquiti CloudKey Gen2 rack mount adapter, capturing metadata and physical specs.

  • Introduce metadata (manufacturer, model, slug, part_number)
  • Define physical attributes (u_height, weight, airflow, imaging flags)
  • Add subdevice role and interface section
Comments suppressed due to low confidence (3)

device-types/Ubiquiti/CKG2-RM.yaml:22

  • The rack mount adapter is a passive mechanical accessory and does not have network interfaces. The interfaces section should be removed to avoid incorrectly modeling an eth0 port.
interfaces:

device-types/Ubiquiti/CKG2-RM.yaml:17

  • [nitpick] Remove or resolve the comment block expressing uncertainty around subdevice modeling before merging. If needed, document the final approach in official docs rather than leaving TODO-style comments.
# This is always a parent device. When standalone, the cloud key is technically is not a parent or child.

device-types/Ubiquiti/CKG2-RM.yaml:3

  • [nitpick] The model name should reflect Gen2 compatibility per the PR description. Consider renaming to CloudKey Gen2 Rack Mount for clarity.
model: CloudKey Rack Mount

@harryajc harryajc added the status: revisions needed This issue requires additional information to be actionable label Jun 20, 2025
@kquinsland kquinsland requested a review from harryajc June 20, 2025 14:29
@kquinsland kquinsland requested a review from harryajc June 24, 2025 00:56
@harryajc harryajc merged commit 6cada78 into netbox-community:master Jun 24, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: revisions needed This issue requires additional information to be actionable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants