Skip to content

Conversation

@mulbc
Copy link

@mulbc mulbc commented Jan 28, 2019

SUMMARY

This module allow appending data to a Google Sheet.
This can be useful if you want to gather lots of data from
different amounts of hosts and process it further.

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

drive_append_to_sheet

ADDITIONAL INFORMATION

Using this module it is possible to archive variables in a playbook run into a Google Sheet.
This is especially interesting if you are trying to gather facts across lots of servers and you will want to do some statistical analysis on that data later on.
The idea for this module was started when I realized that just writing to a local file from different hosts in parallel (with delegate_to: localhost) does not work atomically.
#30413
#47322

This module allow appending data to a Google Sheet.
This can be useful if you want to gather lots of data from
different amounts of hosts and process it further.

Signed-off-by: Chris Blum <[email protected]>
@ansibot
Copy link
Contributor

ansibot commented Jan 28, 2019

@GwenaelPellenArkeup @nitaco @bennojoy @dohoangkhiem @erjohnso @GoogleCloudPlatform @rambleraptor @robwagner33 @walbert947

As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add shipit if you would like to see it merged.

click here for bot help

@ansibot
Copy link
Contributor

ansibot commented Jan 28, 2019

@ansibot ansibot added affects_2.8 This issue/PR affects Ansible v2.8 cloud community_review In order to be merged, this PR must follow the community review workflow. gce module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. new_contributor This PR is the first contribution by a new community member. new_module This PR includes a new module. new_plugin This PR includes a new plugin. support:community This issue/PR relates to code supported by the Ansible community. labels Jan 28, 2019
Copy link
Contributor

@dagwieers dagwieers left a comment

Choose a reason for hiding this comment

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

Apart from the cosmetic/doc changes, I have some other remarks:

  • I don't think it belongs in cloud, because that's about managing cloud infrastructure
  • I wonder if it makes sense to have this kind of highly specialized module in Ansible. I am torn between making one module that is a lot more flexible in what it can do, or a set of modules that offers a complete interface. But this module is very specific/limited.

@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed community_review In order to be merged, this PR must follow the community review workflow. needs_triage Needs a first human triage before being processed. labels Jan 28, 2019
Implemented review comments - This consisted mostly
of changes in the documentation, but also
changed the return label from `message` to `msg`

Signed-off-by: Chris Blum <[email protected]>
@rambleraptor
Copy link
Contributor

Hi! I'm one of the engineers working on the GCP modules.

I agree that this probably doesn't belong in the cloud/ folder. I'm not entirely sure where it would belong though.

Any core members have any ideas?

@mulbc
Copy link
Author

mulbc commented Jan 29, 2019

I had a brief discussion with @dagwieers in IRC yesterday and since there isn't anything for the Google Drive API yet and this might create more than one module (and thus might get new folders in the module hierarchy) he created a wiki page for this:
https://github.com/ansible/community/wiki/GDrive
Current goal is to look at which API methods might be interesting as an Ansible module and plan ahead how the general interface of these future Google Drive modules should look like - then proceed here.

@ansibot ansibot added stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. stale_review Updates were made after the last review and the last review is more than 7 days old. labels Feb 6, 2019
@ansibot ansibot removed the stale_review Updates were made after the last review and the last review is more than 7 days old. label Jun 30, 2019
@ansibot ansibot added the stale_review Updates were made after the last review and the last review is more than 7 days old. label Jul 8, 2019
@ansibot ansibot added community_review In order to be merged, this PR must follow the community review workflow. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. stale_review Updates were made after the last review and the last review is more than 7 days old. labels Mar 3, 2020
@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed community_review In order to be merged, this PR must follow the community review workflow. labels Mar 27, 2020
@ansibot ansibot added the stale_review Updates were made after the last review and the last review is more than 7 days old. label Mar 27, 2020
@ansibot ansibot added collection Related to Ansible Collections work collection:community.general needs_collection_redirect https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md labels Apr 29, 2020
@ansibot
Copy link
Contributor

ansibot commented Aug 16, 2020

Thank you very much for your interest in Ansible. Ansible has migrated much of the content into separate repositories to allow for more rapid, independent development. We are closing this issue/PR because this content has been moved to one or more collection repositories.

For further information, please see:
https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md

@ansibot ansibot closed this Aug 16, 2020
@ansible ansible locked and limited conversation to collaborators Sep 13, 2020
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 bot_closed cloud collection:community.general collection Related to Ansible Collections work gce module This issue/PR relates to a module. needs_collection_redirect https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. new_contributor This PR is the first contribution by a new community member. new_module This PR includes a new module. new_plugin This PR includes a new plugin. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. stale_review Updates were made after the last review and the last review is more than 7 days old. 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.

4 participants