operator-controller

operator-controller copied to clipboard

✨ Modify ClusterExtension to use Helm under the hood to apply contents into the cluster

5

# Description ClusterExtension reconciler needs to perform the following actions: [See this [doc](https://docs.google.com/document/d/1sPNHsmsSVVSPzFqMHsksUSLHvGM9zzGpvtG_Q4E04qI/) for more detail if possible.] 1. **Based on the cluster Extension spec, get the list of bundles...

bentito

🌱 Proposed way to unskip some tests

1

# Description TestClusterExtensionChannelVersionExists mostly restored here. This is a proposed way to change the other skipped tests as well. Items skipped by the last conditional are: - Bundle Deployment related...

bentito

:seedling: Bump github.com/operator-framework/operator-registry from 1.40.0 to 1.43.0

1

Bumps [github.com/operator-framework/operator-registry](https://github.com/operator-framework/operator-registry) from 1.40.0 to 1.43.0. Release notes Sourced from github.com/operator-framework/operator-registry's releases. v1.43.0 What's Changed build(deps): bump google.golang.org/grpc from 1.63.2 to 1.64.0 by @​dependabot in operator-framework/operator-registry#1315 build(deps): bump k8s.io/apimachinery from...

dependabot[bot]

:seedling: Bump github.com/operator-framework/catalogd from 0.12.0 to 0.13.0

2

Bumps [github.com/operator-framework/catalogd](https://github.com/operator-framework/catalogd) from 0.12.0 to 0.13.0. Release notes Sourced from github.com/operator-framework/catalogd's releases. v0.13.0 Installation kubectl apply -f https://github.com/cert-manager/cert-manager/releases/download/v1.11.0/cert-manager.yaml kubectl wait --for=condition=Available --namespace=cert-manager deployment/cert-manager-webhook --timeout=60s kubectl apply -f https://github.com/operator-framework/catalogd/releases/download/v0.13.0/catalogd.yaml kubectl wait...

dependabot[bot]

:sparkles: Use error type rather than strings

1

# Description ## Reviewer Checklist - [ ] API Go Documentation - [ ] Tests: Unit Tests (and E2E Tests, if appropriate) - [ ] Comprehensive Commit Messages - [...

tmshort

[epic] ClusterExtension supports simple registry+v1 bundles only

1

### Components - [x] [Brief](https://docs.google.com/document/d/1Bwhv00KtFfB5JQ_11JAN00YnyehzGPIyYb3mdnc9GxM/edit?usp=drive_link) - [x] #779 - [x] #815 - [ ] #816

joelanford

Clarification: Is this a statement or something that needs to be done in the future? ``` // There must be a limit to number of entries if status is stuck...

tmshort

Nit: I personally love using maps with empty value for sets, but I previously received feedback suggesting to use `sets` package (`k8s.io/apimachinery/pkg/util/sets`). _Originally posted by @m1kola in https://github.com/operator-framework/operator-controller/pull/846#discussion_r1604992484_

tmshort

Review Reason and Type strings in ClusterExtension API.

3

We discussed a similar topic [here](https://github.com/operator-framework/operator-controller/pull/846/#discussion_r1601374857): do we really need these reasons? I don't think we have a use case for this just yet. `ReasonErrorGettingClient` can be replaced with a...

tmshort

As discussed in the meeting earlier today, I'd like to see us move to a model where this functions is a bit easier to unit test by creating logical abstractions...

tmshort