Skip to content

Streamline CRD and RBAC Generation to avoid futher Issues#2348

Merged
J12934 merged 9 commits intosecureCodeBox:mainfrom
J12934:streamline-crd-and-rbac-config
Mar 12, 2024
Merged

Streamline CRD and RBAC Generation to avoid futher Issues#2348
J12934 merged 9 commits intosecureCodeBox:mainfrom
J12934:streamline-crd-and-rbac-config

Conversation

@J12934
Copy link
Copy Markdown
Member

@J12934 J12934 commented Mar 8, 2024

Description

This PR changes the controller-manager config to directly generate the CRDs and RBAC rules in the dirs used by the helm chart rather than the (mostly) unused config folders. These were autogenerated by kubebuilder but we stopped using these quite a while back. So I completly removed them to remove some hassle.

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

J12934 added 6 commits March 8, 2024 15:04
… output them into the correct dirs

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
…icense headers

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
@J12934 J12934 requested a review from Ilyesbdlala March 8, 2024 14:54
@J12934 J12934 self-assigned this Mar 8, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 8, 2024

Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit 7b9aad2
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/65eb290c56c2b600084ed16f

J12934 added 3 commits March 8, 2024 16:03
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.2% Duplication on New Code

See analysis details on SonarCloud

@Ilyesbdlala
Copy link
Copy Markdown
Member

Ilyesbdlala commented Mar 12, 2024

Nice that we finally resolve this ^_^
Is the addition of the nodeSelector to the parsedefinition just something that was forgotten to be copied over to the crd/ folder in another PR?

@J12934
Copy link
Copy Markdown
Member Author

J12934 commented Mar 12, 2024

Nice that we finally resolve this ^_^

Is the addition of the nodeSelector to the parsedefinition just something that was forgotten to be copied over to the crd/ folder in another PR?

Not sure if it was forgotten or later overwritten by something else. But it definitely should have been there 😅

@J12934 J12934 merged commit 3fd4a6a into secureCodeBox:main Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants