Feature/121.3 remove components #128
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…1-apply-blueprint-continuously
…pply-blueprint-continuously # Conflicts: # pkg/application/blueprintSpecChangeUseCase.go # pkg/bootstrap.go
this way, whe use cases don't need to load the blueprint everytime. If there is an concurrent update, we handle the conflicts when we try to update the blueprint ourselves. If we use the same blueprint the whole time we don't need to recalculate the state diff. We only have to do this, if we need to reconcile the CR again, e.g. for non-blocking health checks.
Otherwise, we would publish dozens of this events because we will validate the blueprint as every reconciliation now
We have a condition instead. We also want to prevent event duplication over multiple reconciles.
We show the effective blueprint in status. There could be thousands of this event if we recalculate the effective blueprint on every reconciliation.
add condition for healthy ecosystem
We removed the maintenance mode. Therefore, the only thing this use case still did, was to check the dry run flag.
In previous versions, the state diff was calculated once and then got persisted in the blueprint CR. Therefore, we had to write all fields there. Now we calculate the state diff at every reconciliation. Because of that we can now decide which fields we want to publish in the blueprint CR.
If the ecosytem is unhealthy, an error is thrown and another reconciliation gets triggered. This way, we repeat the health check until everything is healthy. If the ecosystem is healthy afterward, the post-processing will always run.
We need this later to skip some steps while applying the blueprint.
We now throw an error to trigger a reconciliation. It is also now safe to just retry. We can now see the exact state of the self-upgrade via the new generated state diff and the component CR install versions.
…ntinuously' into feature/121.3-remove-components # Conflicts: # pkg/adapter/kubernetes/blueprintcr/v2/blueprintSpecCRRepository_test.go # pkg/domain/blueprintSpec.go # pkg/domain/blueprintSpec_test.go # pkg/domain/ecosystem/ecosystemHealth.go # pkg/domain/ecosystem/ecosystemHealth_test.go # pkg/domain/events.go # pkg/domain/events_test.go
…ntinuously' into feature/121.3-remove-components # Conflicts: # pkg/application/blueprintSpecChangeUseCase_test.go # pkg/domain/events_test.go
…e/121.3-remove-components
…ntinuously' into feature/121.3-remove-components
…ntinuously' into feature/121.3-remove-components
…ntinuously' into feature/121.3-remove-components
…ntinuously' into feature/121.3-remove-components # Conflicts: # pkg/domain/events.go
…ntinuously' into feature/121.3-remove-components
…ntinuously' into feature/121.3-remove-components # Conflicts: # pkg/application/blueprintSpecChangeUseCase_test.go
…ntinuously' into feature/121.3-remove-components
…ntinuously' into feature/121.3-remove-components
…ntinuously' into feature/121.3-remove-components
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.