Skip to content

Replacing hardcoded values in Installation Wizard for Powerstore#1650

Merged
anandrajak1 merged 5 commits into
mainfrom
installation-wizard-fix
Sep 10, 2025
Merged

Replacing hardcoded values in Installation Wizard for Powerstore#1650
anandrajak1 merged 5 commits into
mainfrom
installation-wizard-fix

Conversation

@sidharth30060
Copy link
Copy Markdown
Contributor

@sidharth30060 sidharth30060 commented Sep 9, 2025

Description

Replacing hardcoded values in Installation Wizard for Powerstore

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
https://github.com/dell/csm/issues/1947

Checklist:

  • Have you run a grammar and spell checks against your submission?
  • Have you tested the changes locally?
  • Have you tested whether the hyperlinks are working properly?
  • Did you add the examples wherever applicable?
  • Have you added high-resolution images?

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 9, 2025

Test Results

79 tests  ±0   79 ✅ ±0   3s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit ef7489a. ± Comparison against base commit 81f9a6c.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Contributor

@shanmydell shanmydell left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@prablr79 prablr79 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@anandrajak1 anandrajak1 left a comment

Choose a reason for hiding this comment

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

Approving for the automation in place

@anandrajak1 anandrajak1 merged commit e47c45c into main Sep 10, 2025
8 checks passed
@anandrajak1 anandrajak1 deleted the installation-wizard-fix branch September 10, 2025 12:44
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.

6 participants