Skip to content

Helm PoC latest rebase - DO NOT MERGE #834

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

Closed
wants to merge 1 commit into from
Closed

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
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.

1 participant