Skip to content

Object storage v2 beta controller#2

Draft
visdmin wants to merge 8 commits intomainfrom
object-storage-v2-beta-controller
Draft

Object storage v2 beta controller#2
visdmin wants to merge 8 commits intomainfrom
object-storage-v2-beta-controller

Conversation

@visdmin
Copy link
Owner

@visdmin visdmin commented Oct 31, 2023 •
edited
Loading

Entities and controller to be added:

  • V1Alpha1ObjectStorage2
  • V1Alpha1ObjectStorage2AccessKey

Controller features (V1Alpha1ObjectStorage2):

  • Add entity and empty controller
  • Show existing os2 instance and keep entity status up to date
  • Create new os2 instance
  • Update existing os2 istance according on entity Spec when state does not match
  • Delete os2 instance

Controller features (V1Alpha1ObjectStorage2AccessKey):

  • Add entity and empty controller
  • Create new access keys and store access key secret to k8s secret entity
  • List exising access keys
  • Update exising access according to entity Spec when state does not match (mainly AccessKey.Enabled)
  • Delete access keys

Add UpCloudApiClient project and required methods:

  • GET object storage 2 details
  • POST create object storage 2 instance
  • DELETE delete object storage 2 instance
  • PUT replace object storage 2 instance
  • POST create access key
  • GET get access key details
  • DELETE delete access key
  • PUT update access key

WIP !!!

visdmin and others added 7 commits October 29, 2023 01:46
@visdmin
ObjectStoragev2 controller idea WIP!!!
5b0809e
@visdmin
Adding idea of UpC API client.
2cc9f6d
@visdmin
Adding ideas of how to use the WIP UpC api client into object storage…
124ecc5 
… v2 entity controller.
@visdmin
Adding idea of autoloader that allows the controller to automatically…
a65d116 
… load existing entities from the api.

This maybe very bad idea.
Feature is nice to have, but the implementation of such behaviour should be maybe isolated to separate K8s job ?
Lets keep it here for now, and think about better solution in the future.
Author suspects that it's not the purpose of the operator to do such a task, much better way could to be define CRD `LoadUpCloudEntitiesJob` or something similar, that would be handle the loading of the entities from the UpC api.

- UpCloud Api client tested locally little here before these changes.
- Creation of new entities seems to work fine with auto loader.

```shell
$ kubectl -n default get objectstorage2s.api.upcloud.com
NAME                                                     RECONCILESTATE   ID                                     CONFIGUREDSTATUS   OPERATIONALSTATE   REGION     UPDATEDAT
object-storage-v2-12c5cbc2-3013-491f-a6f4-eeb38edcda4c   UP_TO_DATE       12c5cbc2-3013-491f-a6f4-eeb38edcda4c   started            running            europe-1   2023-10-30T21:24:00
```
@visdmin
Update README.md
ba305cf
@visdmin
Minor fixes and idea about access key creation api endpoint usage.
902c7b8
@visdmin
Merge branch 'object-storage-v2-beta-controller' of https://github.co…
a3c1bf1 
…m/visdmin/upcloud-api-kubernetes-operator into object-storage-v2-beta-controller
@visdmin visdmin added the feature New feature or request label Oct 31, 2023
@visdmin visdmin added this to the release-0.1.0 milestone Oct 31, 2023
@visdmin visdmin self-assigned this Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@visdmin visdmin

Labels

feature New feature or request

Projects

None yet

Milestone

release-0.1.0

Development

Successfully merging this pull request may close these issues.

1 participant

@visdmin