Skip to content
This repository was archived by the owner on May 18, 2026. It is now read-only.

fix: Adds additional validation for source comparison#311

Merged
shashwathi merged 1 commit into
vmware-tanzu:mainfrom
shaheerkootteeri:iss310
Sep 1, 2022
Merged

fix: Adds additional validation for source comparison#311
shashwathi merged 1 commit into
vmware-tanzu:mainfrom
shaheerkootteeri:iss310

Conversation

@shaheerkootteeri
Copy link
Copy Markdown
Contributor

Pull request

What this PR does / why we need it

PR adds additional validations for comparing sources from current workload to new workload

Which issue(s) this PR fixes

Fixes #310

Describe testing done for PR

Added additional unit test cases
Tested in local cluster

Additional information or special notes for your reviewer

Copy link
Copy Markdown
Contributor

@shashwathi shashwathi left a comment

Choose a reason for hiding this comment

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

Thanks for identifying the bug and fixing it right away
can you add a test at apply_test.go level with source change?

Comment thread pkg/commands/workload_test.go Outdated
@shashwathi shashwathi merged commit 1da5f46 into vmware-tanzu:main Sep 1, 2022
@shaheerkootteeri shaheerkootteeri deleted the iss310 branch September 30, 2022 19:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Changing source on workload is throwing error

3 participants