Skip to content

Conversation

@tmshort
Copy link
Contributor

@tmshort tmshort commented May 3, 2024

plug in resolver

Deal with removal of HigherBundleVersion

Removed in e079129

✨ Starting Helm POC - pull in some rukpak (#756)

  • Fix suite_test.go

  • Copy over some rukpak code and replace BundleDeployment

Replace BundleDeployment in the Unpack APIs with a combination of BundleSource and ClusterExtension.

It builds...

  • fixup! Copy over some rukpak code and replace BundleDeployment

Plugin unpacker, add Handler (#757)

A bit of cleanup (#761)

⚠️ Install the helm chart (#762)

  • Install the helm chart

  • fixup! Install the helm chart


Set up right watches and all labels to postrenderer (#763)

⚠️ Update owner keys (#765)

  • Update owner keys

  • fixup! Update owner keys


No more panics (#767)

Add relevant RBAC to enable controller to watch resources (#776)

Changes required for ClusterExtension to install an operator (#789)

  • Add relevant RBAC to enable controller to watch resources

  • Debugging iteration - one

  • ClusterExtension installing the operator - working


Fix some lints (#793)

✨ Getting cluster extension running (#795)

  • Getting cluster extension running

  • Specify namespace to create secret

Set resolved and installed versions (#806)

Remove install references to rukpak (#805)

Consolidate error message generation (#807)

Add make kind-redeploy (#808)

Use rukpak as a library (#821)

Improve caching and fix constant reconciles (#825)

Improve performance by caching objects that only have ClusterExtension as owners.

Fix linter (#826)

Move helm-operator-plugin (#828)

Really fix linter (#833)

Description

Reviewer Checklist

  • API Go Documentation
  • Tests: Unit Tests (and E2E Tests, if appropriate)
  • Comprehensive Commit Messages
  • Links to related GitHub Issue(s)

plug in resolver

Deal with removal of HigherBundleVersion

Removed in e079129

Signed-off-by: Todd Short <[email protected]>

:sparkles: Starting Helm POC - pull in some rukpak (#756)

* Fix suite_test.go

Signed-off-by: Todd Short <[email protected]>

* Copy over some rukpak code and replace BundleDeployment

Replace BundleDeployment in the Unpack APIs with a combination of
BundleSource and ClusterExtension.

It builds...

Signed-off-by: Todd Short <[email protected]>

* fixup! Copy over some rukpak code and replace BundleDeployment

---------

Signed-off-by: Todd Short <[email protected]>

Plugin unpacker, add Handler (#757)

Signed-off-by: Varsha Prasad Narsing <[email protected]>
Co-authored-by: [email protected] <[email protected]>

A bit of cleanup (#761)

Signed-off-by: Todd Short <[email protected]>

:warning: Install the helm chart (#762)

* Install the helm chart

Signed-off-by: Todd Short <[email protected]>

* fixup! Install the helm chart

Signed-off-by: Todd Short <[email protected]>

---------

Signed-off-by: Todd Short <[email protected]>

Set up right watches and all labels to postrenderer (#763)

Co-authored-by: [email protected] <[email protected]>

:warning: Update owner keys (#765)

* Update owner keys

Signed-off-by: Todd Short <[email protected]>

* fixup! Update owner keys

Signed-off-by: Todd Short <[email protected]>

---------

Signed-off-by: Todd Short <[email protected]>

No more panics (#767)

Signed-off-by: Todd Short <[email protected]>

Add relevant RBAC to enable controller to watch resources (#776)

Co-authored-by: [email protected] <[email protected]>

Changes required for ClusterExtension to install an operator  (#789)

* Add relevant RBAC to enable controller to watch resources

* Debugging iteration - one

* ClusterExtension installing the operator - working

---------

Co-authored-by: [email protected] <[email protected]>

Fix some lints (#793)

Signed-off-by: Todd Short <[email protected]>

:sparkles: Getting cluster extension running (#795)

* Getting cluster extension running

* Specify namespace to create secret

Set resolved and installed versions (#806)

Remove install references to rukpak (#805)

Signed-off-by: Todd Short <[email protected]>

Consolidate error message generation (#807)

Signed-off-by: Todd Short <[email protected]>

Add make kind-redeploy (#808)

Signed-off-by: Todd Short <[email protected]>

Use rukpak as a library (#821)

Signed-off-by: Todd Short <[email protected]>

Improve caching and fix constant reconciles (#825)

Improve performance by caching objects that only have ClusterExtension
as owners.

Signed-off-by: Varsha Prasad Narsing <[email protected]>

Fix linter (#826)

Signed-off-by: Varsha Prasad Narsing <[email protected]>

Move helm-operator-plugin (#828)

Signed-off-by: Todd Short <[email protected]>

Really fix linter (#833)

Signed-off-by: Todd Short <[email protected]>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 3, 2024
@netlify
Copy link

netlify bot commented May 3, 2024

Deploy Preview for olmv1 ready!

Name Link
🔨 Latest commit 22b30ca
🔍 Latest deploy log https://app.netlify.com/sites/olmv1/deploys/6635542979400d0008926b3e
😎 Deploy Preview https://deploy-preview-834--olmv1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tmshort tmshort closed this May 6, 2024
@tmshort tmshort deleted the helm-poc-rebase2 branch May 6, 2024 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants