Skip to content

Conversation

@sarahbx
Copy link
Contributor

@sarahbx sarahbx commented Feb 21, 2023

Short description:

Add Grafana Resources for grafana-operator
https://github.com/grafana-operator/grafana-operator/

@redhat-qe-bot
Copy link
Contributor

The following are automatically added:

  • Add reviewers from OWNER file (in the root of the repository) under reviewers section.
  • Set PR size label.
  • New issue is created for the PR.

Available user actions:

  • To mark PR as verified add !verified to a PR comment, to un-verify add !-verified to a PR comment.
    Verified label removed on each new commit push.
  • To cherry pick a merged PR add !cherry-pick <target branch to cherry-pick to> to a PR comment.
  • To add a label by comment use !<label name>, to remove, use !-<label name>
    Supported labels:
  • do-not-merge
  • verified
  • wip
  • lgtm
  • target-version-

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

from ocp_resources.resource import NamespacedResource


class Grafana(NamespacedResource):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Empty class without to_dict implement required is not allowed.
Please introduce to_dict().

from ocp_resources.resource import NamespacedResource


class GrafanaDataSource(NamespacedResource):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Empty class without to_dict implement required is not allowed.
Please introduce to_dict().


"""
super().__init__(name=name, namespace=namespace, yaml_file=yaml_file, **kwargs)
self.label = label
Copy link
Collaborator

Choose a reason for hiding this comment

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

All of those required? Please add verification for required args under if not self.yaml_file:

@sarahbx
Copy link
Contributor Author

sarahbx commented Mar 29, 2023

on hold due to recent release of v5 which altered the specs of the resources.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants