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

Document G8sControlPlane CRD#405

Merged
marians merged 5 commits intomasterfrom
document-g8scp
Apr 15, 2020
Merged

Document G8sControlPlane CRD#405
marians merged 5 commits intomasterfrom
document-g8scp

Conversation

@marians
Copy link
Member

@marians marians commented Apr 9, 2020

Towards https://github.com/giantswarm/giantswarm/issues/8804

  • Adds YAML CRD and CR files generated by unit tests
  • Adds property descriptions to the schema

@marians marians self-assigned this Apr 9, 2020
@marians marians requested review from anvddriesch and xh3b4sd April 9, 2020 20:08
Copy link
Contributor

@xh3b4sd xh3b4sd 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 sorting that out. You do you.

infrastructureRef:
description: |
Reference to an [AWSControlPlane](https://docs.giantswarm.io/reference/cp-k8s-api/awscontrolplanes.infrastructure.giantswarm.io/)
resource defining provider specific details for the c
Copy link
Contributor

Choose a reason for hiding this comment

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

For the what? There got probably something lost.

Copy link
Contributor

@anvddriesch anvddriesch left a comment

Choose a reason for hiding this comment

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

Looks good to me other than what Tim already commented on :)

@marians marians merged commit 80cb457 into master Apr 15, 2020
@marians marians deleted the document-g8scp branch April 15, 2020 09:34
teemow added a commit that referenced this pull request Apr 16, 2020
* master:
  use architect-orb v0.8.8 (#408)
  Replace DeepCopyTime with metav1.Time (#407)
  Add missing releases to changelog (#411)
  Document G8sControlPlane CRD (#405)
  Add Chart CR (#406)
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.

3 participants