Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Combine work-request errors into single reconcileError event. #385

Merged
merged 4 commits into from
Sep 19, 2024

Conversation

sindhusri16
Copy link
Member

@sindhusri16 sindhusri16 commented Sep 17, 2024

Combine work-request errors into single reconcileError event so that processing of this single recent event would provide information about the status of the machine. This is only for ocimachine events and not nlb/lb

Related Fixes #383

@sindhusri16 sindhusri16 requested a review from joekr September 17, 2024 19:58
@sindhusri16 sindhusri16 self-assigned this Sep 17, 2024
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Sep 17, 2024
@sindhusri16
Copy link
Member Author

The change is only the second commit, the previous commit are already merged into main branch. The output of unittests:

make test
fatal: No names found, cannot describe anything.
GOBIN=/home/opc/cluster-api-provider-oci/hack/tools/bin ./scripts/go_install.sh sigs.k8s.io/controller-runtime/tools/setup-envtest setup-envtest v0.0.0-20230131074648-f5014c077fc3
kube-builder assets: /home/opc/.local/share/kubebuilder-envtest/k8s/1.24.2-linux-amd64
KUBEBUILDER_ASSETS="/home/opc/.local/share/kubebuilder-envtest/k8s/1.24.2-linux-amd64" go test -coverprofile=coverage.out ./...
? github.com/oracle/cluster-api-provider-oci [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/config [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/metrics [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/scope/mocks [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/base [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/base/mock_base [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/compute [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/compute/mock_compute [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement/mock_computemanagement [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine/mock_containerengine [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/identity [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/identity/mock_identity [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer/mock_lb [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer/mock_nlb [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/vcn [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/vcn/mock_vcn [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/workrequests [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/workrequests/mock_workrequests [no test files]
? github.com/oracle/cluster-api-provider-oci/feature [no test files]
? github.com/oracle/cluster-api-provider-oci/version [no test files]
ok github.com/oracle/cluster-api-provider-oci/api/v1beta1 92.555s coverage: 23.4% of statements
ok github.com/oracle/cluster-api-provider-oci/api/v1beta2 0.053s coverage: 14.6% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/ociutil 0.025s coverage: 15.9% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/scope 22.261s coverage: 76.1% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/util 0.748s coverage: 61.1% of statements
ok github.com/oracle/cluster-api-provider-oci/controllers 2.559s coverage: 59.2% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/api/v1beta1 32.768s coverage: 15.3% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/api/v1beta2 0.065s coverage: 4.8% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/controllers 0.996s coverage: 56.3% of statements
go tool cover -func=coverage.out -o coverage.txt
go tool cover -html=coverage.out -o coverage.html

@joekr
Copy link
Member

joekr commented Sep 18, 2024

e2e tests

Ran 7 of 28 Specs in 3084.062 seconds
SUCCESS! -- 7 Passed | 0 Failed | 0 Pending | 21 Skipped

@sindhusri16 sindhusri16 merged commit a5dc517 into oracle:main Sep 19, 2024
2 checks passed
joekr pushed a commit that referenced this pull request Sep 19, 2024
* changed wrClient to nlb and lb client to fetch core work-requests.

* combine error events into single message

* concatention changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LoadBalancer/NLB work-request-errors could be included as part of events for better error information.
2 participants