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

Recompile plugin with steampipe-plugin-sdk v5.10.0 and add support for connection key columns #52

Closed
wants to merge 2 commits into from

Conversation

misraved
Copy link
Contributor

No description provided.

@misraved misraved self-assigned this Apr 12, 2024
@misraved
Copy link
Contributor Author

@ParthaI instead of profile_id, I believe the account's euuid should be used as a common column.

Reference - https://techdocs.akamai.com/linode-api/reference/get-account

@ParthaI
Copy link
Contributor

ParthaI commented Jul 15, 2024

@misraved, in our plugin, we use the Go SDK github.com/linode/linodego with version v1.9.1. However, its API GetAccount does not return the EUUID property. Support for this property was added in version v1.37.0. We need to update the Linode SDK first, and then we can add this as a connection key qual.

Additionally, I see there are breaking changes between these versions.

Should we proceed with updating the Linode SDK version?

@ParthaI
Copy link
Contributor

ParthaI commented Jul 15, 2024

Closing this PR since the Steampipe SDK version update has been addressed in #55.

There will be separate PR by updating the Linode Go SDK version and the support for connection key quals.

@ParthaI ParthaI closed this Jul 15, 2024
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.

None yet

2 participants