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

Feature: add cron setting for the ldap settings #16673

Merged
merged 2 commits into from Apr 25, 2019
Merged

Feature: add cron setting for the ldap settings #16673

merged 2 commits into from Apr 25, 2019

Conversation

markelog
Copy link
Contributor

  • Move ldap configuration read to special function

  • Introduce cron setting (no docs for it yet, pending approval)

Copy link
Member

@torkelo torkelo left a comment

Choose a reason for hiding this comment

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

LGTM
@xlson do you have any thoughts?

Copy link
Contributor

@xlson xlson left a comment

Choose a reason for hiding this comment

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

Looks good! Add a comment explaining the setting and merge.

@@ -393,6 +393,7 @@ headers =
enabled = false
config_file = /etc/grafana/ldap.toml
allow_sign_up = true
sync_cron = @hourly
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd add a comment explaining that this is related to ldap sync in enterprise.

Something like: # LDAP backround sync (Enterprise)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, I definitely want to do that! Was thinking to add such comment with the rest of the documentation, but I can do it in this pull right away :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Great! You can merge this when you have added the comment.

* Move ldap configuration read to special function

* Introduce cron setting (no docs for it yet, pending approval)
@markelog markelog merged commit 78cd905 into master Apr 25, 2019
ryantxu added a commit to ryantxu/grafana that referenced this pull request Apr 25, 2019
* grafana/master:
  GraphLegendEditor: use stats picker rather than switches (grafana#16759)
  Feature: add cron setting for the ldap settings (grafana#16673)
  Build: Disables gosec until identified performance problems (grafana#16764)
  Chore: bump jQuery to 3.4.0 including prototype pollution vulnerability fix (grafana#16761)
  elasticsearch: add 7.x version support (grafana#16646)
  Provisioning: Add API endpoint to reload provisioning configs (grafana#16579)
  Config: Show user-friendly error message instead of stack trace (grafana#16564)
  Chore: Lowered implicit anys limit to 5954
  Feature: Enable React based options editors for Datasource plugins (grafana#16748)
  sqlstore: use column name in order by (grafana#16583)
  user friendly guide (grafana#16743)
  Provisioning: Interpolate env vars in provisioning files (grafana#16499)
  admin: add more stats about roles (grafana#16667)
  Feature: Migrate Legend components to grafana/ui (grafana#16468)
  playlist: fix loading dashboards by tag (grafana#16727)
  CloudWatch: Use default alias if there is no alias for metrics (grafana#16732)
  Provisioning: Support FolderUid in Dashboard Provisioning Config (grafana#16559)
ryantxu added a commit to ryantxu/grafana that referenced this pull request Apr 25, 2019
…MetricPanelCtrl

* grafana/master: (34 commits)
  Streaming: support streaming and a javascript test datasource (grafana#16729)
  GraphLegendEditor: use stats picker rather than switches (grafana#16759)
  Feature: add cron setting for the ldap settings (grafana#16673)
  Build: Disables gosec until identified performance problems (grafana#16764)
  Chore: bump jQuery to 3.4.0 including prototype pollution vulnerability fix (grafana#16761)
  elasticsearch: add 7.x version support (grafana#16646)
  Provisioning: Add API endpoint to reload provisioning configs (grafana#16579)
  Config: Show user-friendly error message instead of stack trace (grafana#16564)
  Chore: Lowered implicit anys limit to 5954
  Feature: Enable React based options editors for Datasource plugins (grafana#16748)
  sqlstore: use column name in order by (grafana#16583)
  user friendly guide (grafana#16743)
  Provisioning: Interpolate env vars in provisioning files (grafana#16499)
  admin: add more stats about roles (grafana#16667)
  Feature: Migrate Legend components to grafana/ui (grafana#16468)
  playlist: fix loading dashboards by tag (grafana#16727)
  CloudWatch: Use default alias if there is no alias for metrics (grafana#16732)
  Provisioning: Support FolderUid in Dashboard Provisioning Config (grafana#16559)
  Refactor: Make SelectOptionItem a generic type to enable select value typing (grafana#16718)
  docs: fix upgrade instructions
  ...
ryantxu added a commit to ryantxu/grafana that referenced this pull request Apr 25, 2019
* grafana/master:
  Streaming: support streaming and a javascript test datasource (grafana#16729)
  GraphLegendEditor: use stats picker rather than switches (grafana#16759)
  Feature: add cron setting for the ldap settings (grafana#16673)
  Build: Disables gosec until identified performance problems (grafana#16764)
  Chore: bump jQuery to 3.4.0 including prototype pollution vulnerability fix (grafana#16761)
  elasticsearch: add 7.x version support (grafana#16646)
  Provisioning: Add API endpoint to reload provisioning configs (grafana#16579)
  Config: Show user-friendly error message instead of stack trace (grafana#16564)
  Chore: Lowered implicit anys limit to 5954
  Feature: Enable React based options editors for Datasource plugins (grafana#16748)
  sqlstore: use column name in order by (grafana#16583)
  user friendly guide (grafana#16743)
  Provisioning: Interpolate env vars in provisioning files (grafana#16499)
  admin: add more stats about roles (grafana#16667)
  Feature: Migrate Legend components to grafana/ui (grafana#16468)
  playlist: fix loading dashboards by tag (grafana#16727)
  CloudWatch: Use default alias if there is no alias for metrics (grafana#16732)
  Provisioning: Support FolderUid in Dashboard Provisioning Config (grafana#16559)
ryantxu added a commit to ryantxu/grafana that referenced this pull request Apr 25, 2019
* grafana/master: (31 commits)
  Streaming: support streaming and a javascript test datasource (grafana#16729)
  GraphLegendEditor: use stats picker rather than switches (grafana#16759)
  Feature: add cron setting for the ldap settings (grafana#16673)
  Build: Disables gosec until identified performance problems (grafana#16764)
  Chore: bump jQuery to 3.4.0 including prototype pollution vulnerability fix (grafana#16761)
  elasticsearch: add 7.x version support (grafana#16646)
  Provisioning: Add API endpoint to reload provisioning configs (grafana#16579)
  Config: Show user-friendly error message instead of stack trace (grafana#16564)
  Chore: Lowered implicit anys limit to 5954
  Feature: Enable React based options editors for Datasource plugins (grafana#16748)
  sqlstore: use column name in order by (grafana#16583)
  user friendly guide (grafana#16743)
  Provisioning: Interpolate env vars in provisioning files (grafana#16499)
  admin: add more stats about roles (grafana#16667)
  Feature: Migrate Legend components to grafana/ui (grafana#16468)
  playlist: fix loading dashboards by tag (grafana#16727)
  CloudWatch: Use default alias if there is no alias for metrics (grafana#16732)
  Provisioning: Support FolderUid in Dashboard Provisioning Config (grafana#16559)
  Refactor: Make SelectOptionItem a generic type to enable select value typing (grafana#16718)
  docs: fix upgrade instructions
  ...
@markelog markelog deleted the cron branch May 27, 2019 11:22
@ying-jeanne ying-jeanne added the pr/external This PR is from external contributor label Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/external This PR is from external contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants