Skip to content

Conversation

@circy9
Copy link
Contributor

@circy9 circy9 commented Sep 6, 2022

Description of your changes

  • Update MC & IMC API comments
  • Move ResourceUsage to common (used by both MC and IMC)
  • Define and use MemberClusterConditionType instead of string
  • Change json field name from leaseDurationSeconds to heartbeatPeriodSeconds
  • Use mc and imc as short names for MemberCluster and InternalMemberCluster
  • Fix problems in listing MC: Joined column doesn't work
  • Change conditions from required to optional (same as all the k8s builtin resources)

❯ k get membercluster
NAME JOINED AGE
aks-member-10 5d
aks-member-11 4d23h

❯ k get membercluster aks-member-4 -o=custom-columns='NAME:.metadata.name,JOINED:.status.conditions[?(@.type=="ConditionTypeMemberClusterJoin")].status'
NAME JOINED
aks-member-4

❯ k get membercluster aks-member-4 -o=custom-columns='NAME:.metadata.name,JOINED:.status.conditions[?(@.type=="Joined")].status'
NAME JOINED
aks-member-4 True

Fixes #256

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

make e2e-tests

@circy9 circy9 changed the title fea: Update MC & IMC API comments feat: Update MC & IMC API comments Sep 6, 2022
@circy9 circy9 merged commit 45ad25a into main Sep 6, 2022
@zhiying-lin zhiying-lin deleted the liqian/apicomments branch August 16, 2023 08:40
Arvindthiru pushed a commit to Arvindthiru/fleet-public that referenced this pull request Oct 17, 2025
* Added support for new properties

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* Minor fixes

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* Minor fixes

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

---------

Signed-off-by: michaelawyu <chenyu1@microsoft.com>
Co-authored-by: Ryan Zhang <yangzhangrice@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] field name mismatch

4 participants