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

Add influxdb_user module #2197

Closed

aisch wants to merge 1 commit into ansible:devel from aisch:influxdb_user

aisch commented May 9, 2016

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

influxdb_user

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 users. Modeled on influxdb_database module.

alikins added feature_pull_request new_plugin community_review core_review labels

alikins closed this

alikins reopened this

jimi-c added the in progress label

gregdek added needs_revision and removed community_review core_review labels

Contributor

gregdek commented May 9, 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.]

aisch force-pushed the influxdb_user branch from c7bcae1 to dad6d48 Compare

May 9, 2016 22:33

Author

aisch commented May 9, 2016

ready_for_review

Contributor

gregdek commented May 10, 2016

Thanks @aisch for this new module. When this module receives 'shipit' comments from two community members and any 'needs_revision' comments have been resolved, we will mark for inclusion.

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

gregdek added community_review and removed needs_revision labels


          Add influxdb_user module

6271edd

aisch force-pushed the influxdb_user branch from b8653b4 to 6271edd Compare

May 13, 2016 19:18

Author

aisch commented May 13, 2016

now requires influxdb pr influxdata/influxdb-python#322 (for admin:true|false after create)

gregdek removed the community_review label

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.]

gregdek added the needs_revision label

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.]

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.]

gregdek added the pending_action label

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.]

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.]

Contributor

resmo commented Jul 17, 2016

@aisch I would suggest you remove the dependency of the influx PR so this can be marked for inclusion. If the PR has been merged, the functionality could be implemented afterwards.

resmo removed the pending_action label

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.]

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 added the pending_action label

Contributor

gregdek commented Aug 15, 2016

Closing due to contributor inactivity.

gregdek closed this

gregdek removed in progress needs_revision pending_action labels

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

Labels

feature_pull_request new_plugin

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