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.

Sign up for GitHub

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

Jump to bottom

[VPC-3917] Support PIA get service key operation #1663

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[VPC-3917] Support PIA get service key operation #1663

wants to merge 2 commits into from

Conversation

guptado
Copy link
Contributor

@guptado guptado commented Feb 19, 2025 •
edited
Loading

Testing:

➜  builds git:(VPC-3917/get_sk) ✗ ./doctl --context new_s2 network --type "partner" interconnect-attachment get-service-key 7b48cbc3-cacb-418e-908c-a9a86189a642
Value                                   State      CreatedAt
8f6d51fe-XXXXXXXXXX   CREATED    2025-02-17 15:07:29 +0000 UTC

apinonformoso
apinonformoso reviewed Feb 19, 2025
View reviewed changes
go.mod
@@ -39,6 +39,8 @@ require (
sigs.k8s.io/yaml v1.3.0
)

replace github.com/digitalocean/godo v1.138.0 => github.com/guptado/godo v0.0.0-20250219083849-2f80d81d1f61
Copy link
Contributor

Choose a reason for hiding this comment

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

Here you shouldn't be pointing to a local godo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

aah yes. I have created another pr in godo with new updated model. as soon as we get the new godo releae version i can use it. I did above replace to test my code with new godo changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apinonformoso apinonformoso apinonformoso left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@guptado @apinonformoso