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: substitute event task #43

Merged
merged 41 commits into from
Sep 29, 2022
Merged

feat: substitute event task #43

merged 41 commits into from
Sep 29, 2022

Conversation

odubajDT
Copy link
Contributor

@odubajDT odubajDT commented Sep 27, 2022 •
edited by thisthat
Loading

Closes #13

@thschue thschue changed the title Feat/substitute event task feat: substitute event task Sep 28, 2022
@odubajDT odubajDT marked this pull request as ready for review September 29, 2022 07:30
thisthat
thisthat reviewed Sep 29, 2022
View reviewed changes
lfc-scheduler/Makefile Show resolved Hide resolved
lfc-scheduler/Makefile Show resolved Hide resolved
lfc-scheduler/Makefile Outdated Show resolved Hide resolved
lfc-scheduler/pkg/klcpermit/workflow_manager.go Show resolved Hide resolved
operator/api/v1alpha1/keptntask_types.go Outdated Show resolved Hide resolved
operator/controllers/keptnworkloadinstance/reconcile_predeployment.go Show resolved Hide resolved
operator/main.go Outdated Show resolved Hide resolved
Thomas Schuetz and others added 14 commits September 29, 2022 14:48
@odubajDT
feat: change task execution
4c1c71d 
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
@odubajDT
first skeleton
08e2564 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
polish
6601df0 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
first version
0772e0e 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
not working
15f4cac 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
push
642c34d 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@RealAnna @odubajDT
feat: initialised logger and recorder
a15fd4a 
Signed-off-by: RealAnna <anna.reale@dynatrace.com>
@odubajDT
chore: unified status entries
394ebc8 
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
@RealAnna @odubajDT
feat: adde fiew fixes
698a3d4 
Signed-off-by: RealAnna <anna.reale@dynatrace.com>
@odubajDT
feat: current version (tasks are triggered, order is not correct atm)
126954d 
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
@odubajDT
fix: update after rebase
74083c3 
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
@odubajDT
feat: fixed empty pre-deployment state case
1b6bcd3 
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
@odubajDT
feat: made workload instance output prettier
0ac6b41 
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
@RealAnna @odubajDT
feat: adde fiew fixes
a77b358 
Signed-off-by: RealAnna <anna.reale@dynatrace.com>
odubajDT and others added 15 commits September 29, 2022 14:49
@odubajDT
pipeline fix
d55f9ad 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
emove event controller
852d33b 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
remove buildx
961b878 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
pr review
a240b8e 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
pr review
97f4ff0 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
limit size of job name
28d4426 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
refactor
b658b49 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
forgotten change
590ec89 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
pr review
48d1811 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
generate
f8438b2 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
pr review
7793e51 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@bacherfl @odubajDT
fix json paths
b03c7a7 
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@odubajDT
move deployment to samples
972dc4d 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
deny kube namespaces
fd712c9 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
pr review
26651b8 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
bacherfl
bacherfl previously approved these changes Sep 29, 2022
View reviewed changes
@odubajDT
fix build
72f3cab 
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
thschue
thschue approved these changes Sep 29, 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! lgtm

@odubajDT odubajDT merged commit 3644a7d into main Sep 29, 2022
This was referenced Sep 29, 2022
Closed
Closed
Closed
Closed
Closed
Merged
Merged
@thschue thschue deleted the feat/substitute-event-task 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

@thisthat thisthat thisthat left review comments

@thschue thschue thschue approved these changes

@bacherfl bacherfl bacherfl approved these changes

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

Successfully merging this pull request may close these issues.

Event Controller triggers K8s Jobs
5 participants
@odubajDT @thisthat @bacherfl @thschue @RealAnna