Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

make ClusterFilter and StorageClass toggable in helm-chart #830

Merged

Conversation

nschad
Copy link
Contributor

@nschad nschad commented Aug 6, 2024

How to categorize this PR?
/area control-plane
/kind enhancement
/platform openstack

What this PR does / why we need it:

Allows the operator to configure whether he wants to deploy the ClusterFilters or StorageClass by default. This is required for us as we currently still have 2 Extensions registered on the same type. Therefore we need to be able to configure in one extension, that certain things don't need to be deployed.

To not change anything the default is implemented as always deploy just like before.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

Allows the operator to configure whether he wants to deploy the standard ClusterFilters and/or StorageClass

@nschad nschad requested review from a team as code owners August 6, 2024 07:49
@gardener-robot gardener-robot added needs/review Needs review area/control-plane Control plane related kind/enhancement Enhancement, improvement, extension platform/openstack OpenStack platform/infrastructure labels Aug 6, 2024
@gardener-robot
Copy link

@nschad Thank you for your contribution.

@gardener-robot-ci-2
Copy link
Contributor

Thank you @nschad for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.

@gardener-robot gardener-robot added the size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) label Aug 6, 2024
@CLAassistant
Copy link

CLAassistant commented Sep 16, 2024

CLA assistant check
All committers have signed the CLA.

@nschad nschad force-pushed the helm-chart-toggable-resources branch from 7453cf1 to a4ab011 Compare September 17, 2024 07:41
kon-angelo
kon-angelo previously approved these changes Sep 30, 2024
Copy link
Contributor

@kon-angelo kon-angelo left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/review Needs review labels Sep 30, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 30, 2024
@kon-angelo
Copy link
Contributor

/reviewed ok-to-test

@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Sep 30, 2024
@kon-angelo
Copy link
Contributor

@nschad can you run make generate ?

@nschad
Copy link
Contributor Author

nschad commented Sep 30, 2024

@nschad can you run make generate ?

Done.

@gardener-robot gardener-robot added the needs/rebase Needs git rebase label Sep 30, 2024
@gardener-robot
Copy link

@nschad You need rebase this pull request with latest master branch. Please check.

@gardener-robot gardener-robot removed needs/review Needs review reviewed/lgtm Has approval for merging labels Sep 30, 2024
Signed-off-by: Niclas Schad <niclas.schad@stackit.cloud>
@nschad nschad force-pushed the helm-chart-toggable-resources branch from d070d63 to 40cc790 Compare September 30, 2024 11:26
Signed-off-by: Niclas Schad <niclas.schad@stackit.cloud>
@kon-angelo
Copy link
Contributor

/reviewed ok-to-test

@gardener-robot gardener-robot added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 30, 2024
@kon-angelo kon-angelo removed the needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 30, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Sep 30, 2024
Copy link
Contributor

@kon-angelo kon-angelo left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/rebase Needs git rebase labels Sep 30, 2024
@kon-angelo kon-angelo merged commit 49cb34d into gardener:master Sep 30, 2024
12 checks passed
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Sep 30, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 30, 2024
@nschad nschad deleted the helm-chart-toggable-resources branch September 30, 2024 14:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/control-plane Control plane related kind/enhancement Enhancement, improvement, extension needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) platform/openstack OpenStack platform/infrastructure reviewed/lgtm Has approval for merging reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) status/closed Issue is closed (either delivered or triaged)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants