Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-16334

CR.status.conditions and CR.status.LastSyncTimestamp are not updated in some code branches

    XMLWordPrintable

Details

    • No
    • False
    • Hide

      None

      Show
      None

    Description

      Description of problem:

      1. CR.status.LastSyncTimestamp should also be updated in the "else" code branch: 
      https://github.com/openshift/cloud-credential-operator/blob/4cb9faca62c31ebea9a11b55f7af764be4ee2cd8/pkg/operator/credentialsrequest/credentialsrequest_controller.go#L1054
      
      2. r.Client.Status().Update is not called on the CR object in memory after this line:
      https://github.com/openshift/cloud-credential-operator/blob/4cb9faca62c31ebea9a11b55f7af764be4ee2cd8/pkg/operator/credentialsrequest/credentialsrequest_controller.go#L713
      So CR.status.conditions are not updated. 

      Steps to Reproduce:

      This results from a static code check.
      

      Attachments

        Activity

          People

            btofelrh Brett Tofel
            fxierh Feilian Xie
            Feilian Xie Feilian Xie
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: