-
Notifications
You must be signed in to change notification settings - Fork 108
[ci] add jobs to run the acceptance tests #1
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One question but I'm okay with merging regardless.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great start! Needs some tweaks before the YAML will parse correctly but overall looks like a sensible application of the how-to stuff.
.ci/jobs/defaults.yml
Outdated
|
||
- job: | ||
project-type: pipeline | ||
prune-dead-branches: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is only valid for multibranch pipeline jobs.
.ci/jobs/defaults.yml
Outdated
- meta: | ||
cluster: devops-ci |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For these top-level properties, the child items need to be indented or the YAML won't get parsed correctly.
- meta: | |
cluster: devops-ci | |
- meta: | |
cluster: devops-ci |
|
||
##### JOB DEFAULTS | ||
|
||
- job: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue as above, the other properties that are part of the job map need to be indented.
@@ -0,0 +1,13 @@ | |||
--- | |||
- job: | |||
name: elastic+terraform-provider-elasticstack+main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same indentation comment as elsewhere.
@@ -0,0 +1,21 @@ | |||
--- | |||
- job: | |||
name: elastic+terraform-provider-elasticstack+pull-request |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same indentation comment as elsewhere.
name: elastic+terraform-provider-elasticstack+main | ||
display-name: elastic / terraform-provider-elasticstack - main | ||
description: Main branch testing | ||
project-type: pipeline |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you have this set in the defaults you could get away with not setting it in the jobs.
name: elastic+terraform-provider-elasticstack+pull-request | ||
display-name: elastic / terraform-provider-elasticstack - pull-request | ||
description: Pull request testing | ||
project-type: pipeline |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you have this set in the defaults you could get away with not setting it in the jobs.
@jonahbull thanks for the review and pointing out the formatting issues. Looks like I overdid the formatting steps after some local testing 😄 . Please, take another look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Add URL support to data_view field_formats (#1) - Adds support for `urlTemplate` and `labelTemplate` to data_views.field_formats - Amends API request build process to account for when some/any data_views.field_formats.attributes aren't passed in * Update internal/kibana/data_view/schema.go Co-authored-by: Toby Brain <[email protected]> * Update internal/kibana/data_view/schema.go Co-authored-by: Toby Brain <[email protected]> * PR changes --------- Co-authored-by: Toby Brain <[email protected]>
Adding CI jobs to run the acceptance testing: