Skip to content

Rename datasources to config_templates - #132

Merged
ruflin merged 5 commits into
elastic:masterfrom
ruflin:config_templates
Jun 30, 2020
Merged

Rename datasources to config_templates#132
ruflin merged 5 commits into
elastic:masterfrom
ruflin:config_templates

Conversation

@ruflin

@ruflin ruflin commented Jun 30, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor

This renames the datasources: part in the dataset manifest to config_templates:.
@ruflin ruflin self-assigned this Jun 30, 2020
@ruflin

ruflin commented Jun 30, 2020

Copy link
Copy Markdown
Contributor Author

This requires elastic/package-registry#570 to be merged first and then the dependency is updated to update the last two usages of DataSources.
@ruflin ruflin mentioned this pull request Jun 30, 2020
Closed
29 tasks
@elasticmachine

elasticmachine commented Jun 30, 2020 •
edited
Loading

Copy link
Copy Markdown

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #132 updated]

  • Start Time: 2020-06-30T09:27:49.990+0000

  • Duration: 2 min 21 sec
mtojek
mtojek reviewed Jun 30, 2020
View reviewed changes

@mtojek mtojek left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actually the file datasources.go needs to be renamed.
@mtojek mtojek mentioned this pull request Jun 30, 2020
Merged
11 tasks
@ruflin

ruflin commented Jun 30, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@mtojek good catch, fixed. Wrong thread, but will update the datasources.go
@mtojek
mtojek self-requested a review June 30, 2020 07:34
mtojek
mtojek approved these changes Jun 30, 2020
View reviewed changes

@mtojek mtojek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved, but need to track if somebody pushes an incompatible package.
@ruflin

ruflin commented Jun 30, 2020

Copy link
Copy Markdown
Contributor Author

@mtojek Updated again with quite a few more changes I found.

Is CI validating of the imports-beat code is valid?
@mtojek

mtojek commented Jun 30, 2020

Copy link
Copy Markdown
Contributor

Is CI validating of the imports-beat code is valid?

Nope, unfortunately it doesn't. It wasn't urgent so far (also the future of this script is blurry).

Side note: there was Kubernetes integration pushed to master. I think you need to rebase your PR.
@ruflin
ruflin force-pushed the config_templates branch from 972c6cf to b0aec0a Compare June 30, 2020 09:27
@ruflin
ruflin requested a review from mtojek June 30, 2020 09:27
@ruflin

ruflin commented Jun 30, 2020

Copy link
Copy Markdown
Contributor Author

@mtojek Rebased and updated with most recent packages.
@ruflin
ruflin merged commit 8836aa5 into elastic:master Jun 30, 2020
@ruflin
ruflin deleted the config_templates branch June 30, 2020 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@ruflin @elasticmachine @mtojek