Bug 1796599 - machine instance-state annotation discrepancy with providerStatus.instanceState
Summary: machine instance-state annotation discrepancy with providerStatus.instanceState
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Cloud Compute
Version: 4.4
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 4.4.0
Assignee: Alberto
QA Contact: Jianwei Hou
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-01-30 18:16 UTC by Alberto
Modified: 2020-05-15 15:57 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-05-15 15:57:01 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift cluster-api-provider-aws pull 288 0 None closed Bug 1796599: Save status before Patch() 2021-02-19 14:26:47 UTC
Github openshift cluster-api-provider-azure pull 103 0 None closed Bug 1796599: Save status before Patch() 2021-02-19 14:26:47 UTC

Description Alberto 2020-01-30 18:16:56 UTC
Description of problem:
The instance-state annotation and the providerStatus.instanceState occasionally showing different values [1]. This should never happen as they consume from the same source. This is likely to be an issue on the client failing to patch the status or patching it wrongly. 

[1] https://storage.googleapis.com/origin-ci-test/logs/release-openshift-ocp-installer-e2e-aws-4.4/866/artifacts/e2e-aws/machines.json

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 4 Jianwei Hou 2020-02-06 08:45:25 UTC
Verified in 4.4.0-0.nightly-2020-02-05-181112. Added regression test for this scenario, this will be covered in future tests.

oc get machines -n openshift-machine-api -o jsonpath='{range .items[*]}{.metadata.annotations}{"\t"}{.status.providerStatus.instanceState}{"\n"}{end}'
map[machine.openshift.io/instance-state:running]	running
map[machine.openshift.io/instance-state:running]	running
map[machine.openshift.io/instance-state:running]	running
map[machine.openshift.io/instance-state:running]	running
map[machine.openshift.io/instance-state:running]	running
map[machine.openshift.io/instance-state:running]	running
map[machine.openshift.io/instance-state:running]	running


Note You need to log in before you can comment on or make changes to this bug.