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

Add Parcel Module #221

Merged
merged 5 commits into from
May 3, 2024
Merged

Conversation

rsuplina
Copy link
Contributor

Parcel Module:

  • Facilitates the management of parcels on a Cluster by downloading, distributing, and activating them according to the specified state.
  • Supported states include 'download', 'distribute', and 'activate', each corresponding to specific actions performed on parcels.

Signed-off-by: rsuplina <[email protected]>
@rsuplina rsuplina added the enhancement New feature or request label Apr 15, 2024
@rsuplina rsuplina requested a review from a team April 15, 2024 11:32
Copy link
Member

@wmudge wmudge left a comment

Choose a reason for hiding this comment

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

Need to add a new module for parcel_info which will gather the details on one or all parcels declared for a cluster.

plugins/modules/parcel.py Outdated Show resolved Hide resolved
plugins/modules/parcel.py Outdated Show resolved Hide resolved
plugins/modules/parcel.py Show resolved Hide resolved
plugins/modules/parcel.py Outdated Show resolved Hide resolved
plugins/modules/parcel.py Outdated Show resolved Hide resolved
plugins/modules/parcel.py Show resolved Hide resolved
Signed-off-by: rsuplina <[email protected]>
Signed-off-by: rsuplina <[email protected]>
- The name of the product, e.g. CDH, Impala
type: str
required: yes
version:
Copy link
Member

Choose a reason for hiding this comment

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

This will conflict with the version parameter common to all cloudera.cluster modules via the ClouderaManagerModule class (and the doc_fragment)

I would like to change this common parameter to something like api_version and allow modules like is one to use version as they need. Until then, you will need to "prefix" this parameter, so something like parcel_version?

@github-actions github-actions bot removed the validated label May 3, 2024
@wmudge wmudge self-requested a review May 3, 2024 13:17
@wmudge wmudge merged commit d4718ff into cloudera-labs:devel May 3, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request validated
Development

Successfully merging this pull request may close these issues.

2 participants