-
Notifications
You must be signed in to change notification settings - Fork 225
Integrate k8s events #48
Integrate k8s events #48
Conversation
abbe75a
to
5f8badd
Compare
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.
/lgtm
/hold
Hold for question about codegen
@@ -32,6 +32,9 @@ var _ = duck.VerifyType(&KubernetesEventSource{}, &duckv1alpha1.Conditions{}) | |||
|
|||
// KubernetesEventSourceSpec defines the desired state of the source. | |||
type KubernetesEventSourceSpec struct { | |||
// Namespace that we watch kubernetes events in. | |||
Namespace string `json:"namespace"` |
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.
Do you need to run update-codegen for this?
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 did, and there are no changes :) I was expecting some, but I guess the patching stuff is enough, a new field is added and tagged as required. Magic? 🤔
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: grantr, vaikas-google The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
TIL primitive fields don't show up in deepcopy |
Addresses #446
Proposed Changes
Release Note