Skip to content
This repository has been archived by the owner on Jun 1, 2021. It is now read-only.

Operator #96

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Operator #96

wants to merge 29 commits into from

Conversation

VaibhavPage
Copy link
Contributor

@VaibhavPage VaibhavPage commented Dec 4, 2019

closes #30

@VaibhavPage VaibhavPage mentioned this pull request Dec 9, 2019
8 tasks
pkg/client/ocibuilder/clientset/versioned/fake/register.go Outdated Show resolved Hide resolved
controllers/operator.go Outdated Show resolved Hide resolved
ocictl/pkg/utils/utils.go Outdated Show resolved Hide resolved
controllers/resource.go Outdated Show resolved Hide resolved
controllers/resource.go Outdated Show resolved Hide resolved
@artbegolli
Copy link
Collaborator

artbegolli commented Feb 23, 2020

Pipeline passing few things left to test and do before this can be merged:

  • adding an update/error phase
  • operator main.go, creates NewController and calls Run
  • Makefile command for building the operator
  • ConfigMap which tells the controller namespace to watch, instance id, label ID etc.

@artbegolli
Copy link
Collaborator

@VaibhavPage added main.go and Makefile, take a look. We can chat tomorrow about how to handle the ConfigMap

@artbegolli
Copy link
Collaborator

After manual resync, call ‘go watchConfigmap’

watchControllerConfigMap is being called in ctrl.Run(), is there need to call it again in a manual resync?

@artbegolli artbegolli marked this pull request as ready for review March 4, 2020 09:41
@artbegolli
Copy link
Collaborator

This is dependent on #154

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement K8s operator
3 participants