Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

Remove description for CRDs #887

Merged
merged 3 commits into from
Aug 17, 2018
Merged

Conversation

andresmgot
Copy link
Contributor

Issue Ref: Fixes #881

Description:

In K8s 1.11 the Description field is not available.

TODOs:

  • Ready to review
  • Automated Tests
  • Docs

Copy link
Contributor

@arapulido arapulido left a comment

Choose a reason for hiding this comment

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

The kafka-zookeeper.jsonnet also defines a CRD with description

@arapulido
Copy link
Contributor

@andresmgot it seems that description is not removed, but has been moved to a new field CustomResourceColumnDefinition. We should improve this later on

@andresmgot
Copy link
Contributor Author

andresmgot commented Aug 17, 2018

Good catch. Thanks @arapulido. Honestly the description we are setting doesn't give any useful information.

Copy link
Contributor

@arapulido arapulido left a comment

Choose a reason for hiding this comment

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

lgtm. We should improve the CRDs definitions once 1.12 gets released

@andresmgot andresmgot merged commit 89b32fd into vmware-archive:master Aug 17, 2018
andresmgot pushed a commit that referenced this pull request Aug 17, 2018
* adding readme

* Support setting env variable for bearer token

* Revert "adding readme"

This reverts commit 6ed2ef2.

* formatting

* Delete glide.yaml

Now Gopkg.toml is used instead

* Remove description for CRDs (#887)

* Remove description for CRDs

* Update docs

* Adapt kafka manifest

* Renaming GetOverriddenClientConfig to GetInClusterConfig, removing redundant error handler for getting config

* adding readme

* Support setting env variable for bearer token

* Revert "adding readme"

This reverts commit 6ed2ef2.

* formatting

* Renaming GetOverriddenClientConfig to GetInClusterConfig, removing redundant error handler for getting config

* Adding docs for KUBELESS_TOKEN_FILE_PATH

* Fix format
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.

2 participants