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

postgresql_idx: misc fixes #50199

Merged
merged 2 commits into from Dec 21, 2018
Merged

Conversation

Akasurde
Copy link
Member

SUMMARY
  • Removed unused import
  • Removed 'login_password' from exit_json call
  • Removed unused db, cond variable

Signed-off-by: Abhijeet Kasurde akasurde@redhat.com

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

lib/ansible/modules/database/postgresql/postgresql_idx.py

* Removed unused import
* Removed 'login_password' from exit_json call
* Removed unused db, cond variable

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@ansibot
Copy link
Contributor

ansibot commented Dec 20, 2018

@ansibot ansibot added affects_2.8 This issue/PR affects Ansible v2.8 bug This issue/PR relates to a bug. community_review In order to be merged, this PR must follow the community review workflow. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. support:community This issue/PR relates to code supported by the Ansible community. labels Dec 20, 2018
Copy link
Contributor

@Andersson007 Andersson007 left a comment

Choose a reason for hiding this comment

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

Thank you

@@ -17,7 +17,7 @@
DOCUMENTATION = r'''
---
module: postgresql_idx
short_description: Creates or drops indexes from a PostgreSQL database.
Copy link
Contributor

Choose a reason for hiding this comment

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

See short descriptions of other modules. Most of them ends by ".", e.g. shell_module, postgresql_db, postgresql_user, etc.
Ok, it doesn't matter but I prefer sentences that end with "." )

@@ -127,7 +127,7 @@
idxname: test_gist_idx

# Create gin index gin0_idx not concurrently on column comment of table test
# (Note: pg_trgm extention must be installed for gin_trgm_ops)
Copy link
Contributor

Choose a reason for hiding this comment

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

Good

@Andersson007
Copy link
Contributor

shipit

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@ansibot ansibot removed the needs_triage Needs a first human triage before being processed. label Dec 21, 2018
@Akasurde Akasurde merged commit fc5ea7c into ansible:devel Dec 21, 2018
@Akasurde Akasurde deleted the postgresql_idx_fix branch December 21, 2018 11:01
@Akasurde
Copy link
Member Author

@Andersson007 Thanks for the reviews.

kbreit pushed a commit to kbreit/ansible that referenced this pull request Jan 11, 2019
* Removed unused import
* Removed 'login_password' from exit_json call
* Removed unused db, cond variable

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@dagwieers dagwieers added the postgresql PostgreSQL community label Feb 1, 2019
@dagwieers dagwieers added the database Database category label Feb 13, 2019
@ansible ansible locked and limited conversation to collaborators Jul 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.8 This issue/PR affects Ansible v2.8 bug This issue/PR relates to a bug. community_review In order to be merged, this PR must follow the community review workflow. database Database category module This issue/PR relates to a module. postgresql PostgreSQL community support:community This issue/PR relates to code supported by the Ansible community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants