Skip to content

snapshot-controller - #237

Merged
Andrei Kvapil (kvaps) merged 4 commits into
cozystack:mainfrom
mrkhachaturov:dev-snapshot-controller
Jul 26, 2024
Merged

Andrei Kvapil (kvaps) merged 4 commits into
cozystack:mainfrom
mrkhachaturov:dev-snapshot-controller

Conversation

@mrkhachaturov

Copy link
Copy Markdown
Contributor

Added snapshot-controller to system packages.

It is included in pass-full bundle.
Also added new cluster issuer selfsigned-cluster-issuer.

Comment thread packages/core/platform/bundles/paas-full.yaml Outdated
privileged: true
dependsOn: [piraeus-operator,cilium,kubeovn,cert-manager]

- name: snapshot-controller

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's also added it to distro-full bundle

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove from paas-full and add only to distro-full?

@mrkhachaturov Mr. Khachaturov (mrkhachaturov) Jul 26, 2024 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didnt noticed also ))

@kvaps Andrei Kvapil (kvaps) Jul 26, 2024 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, we need it in both distro-full and paas-full bundles as they ship storage

@kvaps Andrei Kvapil (kvaps) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, many thanks for your contribution!

@kvaps
Andrei Kvapil (kvaps) merged commit a17c622 into cozystack:main Jul 26, 2024
@mrkhachaturov
Mr. Khachaturov (mrkhachaturov) deleted the dev-snapshot-controller branch August 29, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants