Skip to content

Fix Control Center Next Gen custom file copy variable - #2471

Open
박성제 (chuu8319) wants to merge 1 commit into
confluentinc:7.9.xfrom
chuu8319:fix/c3-next-gen-copy-files
Open

Fix Control Center Next Gen custom file copy variable#2471
박성제 (chuu8319) wants to merge 1 commit into
confluentinc:7.9.xfrom
chuu8319:fix/c3-next-gen-copy-files

Conversation

@chuu8319

Copy link
Copy Markdown

Description

This PR fixes a variable mismatch in the Control Center Next Gen custom file copy task on 7.9.x.

The task currently references control_center_copy_files, but for Control Center Next Gen the correct variable is control_center_next_gen_copy_files.

Because of this, custom files defined for Control Center Next Gen are not copied.

This change updates both the 'copy_files' mapping and the when'condition to use control_center_next_gen_copy_files.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Tested manually on 7.9.x with inventory defining control_center_next_gen_copy_files.

Steps:

  1. Define control_center_next_gen_copy_files with a test file.
  2. Run the Control Center Next Gen play.
  3. Observe that before the fix the file is not copied.
  4. Apply the fix and rerun.
  5. Confirm that the file is copied successfully.

Checklist:

  • Any variable/code changes have been validated to be backwards compatible (doesn't break upgrade)
  • I have added tests that prove my fix is effective or that my feature works
  • If required, I have ensured the changes can be discovered by cp-ansible discovery codebase
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@chuu8319
박성제 (chuu8319) requested a review from a team as a code owner April 8, 2026 05:09
@chuu8319

Copy link
Copy Markdown
Author

Friendly ping on this PR.

The change fixes what appears to be a variable mismatch in the Control Center Next Gen role and should be backwards compatible. I've verified the behavior manually on 7.9.x.

Could someone please take a look when time permits? Happy to make any requested adjustments.

Thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant