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

Added support for account, externalId and aws credentials related API support with example. #261

Merged
merged 14 commits into from
Sep 27, 2023

Conversation

sharadkesarwani
Copy link
Contributor

@sharadkesarwani sharadkesarwani commented Sep 14, 2023

Added support for AWS account and set credentials related API support with example.

Jira Ticket

Ref: https://spotinst.atlassian.net/browse/SPOTAUT-14860

added support for create, read and delete account
…equest.go ,fixed acct support

Added support for externalId, modified client.go and request.go to avoid credentials fetching from local, fixed acct support, added support for setCredential also however we need to avoid sending AcctId in payload somehow.
fixed set credential API support
removed commented code from awscredentials.go
…nput to SetCredentialInput

Added setters for credential attributes and renamed CreateCredentialInput to SetCredentialInput and CreateCredentialOutput to SetCredentialOutput
Removed support for externalId creation.
…credential

Fixed input variable names in example file of create account and set credential
…t credential struct

Removed updatedAt field from account and updatedAt, createdAt from set credential struct
Copy link
Contributor

@chandra1-n chandra1-n left a comment

Choose a reason for hiding this comment

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

please fix the review comments

@chandra1-n chandra1-n merged commit 32819e8 into main Sep 27, 2023
8 checks passed
@chandra1-n chandra1-n deleted the SPOTAUT-14860-support-acctMgmt-apis branch September 27, 2023 08:19
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