Skip to content

[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

Merged
merged 4 commits into from
Nov 8, 2021
Merged

Conversation

olksdr
Copy link
Contributor

@olksdr olksdr commented Nov 4, 2021

Adding CI jobs to run the acceptance testing:

  • create the default config
  • create a simple pipeline to run the tests

@olksdr olksdr requested review from JoshMock, Crazybus and a team November 4, 2021 10:27
@olksdr olksdr self-assigned this Nov 4, 2021
Copy link
Member

@JoshMock JoshMock left a 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.

Copy link
Member

@jonahbull jonahbull left a 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.


- job:
project-type: pipeline
prune-dead-branches: true
Copy link
Member

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.

Comment on lines 4 to 5
- meta:
cluster: devops-ci
Copy link
Member

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.

Suggested change
- meta:
cluster: devops-ci
- meta:
cluster: devops-ci


##### JOB DEFAULTS

- job:
Copy link
Member

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
Copy link
Member

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
Copy link
Member

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
Copy link
Member

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
Copy link
Member

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.

@olksdr
Copy link
Contributor Author

olksdr commented Nov 5, 2021

@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.

@olksdr olksdr requested review from jonahbull and JoshMock November 5, 2021 06:08
Copy link
Member

@jonahbull jonahbull left a comment

Choose a reason for hiding this comment

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

LGTM

@olksdr olksdr merged commit 71b0c70 into main Nov 8, 2021
@olksdr olksdr deleted the chore/ci-integration branch November 8, 2021 05:40
tobio added a commit that referenced this pull request Oct 1, 2024
* 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]>
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.

3 participants