Skip to content
This repository was archived by the owner on Nov 22, 2024. It is now read-only.

Added the Cloudflow CRD.#1074

Merged
RayRoestenburg merged 4 commits into
fix_1072from
crd
Jul 23, 2021
Merged

Added the Cloudflow CRD.#1074
RayRoestenburg merged 4 commits into
fix_1072from
crd

Conversation

@RayRoestenburg

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Adding the Cloudflow CRD including openApiV3 schema.

Why are the changes needed?

The operator will in the future not install the CRD. Installing a CRD requires cluster-admin rights, which the operator should not need.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Tested in a Kubernetes Cluster.

@RayRoestenburg RayRoestenburg changed the title Added the CLoudflow CRD. Added the Cloudflow CRD. Jul 22, 2021
@RayRoestenburg RayRoestenburg requested a review from ignasi35 July 22, 2021 14:57
@RayRoestenburg RayRoestenburg changed the base branch from main to fix_1072 July 23, 2021 11:40
@RayRoestenburg RayRoestenburg merged commit 2059c6c into fix_1072 Jul 23, 2021
@RayRoestenburg RayRoestenburg deleted the crd branch July 23, 2021 11:40
RayRoestenburg added a commit that referenced this pull request Jul 23, 2021
* Cloudflow Custom Resource Definition (CRD).

* Comment to remove endpoint statuses from CRD status, not used.

* Added docs to CRD fields.

* Remove tmp file.
RayRoestenburg added a commit that referenced this pull request Jul 23, 2021
* Added support back in for --volume-mount.

 Added test.

Some refactoring, added check for missing volume mounts.

Better error message.

Cleanup.

Fix for compile.

Added a comment.

* Added the Cloudflow CRD. (#1074)

* Cloudflow Custom Resource Definition (CRD).

* Comment to remove endpoint statuses from CRD status, not used.

* Added docs to CRD fields.

* Remove tmp file.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant