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

Add table ovh_log_self #41

Closed

Conversation

romain-pix-cyber
Copy link
Contributor

This PR adds support for a ovh_log_self table.

It fetches logs for previous API calls made in OVH Public Cloud API with one's own account, using this endpoint.

  • There might be some design conventions errors as I am new to steampipe development
  • in order to fetch the logs, i type casted the LogId from int to string to use pre existing methods

Makefile Outdated
Copy link
Owner

Choose a reason for hiding this comment

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

Hi, could you clean the Makefile?

ovh/plugin.go Outdated
@@ -17,6 +17,7 @@ func Plugin(ctx context.Context) *plugin.Plugin {
},
TableMap: map[string]*plugin.Table{
"ovh_bill": tableOvhBill(),
"ovh_log_self": tableOvhLog(),
Copy link
Owner

Choose a reason for hiding this comment

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

Could you sort the lines?

@francois2metz
Copy link
Owner

Hi,

Thanks for your contribution. I'll test it later.

select
id,
date,
acount
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
acount
account

@francois2metz
Copy link
Owner

Hi,

I tested it, works fine, good job. Once you fix the typo in the doc I'll merge and release a new version.

@francois2metz francois2metz changed the title Ovh logs Add table ovh_log_self Dec 28, 2024
@francois2metz
Copy link
Owner

Merged in ad3573f. Thanks!

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