Skip to content

CSPL-3580: Fix: Update Variable Formatting in deploy-eks-cluster.sh to Use ${variablename} for EBS Service Name Annotations #1473

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vivekr-splunk
Copy link
Collaborator

Title: Fix: Update Variable Formatting in deploy-eks-cluster.sh to Use ${variablename} for EBS Service Name Annotations

Description:

This PR addresses the bug in the deploy-eks-cluster.sh script where the incorrect formatting of variables was causing failures when adding IAM roles to the service account. Specifically, the issue stemmed from using $variablename instead of ${variablename} for variable references, leading to incorrectly formatted values in the EBS service name annotations.

Changes Made:

  • Updated the variable formatting in deploy-eks-cluster.sh from $variablename to ${variablename} to ensure proper handling in different shells.

Impact:

  • This fix ensures that the correct variable references are used, preventing incorrect formatting and ensuring that IAM roles, EBS volumes, and role policies are correctly applied.
  • The issue of the wrong name being added to the EBS service name annotations is resolved.

How to Test:

  1. Run the make cluster-up command.
  2. Verify that the cluster is created successfully without failures related to the incorrect formatting of variables.
  3. Ensure that the correct names are added to the EBS service name annotations.

Related Jira Ticket: CSPL-3580 - Incorrect Formatting of IAM Role for Service Account

Expected Outcome:

  • The script should run successfully and add the IAM role and EBS volume correctly, with accurate service name annotations.

Signed-off-by: Vivek Reddy <[email protected]>
@coveralls
Copy link
Collaborator

coveralls commented Mar 21, 2025

Pull Request Test Coverage Report for Build 14001571002

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.008%) to 86.59%

Totals Coverage Status
Change from base Build 13946362371: 0.008%
Covered Lines: 10538
Relevant Lines: 12170

💛 - Coveralls

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.

5 participants