Conversation
🔀 Merge develop into master (Release)
K8s doesn't allow any `_` in object names. However `_` is allowed for user names in database systems. These differences have to be escaped when applying the objects to the API. However the newly introduced `IsResourceRead()` function did not account for that. Which resulted in resources not being found. This in turn caused Crossplane to remove the supposedly non-existing resources from the API. Additionally this commit will set all the objects to be applied with the policy to only create and update the data. Never to delete it. Deletion will have to be a manual and deliberate operation from now on. This also ensures feature parity with the PostgreSQL implementation.
Make sure names are escaped during ready check
This sets the necessary kubebuilder flags in the service CRDs to allow setting the number of instances to 0.
If the vshnmariadb instance is scaled to 0, we also return early and dont create proxysql.
This allows suspending an instance even if the service level is set to guaranteed availability. Since we might need to suspend an instance for a customer, this has to be possible.
The replicaCount from the claim was never passed to the actual helm values. This fixes that.
This disables the cronjob for vshnnextcloud if an instance has 0 replicas.
This passes the instances field to the replicaCount in the helm values for forgejo.
This ensures that collabora is also suspended alongside the main nextcloud pod if instances = 0
Ensure that vshnforgejo can't be scaled beyond 1 replica for now since we don't support HA yet.
CNPG can't handle 0 replicas for a cluster. It instead has a hibernation feature. The hibernation procedure will delete the primary Pod and then the replica Pods, avoiding switchover, to ensure the replicas are kept in sync.
This checks if an instance is suspended an stops the SLI exporter from probing those instances prevent false alerts.
This removes the backup schedules and suspends database operations like repack and vacuum for suspended instances.
Allow instances to be downscaled to 0
The nextcloud helmchart 8.6.0 uses seperate registry and repository configurations, which doesn't work with the previous implementation.
Update Nextcloud Helmchart to 8.6.0
This adds the necessary storage class annotations for APPUiO on Exoscale.
Add Exoscale specific annotations for quotas
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Auto-generated PR to release from develop to master.