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 frequency field to logpush job #876

Merged
merged 1 commit into from
May 6, 2022

Conversation

cmackenzie1
Copy link
Contributor

Description

Adds the field frequency to LogpushJob struct.

The frequency at which Cloudflare sends batches of logs to your destination.
Setting frequency to high sends your logs in larger quantities of smaller files.
Setting frequency to low sends logs in smaller quantities of larger files.

https://api.cloudflare.com/#logpush-jobs-create-logpush-job

Has your change been tested?

Updated the existing test to include the new field with the default value of high. This has been part of the Logpush API for a while, just adding it here as well.

Types of changes

What sort of change does your code introduce/modify?

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • This change is using publicly documented (api.cloudflare.com or developers.cloudflare.com) and stable APIs.

Adds the field `frequency` to `LogpushJob` struct.

```
The frequency at which Cloudflare sends batches of logs to your destination.
Setting frequency to high sends your logs in larger quantities of smaller files.
Setting frequency to low sends logs in smaller quantities of larger files.
```

https://api.cloudflare.com/#logpush-jobs-create-logpush-job
@jacobbednarz jacobbednarz merged commit 5c47323 into cloudflare:master May 6, 2022
cmackenzie1 added a commit to cmackenzie1/terraform-provider-cloudflare that referenced this pull request May 19, 2022
Allows for setting the `frequency` on `cloudflare_logpush_job`'s. There are currently two valid choices, `high` and `low`.
These were added to `cloudflare/cloudflare-go` in cloudflare/cloudflare-go#876
cmackenzie1 added a commit to cmackenzie1/terraform-provider-cloudflare that referenced this pull request May 19, 2022
Allows for setting the `frequency` on `cloudflare_logpush_job`'s. There are currently two valid choices, `high` and `low`.
These were added to `cloudflare/cloudflare-go` in cloudflare/cloudflare-go#876
cmackenzie1 added a commit to cmackenzie1/terraform-provider-cloudflare that referenced this pull request May 19, 2022
Allows for setting the `frequency` on `cloudflare_logpush_job`'s. There are currently two valid choices, `high` and `low`.
These were added to `cloudflare/cloudflare-go` in cloudflare/cloudflare-go#876
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