Skip to content
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

Infrastructure Manager creates secrets when a new Gardener Cluster CR is created. #37

Merged
merged 11 commits into from
Sep 22, 2023

Conversation

Disper
Copy link
Member

@Disper Disper commented Sep 15, 2023

Description

Changes proposed in this pull request:

  • Adds test for secret creation
  • ...
  • ...

/area control-plane
/kind feature

Related issue(s)
#10

@Disper Disper requested a review from a team as a code owner September 15, 2023 14:52
@kyma-bot kyma-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. area/control-plane Related to all activities around Kyma Control Plane kind/feature Categorizes issue or PR as related to a new feature. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Sep 15, 2023
@Disper Disper changed the base branch from kubeconfig_secret_creation to main September 18, 2023 14:08
@kyma-bot kyma-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Sep 18, 2023
@Disper
Copy link
Member Author

Disper commented Sep 18, 2023

By mistake this PR was not created to the main branch and Github Action that runs tests was not triggered.

@Disper Disper removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 18, 2023
@Disper Disper changed the title [WiP] Infrastructure Manager creates secrets when a new Cluster Inventory CR is created. Infrastructure Manager creates secrets when a new Cluster Inventory CR is created. Sep 19, 2023
@akgalwas akgalwas changed the title Infrastructure Manager creates secrets when a new Cluster Inventory CR is created. Infrastructure Manager creates secrets when a new Gardener Cluster CR is created. Sep 20, 2023
Copy link

@VOID404 VOID404 left a comment

Choose a reason for hiding this comment

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

Other than that, it seems fine

Dockerfile Outdated
COPY cmd/main.go cmd/main.go
COPY api/ api/
COPY internal/controller/ internal/controller/
COPY . ./
Copy link

Choose a reason for hiding this comment

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

Is everything in this folder really needed? Folders like docs aren't excluded here nor in the .dockerignore

Copy link
Member Author

Choose a reason for hiding this comment

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

improved in ab86a67

Copy link
Contributor

@akgalwas akgalwas left a comment

Choose a reason for hiding this comment

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

The PR looks good, but let's fix the requeuing issues.

akgalwas
akgalwas previously approved these changes Sep 22, 2023
@Disper Disper dismissed akgalwas’s stale review September 22, 2023 06:21

The merge-base changed after approval.

@kyma-bot kyma-bot added the lgtm Looks good to me! label Sep 22, 2023
akgalwas
akgalwas previously approved these changes Sep 22, 2023
@Disper Disper dismissed akgalwas’s stale review September 22, 2023 06:27

The merge-base changed after approval.

@kyma-bot kyma-bot removed the lgtm Looks good to me! label Sep 22, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Sep 22, 2023
@kyma-bot kyma-bot merged commit 09aad9b into kyma-project:main Sep 22, 2023
5 checks passed
@Disper Disper deleted the kubeconfig_secret_creation branch September 22, 2023 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/control-plane Related to all activities around Kyma Control Plane kind/feature Categorizes issue or PR as related to a new feature. lgtm Looks good to me! size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants