kubebuilder icon indicating copy to clipboard operation
kubebuilder copied to clipboard

Document how to represent time and dates in types.

Open erictune opened this issue 7 years ago • 19 comments
trafficstars

KRM convention for storing dates is to settype: string and format: date in the openAPI.

For date-times it is to use type: string and format: date-time.

This is RFC3339, and you can refer this reference: https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.0.md#data-types.

Suggest putting this in the KubeBuilder book. I didn't find the terms date-time or RFC3339 when searching the kubebuilder book.

erictune avatar Nov 12 '18 23:11 erictune

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle stale

fejta-bot avatar Apr 27 '19 00:04 fejta-bot

/remove-lifecycle stale

erictune avatar Apr 29 '19 17:04 erictune

We talk a bit about using metav1.Time, which does this for us. We could do better with documenting it, but users shouldn't have to care about using RFC3339, etc since we have metav1.Time support in CT.

DirectXMan12 avatar May 30 '19 20:05 DirectXMan12

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle stale

fejta-bot avatar Aug 28 '19 21:08 fejta-bot

Stale issues rot after 30d of inactivity. Mark the issue as fresh with /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle rotten

fejta-bot avatar Sep 27 '19 22:09 fejta-bot

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /close

fejta-bot avatar Oct 27 '19 22:10 fejta-bot

@fejta-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Oct 27 '19 22:10 k8s-ci-robot

@erictune: Reopened this issue.

In response to this:

/reopen

On Sun, Oct 27, 2019 at 3:51 PM Kubernetes Prow Robot < [email protected]> wrote:

Closed #476 https://github.com/kubernetes-sigs/kubebuilder/issues/476.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/kubernetes-sigs/kubebuilder/issues/476?email_source=notifications&email_token=AB5245T4YKMSDF6DBZCGMC3QQYLOHA5CNFSM4GDKOTVKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUO5R4CA#event-2746949128, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB5245TO5S7PJEZ4GZAKKTLQQYLOHANCNFSM4GDKOTVA .

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Oct 27 '19 23:10 k8s-ci-robot

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /close

fejta-bot avatar Nov 26 '19 23:11 fejta-bot

@fejta-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Nov 26 '19 23:11 k8s-ci-robot

/lifecycle frozen /help

DirectXMan12 avatar Dec 05 '19 22:12 DirectXMan12

@DirectXMan12: This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-help command.

In response to this:

/lifecycle frozen /help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Dec 05 '19 22:12 k8s-ci-robot

This shows another one nice to the FAQ section such as :

  • https://github.com/kubernetes-sigs/kubebuilder/issues/1723

camilamacedo86 avatar Mar 29 '21 16:03 camilamacedo86

hello, why this issue is still open.

Sajiyah-Salat avatar Jan 20 '23 14:01 Sajiyah-Salat

Hi @Sajiyah-Salat,

Did you find this info documented already? If so, could you please add here the link for it?

If not, we need to address it via documentation. The FAQ section shows a good place to do so.

camilamacedo86 avatar Feb 11 '23 15:02 camilamacedo86

According to @DirectXMan12 we have metav1.Time we just need to document it. Can I get some resources from which I can get idea of what and how to add? I think this is a document issue. Thank you.

Sajiyah-Salat avatar Feb 12 '23 01:02 Sajiyah-Salat

@camilamacedo86 @varshaprasad96 DO we still need to fix this or it can be closed now?

ashutosh887 avatar Aug 17 '23 07:08 ashutosh887

Hello @camilamacedo86 I didnt find this info documented though. I would like to take this up? https://github.com/OAI/OpenAPI-Specification/blob/main/versions/3.1.0.md#data-types Should we add the whole datatype block in documentation or we should extract data on our own and tweak it. what say?

Sajiyah-Salat avatar Aug 18 '23 02:08 Sajiyah-Salat