Skip to content
This repository was archived by the owner on Oct 30, 2018. It is now read-only.

Add influxdb_privs module #2231

Closed
wants to merge 1 commit into from
Closed

Add influxdb_privs module #2231

wants to merge 1 commit into from

Conversation

aisch
Copy link

@aisch aisch commented May 13, 2016

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

influxdb_privs

ANSIBLE VERSION
ansible 2.0.1.0
  config file = /home/ai/.ansible.cfg
  configured module search path = Default w/o overrides
SUMMARY

Adds a module for creating or dropping influxdb privileges. Modeled on influxdb_database module.

@aisch
Copy link
Author

aisch commented May 13, 2016

requires influxdb pr influxdata/influxdb-python#322

@gregdek
Copy link
Contributor

gregdek commented May 13, 2016

Thanks @aisch for this PR. This PR requires revisions, either because it fails to build or by reviewer request. Please make the suggested revisions. When you are done, please comment with text 'ready_for_review' and we will put this PR back into review.

[This message brought to you by your friendly Ansibull-bot.]

kamsz
kamsz reviewed May 18, 2016
View reviewed changes
database/influxdb/influxdb_privs.py
version_added: 2.2
requirements:
- "python >= 2.6"
- "influxdb >= 0.9"
Copy link
Contributor

Choose a reason for hiding this comment

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

since this module requires a change into influxdb python module, i'd suggest to put version of influxdb-python module here when influxdb PR gets merged

@gregdek
Copy link
Contributor

gregdek commented May 29, 2016

@aisch A friendly reminder: this pull request has been marked as needing your action. If you still believe that this PR applies, and you intend to address the issues with this PR, just let us know in the PR itself and we will keep it open pending your changes. When you do address the issues, please respond with ready_for_review in your comment, so that we can notify the maintainer.

[This message brought to you by your friendly Ansibull-bot.]

@gregdek
Copy link
Contributor

gregdek commented Jun 13, 2016

@aisch Another friendly reminder: this pull request has been marked as needing your action. If you still believe that this PR applies, and you intend to address the issues with this PR, just let us know in the PR itself and we will keep it open. If you have addressed the issues and believe it's ready for review, please comment with the text "ready_for_review". If we don't hear from you within another 14 days, we will close this pull request.

[This message brought to you by your friendly Ansibull-bot.]

@gundalow
Copy link
Contributor 

$ ansible-validate-modules .
============================================================================
./database/influxdb/influxdb_privs.py
============================================================================
ERROR: requests import found, should use ansible.module_utils.urls instead

need_revision

@gregdek
Copy link
Contributor

gregdek commented Jun 29, 2016

@aisch A friendly reminder: this pull request has been marked as needing your action. If you still believe that this PR applies, and you intend to address the issues with this PR, just let us know in the PR itself and we will keep it open pending your changes. When you do address the issues, please respond with ready_for_review in your comment, so that we can notify the maintainer.

[This message brought to you by your friendly Ansibull-bot.]

@gregdek
Copy link
Contributor

gregdek commented Jul 14, 2016

@aisch Another friendly reminder: this pull request has been marked as needing your action. If you still believe that this PR applies, and you intend to address the issues with this PR, just let us know in the PR itself and we will keep it open. If you have addressed the issues and believe it's ready for review, please comment with the text "ready_for_review". If we don't hear from you within another 14 days, we will close this pull request.

[This message brought to you by your friendly Ansibull-bot.]

@gregdek
Copy link
Contributor

gregdek commented Jul 30, 2016

@aisch A friendly reminder: this pull request has been marked as needing your action. If you still believe that this PR applies, and you intend to address the issues with this PR, just let us know in the PR itself and we will keep it open pending your changes. When you do address the issues, please respond with ready_for_review in your comment, so that we can notify the maintainer.

[This message brought to you by your friendly Ansibull-bot.]

@gregdek
Copy link
Contributor

gregdek commented Aug 14, 2016

@aisch Another friendly reminder: this pull request has been marked as needing your action. If you still believe that this PR applies, and you intend to address the issues with this PR, just let us know in the PR itself and we will keep it open. If you have addressed the issues and believe it's ready for review, please comment with the text "ready_for_review". If we don't hear from you within another 14 days, we will close this pull request.

[This message brought to you by your friendly Ansibull-bot.]

@gregdek
Copy link
Contributor

gregdek commented Aug 15, 2016

Closing due to contributor inactivity.

@gregdek gregdek closed this Aug 15, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
new_plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aisch @gregdek @gundalow @kamsz
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy