Skip to content

Comments

feat: Adding Examples for Python SDK to create FCR and FCR based Connections#22

Merged
thogarty merged 2 commits intomainfrom
CXF-96059_01
Aug 28, 2024
Merged

feat: Adding Examples for Python SDK to create FCR and FCR based Connections#22
thogarty merged 2 commits intomainfrom
CXF-96059_01

Conversation

@jkallem-equinix
Copy link
Contributor

CXF-96059 Adding Examples for Python SDK:

Fabric Cloud Router

  1. Create FCR
  2. GET FCR
  3. Update FCR
  4. Delete FCR
    FCR to Port, AWS, Azure Connections:

Connections

  1. Create Connection
  2. GET Connection
  3. Update Connection
  4. Delete Connection

@jkallem-equinix jkallem-equinix requested a review from a team as a code owner August 21, 2024 20:25
Copy link
Contributor

@thogarty thogarty left a comment

Choose a reason for hiding this comment

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

Some nitpicking to clean up. Rest looks good though. Thanks, @jkallem-equinix !

This method interacts with the Equinix Fabric API to fetch detailed information about a specific connection
identified by its UUID. The response is then formatted into a JSON string for easier readability and logged
for debugging purposes.
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: closing triple quotes not at same level of indentation as openers

This method handles the process of establishing a connection between a Fabric Cloud Router (FCR) and a
colocation or another FCR within the Equinix Fabric. It generates the connection request payload,
sends the request to the Equinix Fabric API, and returns the unique identifier (UUID) of the created connection.
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: closing triple quotes not at same level of indentation as openers

Copy link
Contributor

@thogarty thogarty left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks, @jkallem-equinix !

@thogarty thogarty merged commit 785119e into main Aug 28, 2024
@thogarty thogarty deleted the CXF-96059_01 branch August 28, 2024 18:02
@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2024

This PR is included in version 0.4.0 🎉

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