-
Notifications
You must be signed in to change notification settings - Fork 69
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
[release-2.12] Add support for mcoa-managed observability capabilities #1470
Conversation
Hi @periklis. Thanks for your PR. I'm waiting for a stolostron member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good! I'm just concerned with how we will package the CRDs since without them users will not be able to create stanzas unless they install the operators
operators/multiclusterobservability/api/v1beta2/multiclusterobservability_types.go
Show resolved
Hide resolved
.../multiclusterobservability/manifests/base/multicluster-observability-addon/cluster_role.yaml
Outdated
Show resolved
Hide resolved
operators/multiclusterobservability/pkg/rendering/templates/templates.go
Outdated
Show resolved
Hide resolved
operators/multiclusterobservability/api/v1beta2/multiclusterobservability_types.go
Outdated
Show resolved
Hide resolved
operators/multiclusterobservability/pkg/rendering/templates/templates.go
Outdated
Show resolved
Hide resolved
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
…r-observability-addon/deployment.yaml Co-authored-by: Joao Marcal <[email protected]> Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Signed-off-by: Periklis Tsirakidis <[email protected]>
Quality Gate passedIssues Measures |
/test test-e2e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: periklis, philipgough The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
ClusterManagementAddOn In stolostron#1470 we added support for deploying and configuring MCOA with the capabilities field. However if we enabled a field in capabilities and then disable it MCO would not reconcile the AddOnDeploymentConfig correctly due to DeepDerivative, that would return true instead of false.
ClusterManagementAddOn In stolostron#1470 we added support for deploying and configuring MCOA with the capabilities field. However if we enabled a field in capabilities and then disable it MCO would not reconcile the AddOnDeploymentConfig correctly due to DeepDerivative, that would return true instead of false. Signed-off-by: Joao Marcal <[email protected]>
ClusterManagementAddOn In stolostron#1470 we added support for deploying and configuring MCOA with the capabilities field. However if we enabled a field in capabilities and then disable it MCO would not reconcile the AddOnDeploymentConfig correctly due to DeepDerivative, that would return true instead of false. Signed-off-by: Joao Marcal <[email protected]>
ClusterManagementAddOn In #1470 we added support for deploying and configuring MCOA with the capabilities field. However if we enabled a field in capabilities and then disable it MCO would not reconcile the AddOnDeploymentConfig correctly due to DeepDerivative, that would return true instead of false. Signed-off-by: Joao Marcal <[email protected]>
The following PR adds support to reconcile further observability capabilities via the multicluster-observability-addon. The present implementation adds:
MultiClusterObservability
CRD spec by a new section namedcapabilities
as discussed here (TODO: Add link to DDR here).Special notes for your reviewer:
To the test the following PR one needs to publish an image of the referenced dependency PR and applying the following instructions:
Advanced Cluster Management for Kubernetes
from the OperatorHub.MultiClusterHub
resource.MultiClusterHub
resource to pause reconciliation via:kubectl -n open-cluster-management annotate mch multiclusterhub --overwrite mch-pause=true
kubectl -n open-cluster-management edit deployment multicluster-observability-operator
create/update/delete/patch
forAddonDeploymentConfig
resources:kubectl edit clusterrole open-cluster-management:multicluster-observability-operator:multicluster-observability-operator
capabilities
section.MultiClusterObservability
custom resource as the following:Dependencies: