-
Notifications
You must be signed in to change notification settings - Fork 4
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
Init #1
Init #1
Conversation
go.mod
Outdated
gopkg.in/yaml.v3 v3.0.1 // indirect | ||
k8s.io/api v0.30.1 // indirect | ||
k8s.io/apiextensions-apiserver v0.30.1 // indirect | ||
k8s.io/apiserver v0.30.1 // indirect |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we already bump k8s to 1.31 so that we won't need to upgrade it later on?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AFAICT the latest operator-sdk is still on k8s 1.30 :/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But we still can bump it manually
.golangci.yml
Outdated
@@ -0,0 +1,47 @@ | |||
run: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this can be deleted
LICENSE
Outdated
@@ -1,201 +0,0 @@ | |||
Apache License |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you delete this? We need to have License file in repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, result of not rebasing. Will fix
PROJECT
Outdated
manifests.sdk.operatorframework.io/v2: {} | ||
scorecard.sdk.operatorframework.io/v2: {} | ||
projectName: ksan-operator | ||
repo: github.com/copejon/ksan-operator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong repo path
PROJECT
Outdated
crdVersion: v1 | ||
namespaced: true | ||
controller: true | ||
domain: ksan-operator.storage.openshift.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
controller-gen.kubebuilder.io/version: v0.15.0 | ||
name: ksanclusters.ksan.ksan-operator.storage.openshift.io | ||
spec: | ||
group: ksan.ksan-operator.storage.openshift.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in doc we have ksan.openshift.io
OWNERS
Outdated
- jeff-roche | ||
- jerpeter1 | ||
- pacevedom | ||
- pmtk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i don't think that we need that much people, especially for approvers
Initialize the operator framework and a primitive api