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

Refactor for v2 #103

Merged
merged 51 commits into from
Jul 10, 2020
Merged

Refactor for v2 #103

merged 51 commits into from
Jul 10, 2020

Conversation

slok
Copy link
Collaborator

@slok slok commented Jul 10, 2020

From the changelong:

  • Refactor controller package.
  • Refactor metrics package.
  • Refactor log package.
  • Remove operator concept and remove CRD initialization in favor of using only
    controllers and let the CRD initialization outside Kooper (e.g CRD yaml).
  • Default resync time to 3 minutes.
  • Default workers to 3.
  • Disable retry handling on controllers in case of error by default.
  • Remove tracing.
  • Minimum Go version v1.13 (error wrapping required).
  • Refactor Logger with structured logging.
  • Add Logrus helper wrapper.
  • Refactor to simplify the retrievers.
  • Refactor metrics recorder implementation including the prometheus backend.
  • Refactor internal controller queue into a decorator implementation approach.
  • Remove Delete method from controller.Handler and simplify to only Handle method
  • Add DisableResync flag on controller configuration to disable the resync of all resources.

slok and others added 30 commits March 18, 2020 11:21
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Remove Operator and CRD related code
Split controller processor logic into an internal type
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Refactor retriever and add multiretriever
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Refactor metrics and the internal queue
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Remove Delete event concept from Kooper
Don't measure queue events on queue shutdown
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Add multiple Kubernetes versions on integration tests
slok and others added 21 commits April 13, 2020 10:16
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Update Kooper for Kubernetes v1.17
…nality using multiple controllers

Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
Co-authored-by: Sergio Ballesteros <[email protected]>
Signed-off-by: Xabier Larrakoetxea <[email protected]>
@slok slok requested a review from ese as a code owner July 10, 2020 16:13
@slok slok merged commit 81a35ab into master Jul 10, 2020
@slok slok deleted the refactor branch July 10, 2020 16:44
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

Successfully merging this pull request may close these issues.

3 participants