Skip to content

Updates to the controller logic to better handle failures in etc updates #424

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

Merged
merged 26 commits into from
Jul 12, 2023
Merged

Updates to the controller logic to better handle failures in etc updates #424

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
6274768
Updates to the controller logic (wip)
z103cb Jun 20, 2023
80b0b49
Added retry logic for etc errors.
z103cb Jun 22, 2023
24709ee
Fix small race condition
z103cb Jun 23, 2023
4f9f949
Preemption code updates.
z103cb Jun 23, 2023
cc5c337
Small locking issue fixed.
z103cb Jun 23, 2023
feafd15
Fixed failed test.
z103cb Jun 27, 2023
ef5c6a8
Removed focused test.
z103cb Jun 27, 2023
8763ebe
Bug fixes and CRD documentation updates.
z103cb Jun 28, 2023
2f04bfa
Merge main and resolve conflicts
z103cb Jun 28, 2023
ef9af3f
Fixed misspelling
z103cb Jun 28, 2023
2edc4f2
Fix bad merge
z103cb Jun 28, 2023
ed9227b
E2E test improvements
z103cb Jun 30, 2023
7b47bef
Removed test focus
z103cb Jun 30, 2023
e5bd50d
Merge branch 'main' into z103cb/issue297
z103cb Jul 3, 2023
58a4f94
Fixed a few bugs with preemption.
z103cb Jul 3, 2023
641a343
Testcase updates
z103cb Jul 3, 2023
76e829c
Added more messages to the e2e tests.
z103cb Jul 4, 2023
f0e55b2
Error handling and messages updates.
z103cb Jul 5, 2023
5736472
Addressed PR review comments
z103cb Jul 5, 2023
b47716b
Fixed failing test.
z103cb Jul 6, 2023
0ff4d32
Fixed bugs in the manage queque job
z103cb Jul 10, 2023
444a970
Merge remote-tracking branch 'origin/main' into z103cb/issue297
z103cb Jul 10, 2023
4ce6bfe
Fixed Merges issues
z103cb Jul 11, 2023
890dade
fix test
asm582 Jul 11, 2023
6b0461a
remove Fit
asm582 Jul 11, 2023
97f8285
bump build time
asm582 Jul 11, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 10 additions & 8 deletions config/crd/bases/mcad.ibm.com_appwrappers.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@

---
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
Expand Down Expand Up @@ -778,6 +776,10 @@ spec:
QueueJob (by Informer)
format: date-time
type: string
controllerfirstdispatchtimestamp:
description: Microsecond level timestamp when controller first dispatches appwrapper
format: date-time
type: string
failed:
description: The number of resources which reached phase Failed.
format: int32
Expand All @@ -790,8 +792,7 @@ spec:
description: Is Dispatched?
type: boolean
local:
description: Indicate if message is a duplicate (for Informer to recognize
duplicate messages)
description: Indicate if message is a duplicate (for Informer to recognize duplicate messages)
type: boolean
message:
type: string
Expand All @@ -800,15 +801,13 @@ spec:
format: int32
type: integer
queuejobstate:
description: State of QueueJob - Init, Queueing, HeadOfLine, Rejoining,
...
description: State of QueueJob - Init, Queueing, HeadOfLine, Rejoining ...
type: string
running:
format: int32
type: integer
sender:
description: Indicate sender of this message (extremely useful for
debugging)
description: Indicate sender of this message (extremely useful for debugging)
type: string
state:
description: State - Pending, Running, Failed, Deleted
Expand All @@ -828,3 +827,6 @@ spec:
type: object
served: true
storage: true
subresources:
status: {}

18 changes: 10 additions & 8 deletions deployment/mcad-controller/crds/mcad.ibm.com_appwrappers.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@

---
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
Expand Down Expand Up @@ -778,6 +776,10 @@ spec:
QueueJob (by Informer)
format: date-time
type: string
controllerfirstdispatchtimestamp:
description: Microsecond level timestamp when controller first dispatches appwrapper
format: date-time
type: string
failed:
description: The number of resources which reached phase Failed.
format: int32
Expand All @@ -790,8 +792,7 @@ spec:
description: Is Dispatched?
type: boolean
local:
description: Indicate if message is a duplicate (for Informer to recognize
duplicate messages)
description: Indicate if message is a duplicate (for Informer to recognize duplicate messages)
type: boolean
message:
type: string
Expand All @@ -800,15 +801,13 @@ spec:
format: int32
type: integer
queuejobstate:
description: State of QueueJob - Init, Queueing, HeadOfLine, Rejoining,
...
description: State of QueueJob - Init, Queueing, HeadOfLine, Rejoining ...
type: string
running:
format: int32
type: integer
sender:
description: Indicate sender of this message (extremely useful for
debugging)
description: Indicate sender of this message (extremely useful for debugging)
type: string
state:
description: State - Pending, Running, Failed, Deleted
Expand All @@ -828,3 +827,6 @@ spec:
type: object
served: true
storage: true
subresources:
status: {}

Loading