Skip to content
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

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

Merged
merged 2 commits into from
Aug 28, 2024

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 !

examples/services/fabricv4/utils/utils.py Outdated Show resolved Hide resolved
examples/services/fabricv4/oauth2/__init__.py Outdated Show resolved Hide resolved
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

examples/services/fabricv4/__init__.py Outdated Show resolved Hide resolved
examples/services/fabricv4/cloud_router/__init__.py Outdated Show resolved Hide resolved
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
1 check passed
@thogarty thogarty deleted the CXF-96059_01 branch August 28, 2024 18:02
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