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

Implement grafana_folder resource type #170

Merged
merged 4 commits into from
Apr 8, 2019

Conversation

alexconrey
Copy link

Pull Request (PR) description

We found it useful to manage Grafana folders via Puppet. This type was heavily based on the code for grafana_dashboard type.

* initial commit + tests for grafana_folder type
* initial commit to implement grafana_folder resource type

* add required parameters for grafana objects

* remove references to dashboard objects (dashboard used as template)

* update tests

* remove whitespace for rubocop

* rubocop fixes

* fix global var assignment clause logic
@bastelfreak
Copy link
Member

Hi @alexconrey, thanks for the PR. Can you document the new type in the README.md add add acceptance tests for it?

* initial commit to implement grafana_folder resource type

* add required parameters for grafana objects

* remove references to dashboard objects (dashboard used as template)

* update tests

* remove whitespace for rubocop

* rubocop fixes

* fix global var assignment clause logic

* update README.md/created acceptance test for grafana_folder resource
* rubocop whitespace removal

* updates to grafana_folder_spec.rb
@alexconrey
Copy link
Author

@bastelfreak sure thing! I've created the requested acceptance tests as well as updated the README.md file. Please let me know if there's anything else that I need to change 😄

@bastelfreak
Copy link
Member

Thanks for the awesome work!

@bastelfreak bastelfreak merged commit d5067d3 into voxpupuli:master Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants