-
Notifications
You must be signed in to change notification settings - Fork 204
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
cOdiglet to use criwrapper to create ic #2015
base: main
Are you sure you want to change the base?
Changes from 30 commits
0d646fd
341dda6
10c8c79
5f879c1
59579f3
c410796
a666ecd
1f77020
6ab4aaa
68e3def
f4aa885
7a83e2b
4b37c04
3fef066
a071d5d
16a55ab
03bb7e7
b2978f7
f0259b0
6df6b2a
8eaa3bd
722a49e
8fb407c
1c14d62
2a7b28c
3262150
57ff741
68967e0
0d1f240
ace5022
19edc24
bdde925
3053147
673c395
8a410d3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -43,14 +43,25 @@ type OtherAgent struct { | |
Name string `json:"name,omitempty"` | ||
} | ||
|
||
type ProcessingState string | ||
|
||
const ( | ||
ProcessingStateFailed ProcessingState = "Failed" | ||
ProcessingStateSucceeded ProcessingState = "Succeeded" | ||
ProcessingStateSkipped ProcessingState = "Skipped" | ||
) | ||
|
||
// +kubebuilder:object:generate=true | ||
type RuntimeDetailsByContainer struct { | ||
ContainerName string `json:"containerName"` | ||
Language common.ProgrammingLanguage `json:"language"` | ||
RuntimeVersion string `json:"runtimeVersion,omitempty"` | ||
EnvVars []EnvVar `json:"envVars,omitempty"` | ||
OtherAgent *OtherAgent `json:"otherAgent,omitempty"` | ||
LibCType *common.LibCType `json:"libCType,omitempty"` | ||
ContainerName string `json:"containerName"` | ||
Language common.ProgrammingLanguage `json:"language"` | ||
RuntimeVersion string `json:"runtimeVersion,omitempty"` | ||
EnvVars []EnvVar `json:"envVars,omitempty"` | ||
OtherAgent *OtherAgent `json:"otherAgent,omitempty"` | ||
LibCType *common.LibCType `json:"libCType,omitempty"` | ||
CriErrorMessage *string `json:"criErrorMessage,omitempty"` | ||
EnvFromContainerRuntime []EnvVar `json:"envFromContainerRuntime,omitempty"` | ||
RuntimeUpdateState *ProcessingState `json:"runtimeUpdateState,omitempty"` // Tracks whether the new runtime detection process has been executed. If empty, the process has not been executed. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does it make sense to save this per workload and not per container? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add a comment that this property is temporary and only used for the migration and showed be removed after some time |
||
} | ||
|
||
// +kubebuilder:object:generate=true | ||
|
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
isnt status a better place for error messages instead of spec?