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

Remove Compass integration code from the Provisioner #3088

Closed
1 of 3 tasks
akgalwas opened this issue Sep 27, 2023 · 6 comments · Fixed by #3399
Closed
1 of 3 tasks

Remove Compass integration code from the Provisioner #3088

akgalwas opened this issue Sep 27, 2023 · 6 comments · Fixed by #3399
Assignees
Labels
2023-Q4 Planned for Q4 2023

Comments

@akgalwas
Copy link
Contributor

akgalwas commented Sep 27, 2023

Description

Provisioner should no longer register runtime in the Compass Director, and create the secret for Compass Runtime Agent.

Reasons

Compass Manager will take over Compass integration responsibility. This issue is the last step of transition process.

Acceptance criteria

Proposal (by @koala7659 )
We can deliver this feature controlled with a configuration flag to selectively disable old functionality on specific environment (DEV, STAGE, PROD).

@akgalwas akgalwas self-assigned this Sep 27, 2023
@akgalwas akgalwas added the 2023-Q4 Planned for Q4 2023 label Sep 27, 2023
@akgalwas akgalwas changed the title Remove Compass integration code from the Provisioner Compass Manager - Remove Compass integration code from the Provisioner Oct 10, 2023
@kyma-bot
Copy link
Contributor

kyma-bot commented Dec 9, 2023

This issue or PR has been automatically marked as stale due to the lack of recent activity.
Thank you for your contributions.

This bot triages issues and PRs according to the following rules:

  • After 60d of inactivity, lifecycle/stale is applied
  • After 7d of inactivity since lifecycle/stale was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Close this issue or PR with /close

If you think that I work incorrectly, kindly raise an issue with the problem.

/lifecycle stale

@kyma-bot kyma-bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 9, 2023
@tobiscr tobiscr removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 13, 2023
@tobiscr tobiscr changed the title Compass Manager - Remove Compass integration code from the Provisioner Remove Compass integration code from the Provisioner Dec 28, 2023
@tobiscr
Copy link
Contributor

tobiscr commented Dec 28, 2023

@akgalwas : has this to be done before we migrate the provisioner logic to the Infrastructure Manager or can we handle it together as part of kyma-project/infrastructure-manager#112 ?

@koala7659
Copy link
Contributor

Some remarks for people who think about architecture of new control plane with compass-manager, Compass integration, and KIM (Kyma Infrastructure Manager)
I have just discovered some old code in Provisioner that marks current provisioning status of the Runtime in Director using RuntimeStatusCondition where possible states can be

const (
    RuntimeStatusConditionInitial      RuntimeStatusCondition = "INITIAL"
    RuntimeStatusConditionProvisioning RuntimeStatusCondition = "PROVISIONING"
    RuntimeStatusConditionConnected    RuntimeStatusCondition = "CONNECTED"
    RuntimeStatusConditionFailed       RuntimeStatusCondition = "FAILED"
)

We set such status during some operations with UpdateRuntime mutation.
As planned I'm disabling now (and in future will remove) all code that talks to Director in Provisioner.
Since we integration with Compass optional - only when user enables Application-connector module setting such status will not have any sense anymore.
This code cannot be migrated to compass-manager
I hope this will not make any braking change for the Compass Point of View.

@koala7659
Copy link
Contributor

PR with implementation: #3399

@koala7659
Copy link
Contributor

Test deployment on mps-dev: kcp-pg-provisioner

@koala7659
Copy link
Contributor

@akgalwas Should we stop from now on returning CompassRuntimeID ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2023-Q4 Planned for Q4 2023
Projects
None yet
4 participants