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.

Sign up for GitHub

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

Jump to bottom

Update cm_service module #265

Merged
merged 58 commits into from
Jan 14, 2025
Merged

Update cm_service module #265

merged 58 commits into from
Jan 14, 2025

Conversation

wmudge
Copy link
Member

@wmudge wmudge commented Jan 10, 2025

Addresses full lifecycle management of the Cloudera Manager service and its subelements (service-wide config, role config groups, and roles).

Requires #258 , #259, #260, #261, and #262 -- will rebase once these PRs are committed or resolved. However, this PR represents the culmination of all this prior work, so we might commit this PR as is and close the above mentioned PRs as obsolete.

wmudge added 30 commits December 18, 2024 12:31
@wmudge
Update pytest marker for 'service_config'
4df1dcd 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update message for cms_service_config changes
4d0dea7 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Move role and role config group result parsing to separate modules. U…
c9798f1 
…pdate general output parsing function.

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Enable service-wide configuration management. Update return object.
e72cb3b 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update to use pytest fixtures for Cloudera Manager Service
7612a7e 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update conn object from fixture
0b5c98f 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Create generic configuration list reconcilation utility
a91c59f 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add cm_service_role_config module
fdfbfb1 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add utilities for discovering base role config group for a given role…
c0a82f9 
… type

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add utilities for getting roles by role type
eab3d18 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add utility to set Cloudera Manager Service role config group configu…
6191de9 
…rations for tests

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add cm_service_role_config_group module and tests
8f6334e 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add 'config' dictionary to parsed Role Config Group results
a4e6394 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update utlity function for Role Config Group tests to handle all para…
1a275f2 
…meters, not just configuration values

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Move host_monitor and host_monitor_config fixtures and update pytest …
44af849 
…Marker

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update to use reworked host_monitor_config fixture
72418dc 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add cm_service_role_config_group module and tests
c7602e5 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add host utilities
f7835a8 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add utility function for role config group retrieval
e4f5e9f 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add config parameters to parse_role_result. Add utility for role data…
e287cbf 
… model creation.

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Rename Host Monitor fixtures. Add host_monitor_state for general role…
6904d89 
… testing

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add cm_service_role and tests
26402b2 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add 'role' marker
092d3dc 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Consolidate host_monitor role
904b9c2 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Fix invalid fixture references
44e0fc0 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update testing requirements.txt
159d0d0 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Fix name of Cloudera Manager client library
516af16 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add pytest comment and update formatting
4a446ac 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update docstrings and add missing imports for fixtures and fixture ut…
3558133 
…ilities

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Remove unused imports
7c5aa31 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update cm_service_info to use common utilities for collection and par…
b57bb9f 
…sing of Cloudera Manager service

Signed-off-by: Webster Mudge <[email protected]>
wmudge added 18 commits January 10, 2025 16:44
@wmudge
Rename and move return value reference YAML
a62cb3a 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add new modules to module action group in collection
256abbe 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Remove unused function
853d3f3 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add diff_mode and platform documentation attributes
68ce861 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update cm_service_role_config_group and tests
d47f5fc 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add cm_service_role_config_group_info module
95f8694 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Remove host_monitor_state fixture
becefc7 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update to remove custom Exceptions
eefc7f2 
Remove get_role_config_group

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Rename get_roles to read_roles and update to retrieve role override c…
abc0995 
…onfigurations

Add read_role, read_roles_by_type functions
Update create_role to handle errors for invalid role config group assignment

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update to remove custom Exception handling
5f1ef32 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add cm_service_role_info module and tests
d316425 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update documentation and reconcilation logic for maintenance and oper…
200b915 
…ational state

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update for non-existence of CM service
13d05e9 
Update test to use pytest and fixtures

Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Remove view parameter
17e25ed 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add documentation for CM service role return values
77c8e35 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add 'role_state' to pytest markers
6634e82 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Update pytest fixture usage
67493b3 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge
Add test for cm_service_role_config_info
d3f5d57 
Signed-off-by: Webster Mudge <[email protected]>
@wmudge wmudge marked this pull request as ready for review January 13, 2025 19:48
jimright
jimright approved these changes Jan 14, 2025
View reviewed changes
Copy link
Contributor

@jimright jimright left a comment

Choose a reason for hiding this comment

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

Looks good. Approved.

@wmudge wmudge added this to the Release 4.5.0 milestone Jan 14, 2025
@wmudge wmudge merged commit a397664 into cloudera-labs:devel Jan 14, 2025
8 checks passed
@wmudge wmudge deleted the feature/cm_service-module branch January 14, 2025 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jimright jimright jimright approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request validated
Milestone
Release 4.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@wmudge @jimright