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.

Sign up for GitHub

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

Jump to bottom

feat: Webhook creates Service, Service creates ServiceRun, ServiceRun creates Event #30

Merged
merged 20 commits into from
Sep 26, 2022
Merged

feat: Webhook creates Service, Service creates ServiceRun, ServiceRun creates Event #30

merged 20 commits into from
Sep 26, 2022

Conversation

odubajDT
Copy link
Contributor

No description provided.

thschue
thschue requested changes Sep 21, 2022
View reviewed changes
Copy link
Contributor

@thschue thschue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work, some small changes but then we're good to go ...

operator/controllers/service_controller.go Outdated Show resolved Hide resolved
operator/controllers/servicerun_controller.go Outdated Show resolved Hide resolved
operator/webhooks/pod_mutating_webhook.go Outdated Show resolved Hide resolved
thschue
thschue previously requested changes Sep 21, 2022
View reviewed changes
operator/webhooks/pod_mutating_webhook.go Outdated Show resolved Hide resolved
@odubajDT odubajDT changed the title ServiceController creates ServiceRun resource for each version change Webhook creates Service, Service creates ServiceRun, ServiceRun creates Event Sep 23, 2022
@odubajDT odubajDT changed the title Webhook creates Service, Service creates ServiceRun, ServiceRun creates Event feat: Webhook creates Service, Service creates ServiceRun, ServiceRun creates Event Sep 26, 2022
bacherfl
bacherfl previously approved these changes Sep 26, 2022
View reviewed changes
odubajDT and others added 8 commits September 26, 2022 11:06
@odubajDT
mutating webhook not working
83fc027 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
not working webhook
8f47556 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
debug
fa4d964 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@bacherfl @odubajDT
added index for spec.replicaSetUID for serviceRun CRD
d5eb992 
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@odubajDT
working version
07705a2 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
generation
9501d03 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
using ownerreference to service from serviceRun
0686238 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
update gitignore
15c562f 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
working with scheduler
2948e91 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
do not delete the job for now
0c9fb1b 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
servicecontroller logic back to webhook
8cd3837 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
back to webhook
72c7dc8 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
working version with webhook creating service
929f1f7 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
small comment
634e47f 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
pr review
ea13209 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
renaming owner
436ffc9 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
add changes to scheduler to make it deployable
007425e 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
minro polishing
ecbd2d4 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
minro fixes
f7d9da3 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
generate
23d2f52 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
thschue
thschue approved these changes Sep 26, 2022
View reviewed changes
Copy link
Contributor

@thschue thschue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@thschue thschue merged commit 5ae58c3 into main Sep 26, 2022
This was referenced Sep 29, 2022
Closed
Closed
Closed
Closed
Closed
Merged
Merged
@thschue thschue deleted the mutating-webhook-logic branch October 13, 2022 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thschue thschue thschue approved these changes

@bacherfl bacherfl bacherfl left review comments

@thisthat thisthat Awaiting requested review from thisthat

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@odubajDT @bacherfl @thschue