up/profile: check for Spaces on 'up profile set space' - Githubissues

upbound / up

The @upbound CLI

Apache License 2.0

51 stars 41 forks source link

up/profile: check for Spaces on 'up profile set space' #406

Closed sttts closed 9 months ago

sttts commented 9 months ago

Description of your changes

We check that there is a upbound-system/mxe-controller deployment.

Based on https://github.com/upbound/up/pull/405

Follow-up of https://github.com/upbound/up/pull/400.

I have:

[x] Read and followed Upbound's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] ~~Added backport release-x.y labels to auto-backport this PR, as appropriate.~~

How has this code been tested

manually running up profile set space against a Space and against a non-Space kube cluster
added unit tests for happy case and two unhappy cases.