Skip to content

Conversation

Guibi1
Copy link
Contributor

@Guibi1 Guibi1 commented Sep 3, 2025

Added the Cisco Assurance Sensor GT (SKY-GT-AA) using official datasheet.

@Copilot Copilot AI review requested due to automatic review settings September 3, 2025 18:38
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

This PR adds device type configuration for the Cisco Assurance Sensor GT (SKY-GT-AA) based on the official datasheet, providing hardware specifications and port configurations for network management systems.

  • Added complete device specification including physical dimensions, power requirements, and network interfaces
  • Configured dual power supplies with IEC-60320-C6 connectors and 22W maximum draw each
  • Defined network interfaces including copper Gigabit Ethernet, SFP ports, and dedicated management interface

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Guibi1 Guibi1 force-pushed the cisco-assurance-gt branch 2 times, most recently from 1a56847 to 5f3e762 Compare September 9, 2025 18:22
weight_unit: lb
comments: '[Assurance Sensor Datasheet](https://www.cisco.com/c/en/us/products/collateral/cloud-systems-management/provider-connectivity-assurance-sensors/provider-connect-assurance-sensors-gts-ds.html)
[Assurance Sensor Installation Overview](https://www.cisco.com/c/en/us/td/docs/cloud-systems-management/provider-connectivity-assurance/hardware/assurance-sensor-gt/gt-install/m_overview.pdf)'
power-ports:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are these field replaceable PSUs? If so they should be modeled as module-bays.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No they aren't. It's a simple DC plug.

Copy link
Collaborator

Choose a reason for hiding this comment

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

If they are a DC plug they should be type: dc-terminal

@harryajc harryajc added the status: revisions needed This issue requires additional information to be actionable label Sep 11, 2025
weight_unit: lb
comments: '[Assurance Sensor Datasheet](https://www.cisco.com/c/en/us/products/collateral/cloud-systems-management/provider-connectivity-assurance-sensors/provider-connect-assurance-sensors-gts-ds.html)
[Assurance Sensor Installation Overview](https://www.cisco.com/c/en/us/td/docs/cloud-systems-management/provider-connectivity-assurance/hardware/assurance-sensor-gt/gt-install/m_overview.pdf)'
power-ports:
Copy link
Collaborator

Choose a reason for hiding this comment

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

If they are a DC plug they should be type: dc-terminal

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