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

rename namespace selector #5

Merged
merged 2 commits into from
Feb 5, 2019
Merged

Conversation

pb82
Copy link
Collaborator

@pb82 pb82 commented Feb 5, 2019

Renames monitoringNamespaceSelector to dashboardNamespaceSelector to make it more clear what is selected here.

Verification steps:

  1. Run the operator from this branch and deploy deploy/examples/Grafany.yaml
  2. Create a second namespace and label it with oc label namespace <ns> monitoring-key=middleware
  3. Watch the operator logs for: "Checking namespace <ns> for dashboards

@pb82 pb82 requested review from david-martin and sedroche February 5, 2019 08:55
@pb82 pb82 merged commit be29e01 into grafana:master Feb 5, 2019
@pb82 pb82 deleted the rename-match-namespace branch February 5, 2019 09:08
@david-martin david-martin mentioned this pull request Apr 30, 2019
9 tasks
kd7lxl referenced this pull request in kd7lxl/grafana-operator Mar 11, 2021
hubeadmin pushed a commit that referenced this pull request Sep 7, 2021
* General linter from github

* Follow the docs

* Fix CI image

* comment out yaml githubactions

* Add api.md to markdown ignore

* Test again

* Update markdown ignore again

* Fix markdown lint (#5)

* Fix markdown lint

* fix lint github action

* don't think i need it

* and i did ^ ^

* file with error in it

* Fix dashboards

* trigger api.md bad file

* linting markdown deploy examples

* Fix row and remove changes to api.md

* remove more api-md

* put super-lint in the big file

* enable github action lint check

* Disable Github actions lint

* Update ./README.md and ignore rules

* Fix some long rows in md

* does the ignore work?

* yes it does

* Update SECURITY.MD

* extra space in env_vars.md
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.

2 participants