KAAP-295 replace "_" with "-" in tenant group name

[srm6867]

Overview

What this PR does / why we need it

Special notes for your reviewer

Does this PR introduce a user-facing change?

Summary by Bito

This PR refactors tenant group name generation in the keystone connector by standardizing string manipulation. It converts domain and project names to lowercase and replaces underscores with hyphens for consistency. The changes use local variables to improve code readability and maintainability.

Unit tests added: False

Estimated effort to review (1-5, lower is better): 1

[bito-code-review]

Code Review Agent Run #119006

Actionable Suggestions - 0

Review Details

• Files reviewed - 1 · Commit Range: 8de0061..8de0061

  • connector/keystone/keystone.go
• Files skipped - 0
• Tools

  • Whispers (Secret Scanner) - ✔︎ Successful
  • Detect-secrets (Secret Scanner) - ✔︎ Successful

---

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

• /review - Manually triggers a full AI review.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at mithil@platform9.com.

Documentation & Help

• Customize agent settings
• Review rules
• General documentation
• FAQ

AI Code Review powered by

[bito-code-review]

Changelist by Bito

This pull request implements the following key changes.

Key Change	Files Impacted
Feature Improvement - Refactored String Formatting for Standardized Tenant Naming	- keystone.go - Updated the function to convert domain and project names to lowercase and replace underscores with hyphens, ensuring consistent tenant group naming.

[indradhanush]

Is this backwards compatible?

[srm6867]

This is fixed with this PR - https://github.com/platform9/dex/pull/37/files
closing this one.