deis / deis/workflow-manager-api

implement "firstSeen" and "lastSeen" cluster properties

Open
#115 0 comments 0 reactions 1 assignee Claimed by @jackfrancis View on GitHub
refactor
Dominant language
Go
Stars
5
Forks
0
PR merge metrics
No merged PRs in 30d

Description

There are really two issues:
- we're not doing anything useful with the existing "firstSeen" and "lastSeen" properties (the properties are inheriting their zero value, documented here: https://golang.org/pkg/time/#Time)
- we're polluting the `data` properties in the cluster db record

A proposal: get rid of these two properties in the `Cluster` types we're using to submit to the database (or use another type definition); and then add these metadata in a meaningful way using a different type struct.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.