Skip to content

Commit

Permalink
Merge pull request #1078 from yaacov/allow-same-namespace-when-plan-t…
Browse files Browse the repository at this point in the history
…o-remote-ocp

🐞 Allow to choose same ns on different provider
  • Loading branch information
yaacov authored Apr 8, 2024
2 parents 414b1bb + 458d8f3 commit 3fd9e37
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,10 @@ export const PlansCreateForm = ({
key={ns?.name || index}
value={ns?.name}
label={ns?.name ?? String(index)}
isDisabled={namespacesUsedBySelectedVms.includes(ns?.name)}
isDisabled={
namespacesUsedBySelectedVms.includes(ns?.name) &&
plan.spec.provider?.destination?.name === plan.spec.provider.source.name
}
/>
)),
]}
Expand Down

0 comments on commit 3fd9e37

Please sign in to comment.