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

Use Kubernetes native config options instead of etcd #81

Open
meiserloh opened this issue Oct 2, 2024 · 0 comments
Open

Use Kubernetes native config options instead of etcd #81

meiserloh opened this issue Oct 2, 2024 · 0 comments

Comments

@meiserloh
Copy link
Contributor

No description provided.

alexander-dammeier added a commit that referenced this issue Oct 4, 2024
alexander-dammeier added a commit that referenced this issue Oct 4, 2024
alexander-dammeier added a commit that referenced this issue Oct 4, 2024
alexander-dammeier added a commit that referenced this issue Oct 4, 2024
The hash is needed as the dogu-op v2 violates
the go standards because it did not contain the
v2 version in the module name. Therefore an
update via 'go mod' or 'go get' is not
possible.

As the new dogu-op version do not contain the
dogu encryption functions anymore, we needed
to strip the code from encryptionAdapter.go
and delete the tests. This is no problem as
this code should be removed anyway because
we do not want to encrypt sensitive dogu config ourselves anymore.
alexander-dammeier added a commit that referenced this issue Oct 4, 2024
this fixes further problems with the
k8s.io/client-go update
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
We need a wrapper because of our specific errors
in the blueprint operator. Maybe we can introduce
them in the registry-lib as well, so we can
completely remove this wrapper adapter.

For now, the plan is to replace the old etcd
adapter step by step with the new adapter.
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
We need a wrapper because of our specific errors
in the blueprint operator. Maybe we can introduce
them in the registry-lib as well, so we can
completely remove this wrapper adapter.

For now, the plan is to replace the old etcd
adapter step by step with the new adapter.
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
headers will be sent at the write call already.
Calling writeHeader cannot change the status code
afterward.
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
just use SensitiveDoguConfigEntryRepo directly
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
just use SensitiveDoguConfigEntryRepo directly
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
It was unused and it should also not be used as
it hides conflict errors which we need to handle
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
with the current mockery config they will not be
updated automatically with 'make mocks'. I needed
to copy the interface into another package to
generate the mocks. Then I could copy them on the
right place.
meiserloh added a commit that referenced this issue Oct 7, 2024
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
Introduce new types so that we can
refactor the operator step by step.
alexander-dammeier added a commit that referenced this issue Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant