migtools / mig-operator

OpenShift Migration Operator
Apache License 2.0
66 stars 46 forks source link

Fixes typos for Configmap MIG-1318 #873

Closed mvazquezc closed 1 year ago

jmontleon commented 1 year ago

I have no problem with this change but I am not seeing the problem described on 4.12 with 1.7.7. I just did a clean install:

$oc version
Client Version: 4.12.6
Kustomize Version: v4.5.7
Server Version: 4.12.6
Kubernetes Version: v1.25.4+18eadca
$oc get po
NAME                                                READY   STATUS    RESTARTS   AGE
migration-controller-85b6586f8f-m2zft               2/2     Running   0          2m8s
migration-log-reader-f94757c5f-mrrts                2/2     Running   0          2m7s
migration-operator-57b5659c47-gbjvt                 1/1     Running   0          5m47s
migration-ui-84f67d48fb-r279l                       1/1     Running   0          119s
openshift-adp-controller-manager-7cfbb7d949-x5cdh   1/1     Running   0          5m49s
restic-jcg7p                                        1/1     Running   0          2m31s
velero-6bd9b6c86-pgglv                              1/1     Running   0          2m31s
$oc logs --tail=10 migration-operator-57b5659c47-gbjvt
{"level":"info","ts":1678894107.9748218,"logger":"runner","msg":"Ansible-runner exited successfully","job":"5911634533963772251","name":"migration-controller","namespace":"openshift-migration"}

----- Ansible Task Status Event StdOut (migration.openshift.io/v1alpha1, Kind=MigrationController, migration-controller/openshift-migration) -----

PLAY RECAP *********************************************************************
localhost                  : ok=51   changed=0    unreachable=0    failed=0    skipped=22   rescued=0    ignored=0   

----------
$
jmontleon commented 1 year ago

This doesn't reproduce against a 4.12.6 cluster we have available, but it does against a 4.12.2 cluster. I will continue to do some investigation to see if we can figure out where the issue is coming from, but for the sake of affected environments we can merge it now.

jmontleon commented 1 year ago

/lgtm