Skip to content

Fixing unknown flags like --client and --save error in HelmDeployV1 task flags #21682

Merged
v-abhishera merged 15 commits intomasterfrom
users/v-abhishera/helmdeployV1flagsfix
Jan 14, 2026
Merged

Fixing unknown flags like --client and --save error in HelmDeployV1 task flags #21682
v-abhishera merged 15 commits intomasterfrom
users/v-abhishera/helmdeployV1flagsfix

Conversation

@v-abhishera
Copy link
Copy Markdown
Contributor

@v-abhishera v-abhishera commented Jan 8, 2026

Context Fixing unknown flags like --client and --save error in HelmDeployV1 task flags https://dev.azure.com/mseng/AzureDevOps/_workitems/edit/2343801


Task Name HelmDeployV1


Description

The --client and --save flags are not supported in Helm v3 or later versions, which caused the unknown flags error in the task command that checks the Helm version first to determine if it is v3 or higher.


Risk Assessment Low


Change Behind Feature Flag Yes


Unit Tests Added or Updated Yes


Checklist

@v-abhishera
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

…est setup

- Updated all references in test file and shared test utilities
- Fixed helm version command flag order (--client --short)
- Added mock for getPipelineFeature to control version detection behavior
- Corrected --save flag logic in package command tests
@v-abhishera
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@v-abhishera
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

- Test both true and false scenarios
- Mock legacy (--client --short) and new (--short) commands
@v-abhishera
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@v-abhishera
Copy link
Copy Markdown
Contributor Author

/azp run

@v-abhishera v-abhishera marked this pull request as ready for review January 13, 2026 12:44
@v-abhishera v-abhishera requested review from a team and manolerazvan as code owners January 13, 2026 12:44
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@v-abhishera
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@v-abhishera v-abhishera merged commit 56a6dde into master Jan 14, 2026
14 checks passed
@v-abhishera v-abhishera deleted the users/v-abhishera/helmdeployV1flagsfix branch January 14, 2026 09:01
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.

2 participants