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

Prepare main func for running multiple components #685

Merged

Conversation

szwedm
Copy link
Contributor

@szwedm szwedm commented Apr 29, 2024

Description

BTP Manager will run multiple components in separate goroutines to divide the current module reconciliation and future Service Manager integration.

Related issue(s)

See #442

@kyma-bot kyma-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 29, 2024
@kyma-bot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 29, 2024
@szwedm szwedm force-pushed the sm-integration branch from e078a3e to 3a9db9e Compare May 6, 2024 10:50
@szwedm szwedm force-pushed the prepare-main-func-for-multiple-cmps branch from f013935 to b449e0b Compare May 6, 2024 12:50
@szwedm szwedm force-pushed the prepare-main-func-for-multiple-cmps branch from b449e0b to 03c2cbc Compare May 7, 2024 09:42
@kyma-bot kyma-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 7, 2024
@szwedm szwedm force-pushed the prepare-main-func-for-multiple-cmps branch 2 times, most recently from e875019 to 2d6cf08 Compare May 7, 2024 09:45
@kyma-bot kyma-bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels May 7, 2024
@szwedm szwedm force-pushed the prepare-main-func-for-multiple-cmps branch from 2d6cf08 to 6ff6943 Compare May 7, 2024 09:51
@szwedm szwedm marked this pull request as ready for review May 7, 2024 09:52
@szwedm szwedm requested a review from a team as a code owner May 7, 2024 09:52
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 7, 2024
@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 8, 2024
@szwedm
Copy link
Contributor Author

szwedm commented May 8, 2024

/test pull-btp-manager-build

@szwedm szwedm self-assigned this May 8, 2024
@kyma-bot kyma-bot added the lgtm Looks good to me! label May 9, 2024
@ukff ukff merged commit b8b6e5b into kyma-project:sm-integration May 9, 2024
12 of 14 checks passed
kyma-gopher-bot pushed a commit that referenced this pull request May 9, 2024
* Run components in separate goroutines

* Separate components

* Pass pointers of parsed config values to setupManager

* Add if to print secrets name and namspace when SecretList is not empty

* Check SI CRD existance before fetching secrets

* Add logs for debug
kyma-gopher-bot pushed a commit that referenced this pull request May 15, 2024
* Run components in separate goroutines

* Separate components

* Pass pointers of parsed config values to setupManager

* Add if to print secrets name and namspace when SecretList is not empty

* Check SI CRD existance before fetching secrets

* Add logs for debug
szwedm added a commit that referenced this pull request May 28, 2024
* Run components in separate goroutines

* Separate components

* Pass pointers of parsed config values to setupManager

* Add if to print secrets name and namspace when SecretList is not empty

* Check SI CRD existance before fetching secrets

* Add logs for debug
szwedm added a commit that referenced this pull request Aug 21, 2024
* Run components in separate goroutines

* Separate components

* Pass pointers of parsed config values to setupManager

* Add if to print secrets name and namspace when SecretList is not empty

* Check SI CRD existance before fetching secrets

* Add logs for debug
kyma-gopher-bot pushed a commit that referenced this pull request Aug 22, 2024
* Run components in separate goroutines

* Separate components

* Pass pointers of parsed config values to setupManager

* Add if to print secrets name and namspace when SecretList is not empty

* Check SI CRD existance before fetching secrets

* Add logs for debug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants