feat: add problem reporting via resource status

[csatib02]

Adds the problems and problemsCount fields to resource statuses, so users have more direct insight about the specifc problems that can occur.

TODO:

Add additional e2e/integration tests for this feature: #174

[Copilot]

Pull Request Overview

This PR introduces problem reporting fields to various resource statuses to provide better clarity on encountered issues. Key changes include adding getter/setter methods for problems and problem counts to models, updating CRDs and their deep copy functions, and refactoring resource reconciliation error handling.

Reviewed Changes

Copilot reviewed 22 out of 22 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
pkg/sdk/model/types.go	Added interface methods to get and set problems and problem counts.
pkg/resources/manager/tenant_resource_manager.go	Updated tenant resource handling to report problems and refactored subscription output validation.
pkg/resources/manager/bridge_manager.go	Renamed and updated the function to validate bridge connections.
controllers/telemetry/route_controller.go	Refactored reconciliation steps using a tenantReconcileStep pattern and changed status update error handling.
controllers/telemetry/collector_controller.go	Updated collector reconciliation error handling with problem reporting.
config/crd & charts/crds	Added new fields for problem reporting to CRDs.
api/telemetry/v1alpha1/*	Updated API types, deepcopy functions, and associated getters/setters to include problems and problem counts.

Comments suppressed due to low confidence (2)

pkg/resources/manager/tenant_resource_manager.go:201

• The function now returns invalid outputs as []v1alpha1.Output rather than []v1alpha1.NamespacedName. Verify that downstream code handles this updated return type appropriately to avoid any type mismatches.

func (t *TenantResourceManager) ValidateSubscriptionReferencedOutputs(ctx context.Context, subscription *v1alpha1.Subscription) ([]v1alpha1.NamespacedName, []v1alpha1.Output) {

controllers/telemetry/route_controller.go:233

• There is an extraneous return statement with a mismatched return type in the handleOwnedResources diff block. Please remove this leftover code to ensure that only the correct implementation remains.

	return ctrl.Result{}, wrappedErr